Thread: New assignment.

You really need to be consistent with indenting and spacing. Also, as a newcomer to the language, it's a good idea to put bracket around *every* condition. Here is a cleaned-up version of the code with comments on the remaining problems:

Code:

#include <iostream>

using namespace std;

int main()
{
	double 
		n, 
		count=0, 
		sum=0, 
		min=0, // No, set high. 
		max=100; // No, set low.
	for (;;)
	{
		if (cin.eof()) 
		{
			break;
		}	
		while (cin >> n)
		{
			if (n < min) 
			{
				min = n; 
			}
		/*
			This is wrong - doesn't depend on previous comparison.	
		*/
			else if (n > max)  
			{
				max = n;
			}
			if (n < 0 || n > 100)
			{
				cout << "out of range; ignored." << endl;
			}
			else 
			{
				++count;
				sum += n;
			/*
				This is wrong - calculation needed outside of loop.
				Also, 'range' is undeclared.
			*/				
				range = max-min;
			}
		}	
	}
/*
	Note: cast is unnecessary.
*/
	cout << "The average is " << float (sum)/count << endl;
	cout << "The range is " << range << endl;
}

Ok, how does this look:

Code:

#include <iostream>

using namespace std;

int main()
{
	double n, 	count=0, 	sum=0, min=100, max=0, range=0;
	for (;;)
	{
		if (cin.eof()) 
		{
			break;
		}	
		while (cin >> n)
		{
			if (n < min) 
			{
				min = n; 
			}
			if (n > max)  
			{
				max = n;
			}
			if (n < 0 || n > 100)
			{
				cout << "out of range; ignored." << endl;
			}
			else 
			{
				++count;
				sum += n;
				range = max-min;
			}
		}	
	}
cout << "The average is " << float (sum)/count << endl;
cout << "The range is " << range << endl;
}

You should still move the range calculation outside of the loop, and get rid of the cast in the printout.

Ok, move the range calculation out of loop, See below, am I right? Also, what do you mean "cast"

Code:

#include <iostream>

using namespace std;

int main()
{
	double n, 	count=0, 	sum=0, min=100, max=0, range=0;
	for (;;)
	{
		if (cin.eof()) 
		{
			break;
		}	
		while (cin >> n)
		{
			if (n < min) 
			{
				min = n; 
			}
			if (n > max)  
			{
				max = n;
			}
			if (n < 0 || n > 100)
			{
				cout << "out of range; ignored." << endl;
			}
			else 
			{
				++count;
				sum += n;
			}
		}	
	}
                                               range = max-min;

cout << "The average is " << float (sum)/count << endl;
cout << "The range is " << range << endl;
}

>> Ok, move the range calculation out of loop

Yes, out of the loop and far to the right. Is it really so hard for you to indent properly?

>> Also, what do you mean "cast"

As in "conversion to some other type". You are "casting" 'sum' to a float, which is unnecessary.

Otherwise, everything looks okay.

I am sorry about the indenting. WHen I paste, it, somehow it changes.

Ok. I am going to repost. can you try and compile it for me and let me know if it works?

I can't compile on the computer I am using right now. thank you soo very much

here goes:

Code:

#include <iostream>

using namespace std;

int main()
{
	double n, 	count=0, 	sum=0, min=100, max=0, range=0;
	for (;;)
	{
		if (cin.eof()) 
		{
			break;
		}	
		while (cin >> n)
		{
			if (n < min) 
			{
				min = n; 
			}
			if (n > max)  
			{
				max = n;
			}
			if (n < 0 || n > 100)
			{
				cout << "out of range; ignored." << endl;
			}
			else 
			{
				++count;
				sum += n;
			}
		}	
	}
                                range = max-min;

cout << "The average is " <<  (sum)/count << endl;
cout << "The range is " << range << endl;
}

Originally Posted by Titanguy

"Also, in your original code, count should be an integer.

In C/C++, floating point values (floats and doubles) are designed for numbers which can have decimal places in them. Integer types, on the other hand, are whole numbers that have no fractional or decimal parts to them.

When counting things, it usually makes sense to use an integer type. In your example, you can have 5, 10, 12, 28, or whatever numbers input, but you can't have 3.1, 8.6 or 9.2 -- you read numbers one at a time and can't read less than one, so you'll never have anything after the decimal point. So it makes more sense to count the numbers you've read using an integer type.

So in your code, use :

int count = 0;

Instead of making it a double.

It makes no difference for the stuff you're doing now, but you'll eventually hit cases where it does. Might as well get in the habit of doing it correctly now.