i need this program to find the average what is wrong with it?
# include <iostream>
using namespace std;
int main ()
{
int count, total, measure1, measure2, measure3;
total = 0;
for (count = 0;
count < -1;
count ++)
cout<< " Please enter the measurements " <<endl;
cin>> measure1;
cin>> measure2;
cin>> measure3;
{
total = (measure1 + measure2 + measure3) / total;
}
cout<< " The average is" <<total <<endl;
system( "pause");
return 0 ;
}
using namespace std;
int main ()
{
int count, total, measure1, measure2, measure3;
total = 0;
for (count = 0;
count < -1;
count ++)
cout<< " Please enter the measurements " <<endl;
cin>> measure1;
cin>> measure2;
cin>> measure3;
{
total = (measure1 + measure2 + measure3) / total;
}
cout<< " The average is" <<total <<endl;
system( "pause");
return 0 ;
}
You have created an infinite loop:
This loop will never end, because it starts with count = 0, and increases count by one for each iteration. Count will never become less than -1. Go look for a tutorial on for loops.
Actually I don't see why you have the loop at all. Try to remove the line above, and keep the cout line. My guess is that after removing the loop, the program will work. Try entering three numbers, and I think it will response with the average.
|
|
This loop will never end, because it starts with count = 0, and increases count by one for each iteration. Count will never become less than -1. Go look for a tutorial on for loops.
Actually I don't see why you have the loop at all. Try to remove the line above, and keep the cout line. My guess is that after removing the loop, the program will work. Try entering three numbers, and I think it will response with the average.
Last edited on
closed account (z05DSL3A)
It may even blow out a divide by zero exception on
total = (measure1 + measure2 + measure3) / total;
Oh. I didn't see that. The correct way to compute the average is of course to divide by 3, not total (which is zero).
I realized that you don't actually have an infinite loop. Instead you have a "loop" that is never executed because of the always false condition. The result is that you never see the "Please enter the measurements" output, but you can probably enter three numbers, and the program will try to calculate the average (and get division by zero).
The conclusion is still the same though: The loop is wrong and it makes no sense to have it there. Remove it, fix the division, and it will probably work.
The conclusion is still the same though: The loop is wrong and it makes no sense to have it there. Remove it, fix the division, and it will probably work.
Topic archived. No new replies allowed.