Thread: a small program

I wrot a code for the following program but it seem doesn't work right Please show me my mistakes, Thanks

A program that reads in the numerators and denominators of two fractions. The program should print the product of the two fractions as fraction and as a percent.

my code is
[cod]

#include <iostream>
using namespace std;


int main()

{
int Fn ,Fd, Sn, Sd;

int FracAns=0;
float PercAns =0;

cout<<"enter the numerator and the denomenator for the first fraction"<<endl;
cin>>Fn;
cin>>Fd;
cout<<"Enter the numerator and the denomentaor"<<endl;
cout<<"for the second fraction"<<endl;
cin>>Sn;
cin>>Fd;
if (Fd || Sd==0)
{

cout<<"You cant not divide by zero, try again"<<endl;
}

FracAns = Fn*Sn / Fd*Sd ;
PercAns = FracAns / 100;
cout<< "the procuct of the two fraction as a fraction is: "<<endl;
cout<<FracAns<<endl;
cout<< " as percentage is "<<PercAns<<endl;

return 0 ;
}

[\code]

Help would be apprciated.

Hes not adding fractions hes dividing them. 1/4 divided by 0/4 is what? If you were adding then you would have to do

if((Fd*Sd) == 0)

but that wasnt the case.

FracAns = Fn*Sn / Fd*Sd ;
PercAns = FracAns / 100;

he is infact trying to divide the fractions... the reason I put

if ((Fd*Fn*Sn*Sd) == 0)

Was to check if any of the numbers are equal to zero If they were zero then stop the program ... ANYTHING TIMES 0 is 0 right?

Either my math really sucks or yours does.

Thank you guys for all the answers, I appreciate them all.
I think I made a typo when I wrote Sd in the second fraction it should be Sd.
I fixed the typo and I put Sd instead of Fd and I ran the program but all what I get as result is 0.
I just need the product of two fraction as
1) fraction like m/n * k/l = d/b and as
2) percetage like .02

it legal if you use 0/m the result will be zero.
but it is illegal to use m/0 because it is undefined.
I wanted to make a condition that it is illegal to use 0 in either fraction as a denominator.
and if the test is true( which means the den. is 0) I should return to the beginning of the program but not exit it.

Thanks again and waiting for any suggestion.

I agree with SilentStrike.
D-Man, you can say this if ((Fd*Sd*Sn) == 0)
Because first numerators can be 0.
Absy,
I think you can make a class for the fraction, where yo can make a function to simplify, and you can overload the stream extraction and stream insertion operator, which will help you organize the input and output, and you can also overload the mathimatical operators...
Try it, it's not that hard
your program will be much better.

Originally posted by Absy

it legal if you use 0/m the result will be zero.
but it is illegal to use m/0 because it is undefined.
I wanted to make a condition that it is illegal to use 0 in either fraction as a denominator.
and if the test is true( which means the den. is 0) I should return to the beginning of the program but not exit it.

Thanks again and waiting for any suggestion. [/B]

Blah. And by that I meant:

Code:

do
{
     cout << "Enter the numerator and denominator for the first fraction." << endl;
     // etc.
}
while(Fd == 0 || Sd == 0);

If you like something a little less compact, but nicer:

Code:

cout << "Enter the numerator and denominator for the first fraction." << endl;
     // etc.
while(Fd == 0 || Sd == 0)
{
     cout << "You can't divide by zero.  Try again." << endl;
     cout << "Enter the numerator and denominator for the first fraction." << endl;
     // etc.

}

And one very important mistake, which someone pointed out already, was that you can't do if(Fd || Sd == 0)...these compund statements are not like in English, where you can say, "Susy and Jim went to the mall." In C++ you have to say, "Susy went to the mall and Jim went to the mall." You can't compare more than one value at once. Hope I helped some.

Hi everybody.
Please read the requirement very carefully. The program I want is not that complicated it should be easy but I'm making a mistake somewhere and I'm not getting the result I want the program is;

A program that reads in the numerators and denominators of two fractions. The program should print the product of the two fractions as fraction and as a percent.


my code is

Code:

#include <iostream>
using namespace std;


int main()

{
	int Fn ,Fd, Sn, Sd;
	
	int FracAns;
	float PercAns;

	cout<<"enter the numerator and the denomenator for the first fraction"<<endl;
	cin>>Fn;
	cin>>Fd;
	cout<<"Enter the numerator and the denomentaor"<<endl;
	cout<<"for the second fraction"<<endl;
	cin>>Sn;
	cin>>Fd;
	if (Fd==0 || Sd==0)
		{
		
			cout<<"You cant not divide by zero, try again"<<endl;
		}
	else

		FracAns =  Fn*Sn / Fd*Sd ;
		PercAns = FracAns / 100;
	cout<< "the procuct of the two fraction as a fraction is: "<<endl;
		cout<<FracAns<<endl;
	cout<< " as percentage is "<<PercAns<<endl;

	return ;
}

//I hope I'm using the right code tags here.

my program does not getting the result I want either for math. problem or for programming one please I just want to know what is wrong I'm doing. I'm just doing that for fun and as a chanllenge for me and for everybody in this beautiful forum. Thanks.

Uh, part of the proble might be that you haven't taken a lot of the advice given. First off, FracAns in your program will wind up as a decimal, not a fraction. If you make FracAns a float instead of an int, that is, and to do that you would also have to make Fd and Sd floats. Is this what you want to do? If PercAns is 50, for example, would you want FracAns to be .5 or 1/2? Because you're not doing either correctly.

Another problem that you talked about earlier, and that I offered a solution to earlier, was this:

Code:

if (Fd==0 || Sd==0)
		{
		
			cout<<"You cant not divide by zero, try again"<<endl;

This will not allow the user to enter valid values for Fd and Sd. Loops are you friends.

Another thing is that you need brackets around just about everything after else.

A typo that someone else pointed out earlier, you have not corrected. It's here: cin>>Sn; cin>>Fd; That's a bit of a problem, now isn't it? Not only is Fd not the value you intended, but you are performing operations on Sd without ever having given it a value.

One more thing that has been pointed out to you already is this: FracAns = Fn*Sn / Fd*Sd ; You should put parenthesis around Fn*Sn and Fd*Sd.

Now, next time, could you please listen to the suggestions people already gave you before asking for more?