help with a function

[csharp]

I am trying to write a function to find the maximum, minimum and average of group of numbers entered by user.

I did max and average I have problem with finding min number

  void readE(double & max, double & min, double & average)
{
	double number, count = 0, sum = 0;
	//max = 0; min = 0; average = 0;
	cout << "Gimme #'s to do sum Math: ";
	for (number; cin >> number; count++)
	{
		sum += number;
		
		if (number > max)
			max = number;
		if (max>min)
			min;
		
	}
	average = (sum / count);
	recover();
	if (count == 0) die("Input Failure");
	cout << "The largest # seen is: " << max << endl; 
	cout << "The smallest # seen is: " << min << endl;
	cout<<"The sum of the #'s seen is: " << average << endl;
}

[long double main]

Finding the minimum is the same as finding the maximum, just with the > flipped to a <.

[csharp]

it returns 0 !

[long double main]

It'll work if you enter a negative number. :)

To fix it, you'll need to either initialize min to a big number (say, std::numeric_limits<double>::max()) before the loop or put a if (count == 0) min = number; in your loop.

[csharp]

this also doesn't work

I have tried most of these workarounds

[long double main]

Okay, so what do you have so far?

[csharp]

it doesn't work if you have something like

9 8 7 6

max =9
min = 9
average=7.5

[long double main]

Can you post your code for that function?

[csharp]

it's in my first post

[long double main]

Works for me:

void readE(double & max, double & min, double & average)
{
    double number, count = 0, sum = 0;
    //max = -1e100; min = 1e100; average = 0;
    cout << "Gimme #'s to do sum Math: ";
    for (; cin >> number; count++)
    {
        if (count == 0)
        {
            min = number;
            max = number;
        }
        sum += number;
        
        if (number > max)
            max = number;
        if (number < min)
            min = number;
    }
    average = (sum / count);
    recover();
    if (count == 0) die("Input Failure");
    cout << "The largest # seen is: " << max << endl; 
    cout << "The smallest # seen is: " << min << endl;
    cout<<"The sum of the #'s seen is: " << average << endl;
}

[csharp]

Yes it does work with me

can you please tell comment on the steps so I know what going on

[long double main]

I only added these lines:

if (count == 0)
{
    min = number;
    max = number;
}
// ...
if (number < min)
    min = number;

Those last two lines are essentially my first post:

I wrote:

Finding the minimum is the same as finding the maximum, just with the > flipped to a <.

The first few lines set the min and max to the number the user entered, but only on the first iteration of the loop (when count is 0).
That fixes the problem with min becoming 0 using my second suggestion:

I wrote:

To fix it, you'll need to either [...] or put a if (count == 0) min = number; in your loop.