Programming Forums - View Single Post

The Dark · Apr 5th, 2006, 2:44 AM

Two problems I can see:
1. The one that is causing your output problem is this code:

	cout << areaOfPizza <<
	cout << " square inches, then at " << price<< " cents per square\n";

You have an extra cout in there, which is putting out the value of cout in hex.
Change this to either

(Toggle Plain Text)

	cout << areaOfPizza <<
	" square inches, then at " << price<< " cents per square\n";

or

(Toggle Plain Text)

	cout << areaOfPizza;
	cout << " square inches, then at " << price<< " cents per square\n";

this happens several time in the code.

2. This calculation:

(Toggle Plain Text)

double getAreaOfPizza( double P_I, int size )
{
	return (P_I*(size/2)*(size/2));
}

will perform the size/2 as integer calculations first (because of the brackets), this will lead to a wrong result when you use an odd number.
Change this to

(Toggle Plain Text)

double getAreaOfPizza( double P_I, int size )
{
	return (P_I*size/2*size/2);
}

or

(Toggle Plain Text)

double getAreaOfPizza( double P_I, int size )
{
	return (P_I*(size/2.0)*(size/2.0));
}

As a side note, I wouldn't expect an area calulation to have PI passed in as a variable. It aint gonna change much, so you could probably have it as a global constant or a local constant inside the area function.

Apr 5th, 2006, 2:44 AM	#2
The Dark Expert Programmer Join Date: Jun 2005 Posts: 852 Rep Power: 4	Two problems I can see: 1. The one that is causing your output problem is this code: (Toggle Plain Text) cout << areaOfPizza << cout << " square inches, then at " << price<< " cents per square\n"; cout << areaOfPizza << cout << " square inches, then at " << price<< " cents per square\n"; You have an extra cout in there, which is putting out the value of cout in hex. Change this to either (Toggle Plain Text) cout << areaOfPizza << " square inches, then at " << price<< " cents per square\n"; cout << areaOfPizza << " square inches, then at " << price<< " cents per square\n"; or (Toggle Plain Text) cout << areaOfPizza; cout << " square inches, then at " << price<< " cents per square\n"; cout << areaOfPizza; cout << " square inches, then at " << price<< " cents per square\n"; this happens several time in the code. 2. This calculation: (Toggle Plain Text) double getAreaOfPizza( double P_I, int size ) { return (P_I(size/2)(size/2)); } double getAreaOfPizza( double P_I, int size ) { return (P_I(size/2)(size/2)); } will perform the size/2 as integer calculations first (because of the brackets), this will lead to a wrong result when you use an odd number. Change this to (Toggle Plain Text) double getAreaOfPizza( double P_I, int size ) { return (P_Isize/2size/2); } double getAreaOfPizza( double P_I, int size ) { return (P_Isize/2size/2); } or (Toggle Plain Text) double getAreaOfPizza( double P_I, int size ) { return (P_I(size/2.0)(size/2.0)); } double getAreaOfPizza( double P_I, int size ) { return (P_I(size/2.0)(size/2.0)); } As a side note, I wouldn't expect an area calulation to have PI passed in as a variable. It aint gonna change much, so you could probably have it as a global constant or a local constant inside the area function.