Will this code work?

here is the problem:

Allenton Water Department wants a program that calculates a customers monthly water bill. The clerk will enter the current and previous meter readings. The program should calculate and display the number of gallons of water used and the total charge for the water. The charge for water is $7 per 1000 gallons. However, there is a minimum charge of $16.67. ( in other words, every customer must pay at least $16.67)


does this code make sense?


//ch5 - 13.cpp : calculates and displays number of gallons & total charge for the water
//created/revised by <> on <8-18-10>

#include <iostream>
using namespace std;

int main()
{
//declare variables
float previousReadings = 0.0;
float currentReadings = 0.0;
float gallonsUsed = 0.0;
float totalCharge = 0.0;

//enter input items
cout <<"Enter the current readings:";
cin >> currentReadings;
cout <<"Enter the previous readings:";
cin >> previousReadings;

//calculates the gallons used
gallonsUsed = currentReadings - previousReadings;

//calculates the total charge
if (totalCharge >= 16.67)
{
totalCharge = (gallonsUsed / 1000.0) *7.0;
cout <<"totalCharge:" << totalCharge << endl;
}
else
{
totalCharge = 16.67;
cout <<"total charge:" << totalCharge << endl;
}
//end if

cout <<"gallons used:" << gallonsUsed << endl;
cout <<"total charge:" << totalCharge << endl;

return 0;
} //end of main function

you forgot { } brackets in your if but it should compile once you fixed that.

If it does what your question was I dont know I didnt read that I'm not going to do your homework the forum policy even states we dont do your homework

Edit: Oh and there's some other stuff wrong too but rather than telling you what it is I prefer to spend my time saying that there's something more wrong with your code and that I prefer to say this

thanks so far. could i get a hint as to what else would be wrong? like is it the if statement, or does if have to do with my logic when trying to solve the problem?


by the way, im not asking you or anyone to do my homework that is not the reason i started this thread.

use floats instead of ints

ade161, out of curiosity has your instructor taught how to step through and debug a program?

What do you think the value of totalCharge is when you execute this line of code?


if (totalCharge <= 16.67)

Quote:

---

Originally Posted by Arjay

ade161, out of curiosity has your instructor taught how to step through and debug a program?

---

not yet.

i am studying ahead.

Quote:

---

Originally Posted by gcdef

what do you think the value of totalcharge is when you execute this line of code?


If (totalcharge <= 16.67)

---

16.67

:D it is *\(0.0)/*

Quote:

---

Originally Posted by ade161

16.67

---

Why would you think that? And if you already know its value at that point, why use an if statement?

Quote:

---

Originally Posted by GCDEF

Why would you think that? And if you already know its value at that point, why use an if statement?

---

because they have to be charged 16.67 no matter what.

Quote:

---

Originally Posted by TheComputer

:D it is *\(0.0)/*

---

what are you indicating?

Hello, ade buddy :wave:

As the minimal totalcharge is 16.67, formula to calculate totalcharge should come first before the conditional check against the minimum value. Other than that, your program looks fine to me

Quote:

---

Originally Posted by Pocoya

Hello, ade buddy :wave:

As the minimal totalcharge is 16.67, formula to calculate totalcharge should come first before the conditional check against the minimum value. Other than that, your program looks fine to me

---

how does this look?

//ch5 - 13.cpp : calculates and displays number of gallons & total charge for the water
//created/revised by <> on <8-18-10>

#include <iostream>
using namespace std;

int main()
{
//declare variables
float previousReadings = 0.0;
float currentReadings = 0.0;
float gallonsUsed = 0.0;
float totalCharge = 0.0;

//enter input items
cout <<"Enter the current readings:";
cin >> currentReadings;
cout <<"Enter the previous readings:";
cin >> previousReadings;

//calculates the gallons used
gallonsUsed = currentReadings - previousReadings;

//calculates the total charge
if (totalCharge >= 16.67)
{
totalCharge = (gallonsUsed / 1000.0) *7.0;
cout <<"totalCharge:" << totalCharge << endl;
}
else
{
totalCharge = 16.67;
cout <<"total charge:" << totalCharge << endl;
}
//end if

cout <<"gallons used:" << gallonsUsed << endl;
cout <<"total charge:" << totalCharge << endl;

return 0;
} //end of main function

Code:

---

//calculates the gallons used
gallonsUsed = currentReadings - previousReadings;
totalCharge = (gallonsUsed / 1000.0f) * 7.0f;

//calculates the total charge
if (totalCharge < 16.67f)
{
totalCharge = 16.67f;
}

cout <<"total charge:" << totalCharge << endl;
//end if

---