Private, public

[Dkob1]

Need help to fix errors:

#include <iostream>
using namespace std;

class TWO
{
private: int age[5];
public:
//read data into array
void ReadData(int age[], int n);
//member to return the average of data in array age of object p
friend  int FindAverage(TWO p);
//member to find and return the max and min age
void FindMaxMin(int &maxAge, int& minAge, int age[], int n);
//display ages > age average
void DisplayAboveAverage(int ageAve, int age[], int n);
//display the max and min ages
void DisplayMaxMinAge(int maxAge, int minAge, int age[], int n);
//release all memory used by objects
~TWO() {}
};

int main()
{
int a[]; int n;
int maxAge;
int minAge;
int ageAve;
TWO x;

x.ReadData(a[],n);
FindAverage(x);
x.FindMaxMin(maxAge,minAge);
x.DisplayAboveAverage(ageAve);
x.DisplayMaxMinAge(maxAge, minAge);

return 0;
}

// class functions
void TWO :: ReadData(int x[], int n)
{
for(int i = 0; i < 5; ++i)
	{	cout << "Enter 5 ages: "; 
		cin >> x[i];
	}
}
void FindAverage(TWO p, int x[], int n)
{	int total = 0;
	for(int i = 0; i < 5; ++i)
	{	
		total += x[i];
	}

}
void TWO :: FindMaxMin(int &maxAge, int& minAge, int x[], int n)
{
	
	int maxAge = x[0];
	for(int i = 0; i < 5; ++i)
	if(maxAge < x[i])
		maxAge = x[1];
	int minAge = x[0];
	for(int i = 0; i < 5; ++i)
	if(minAge > x[i])
		minAge =  x[3];
}
void TWO :: DisplayAboveAverage(int ageAve, int x[], int n)
{
	int total = 0;
	for(int i = 0; i < 5; ++i)
	{	
		total += x[i];
		if(ageAve > total)
			cout << 
	}

}
void TWO :: DisplayMaxMinAge(int maxAge, int minAge, int x[], int n)
{
	int maxAge = x[0];
	for(int i = 0; i < 5; ++i)
	if(maxAge < x[i])
		cout << "Maximum age is " << x[1];
	int minAge = x[0];
	for(int i = 0; i < 5; ++i)
	if(minAge > x[i])
		cout <<  x[3];
}

Output should look like this:

Enter 5 ages: 22 27 19 17 25
  Ages above average: 27 25
  Maximum age is 25
  Minimum age is 17

[xenovia12]

error line 74.

void TWO :: DisplayMaxMinAge(int maxAge, int minAge, int x[], int n)
{
	int maxAge = x[0]; // redefinition
	for(int i = 0; i < 5; ++i)
	if(maxAge < x[i])
		cout << "Maximum age is " << x[1];
	int minAge = x[0]; // redefinition
	for(int i = 0; i < 5; ++i)
	if(minAge > x[i])
		cout <<  x[3];
}

line 24: int a[]; the size of an static array should be specified at the moment you declare it.

line 30,32,33,34: the function calls doesnt match with the function prototypes.

[N495t4r]

Declaration of unused variables/arrays
e.g int a[]; int n;

two few arguments for call to functions in 'class TWO'

the size of the array int X[] is unknown;

This should work fine
http://cpp.sh/32m3q

[Dkob1]

yea fixed some stuff but still not really working right.

[Dkob1]

The one @N495t4r showed that you said worked didn't work for me, gave me a ton of errors.

[TarikNeaj]

@N495t4r's code works Fine.

But you probably dont want to use global variables.

[Dkob1]

Oh, cause when I tried it on that site it worked fine and got this output:

Enter 5 ages: 22 27 19 17 25
Enter 5 ages: Enter 5 ages: Enter 5 ages: Enter 5 ages: 
AVERAGE: 22
Maximum age is 27
Minimum age is 17
Ages above Average: 27 25  

but when I tried on Visual Studio 2013 it had some errors like uninitialized or something like that. How do I make the "Enter 5 ages" part not come up 5 times though?
thanks

[TarikNeaj]

Just try setting ageAve to = 0 in the beginning.

The reason its asking 5 time is becuase of your loop

for(int i = 0; i < 5; ++i)
	{	cout << "Enter 5 ages: "; 
		cin >> x[i];
	}

You see here how this runs 5 times?

Isint that what you want though?