[honumike]

I have this code that is meant to determine how to charge a homeowner for a water bill by certain usage. The code is included and complies, but does not perform calculations. What am I not doing here? Code follows:

#include <iostream>
using namespace std;


void main()//should this be int main?
{
	int Amount;
	double Charge;
	cout << "Enter number of cubic feet \n ";
	cin >> Amount;
	cin. ignore();

	if (Amount <= 1000)// I had previously inserted braces after the ) and again after 15.00;
		Charge = 15.00;
	
	else (Amount <= 2000); // I had else if here at one point and it did not work
		Charge = .0175 * Amount;
	
	if (Amount <= 3000)
		Charge = .02 * Amount;
	
	else  Charge = 70.00; // There was a cin.get(); following this last else but it did not work

	
}

** Edit **

[no2pencil]

honumike, on 12 Nov, 2009 - 09:37 PM, said:

void main()//should this be int main?

Yes. But you'll also need to give it a return type because you are declaring it to return int, & so it will then need to.

In your code above, you gather the users input, & you calculate it. But what do you want it to do from there?

[honumike]

no2pencil, on 12 Nov, 2009 - 07:41 PM, said:

honumike, on 12 Nov, 2009 - 09:37 PM, said:

void main()//should this be int main?

Yes. But you'll also need to give it a return type because you are declaring it to return int, & so it will then need to.

In your code above, you gather the users input, & you calculate it. But what do you want it to do from there?

I wan't it to tell me which method of billing will be appropriate. It will be one of the four listed. I had the main as "int" but it did not work either.

[OliveOyl3471]

You want it to tell you...? You mean, you want something printed? Then you use cout:

#include <iostream>
using namespace std;


int main()//should this be int main?
{
	int Amount;
	double Charge;
	cout << "Enter number of cubic feet \n ";
	cin >> Amount;
	cin. ignore();

	if (Amount <= 1000)// I had previously inserted braces after the ) and again after 15.00;
		Charge = 15.00;
	
	else if (Amount <= 2000) // I had else if here at one point and it did not work 
//you were right to use else if, but remove the semicolon...and using curly braces {} is better 
//though it does work without them, as long as the else includes only one line
		Charge = .0175 * Amount;
	
	else if (Amount <= 3000)
		Charge = .02 * Amount;
	
	else  Charge = 70.00; // There was a cin.get(); following this last else but it did not work

	cout<<"Charge is: "<<Charge<<endl;
	system("Pause"); //pause to see output
	return 0; //because int main must return an int
}