Will someone tell me how to handle only input between -1 and 100.?

[naushil mehta]

This is the comment from my instructor,thanks

-1 pt for readArray() handles the out of range data (input <-1) as well as(input >100) the same valid data!
when program encounters the out of range data ( input <-1 || input >100), program prompt user “wrong input, and enter again”. Do not enter the outof range (invalid data) to the score[] array.

-1 pt  for readArray() handles the out of range data (input <-1) as well as(input >100) the same valid data!
when program encounters the out of range data ( input <-1 || input >100), program prompt user “wrong input, and enter 
again”. Do not enter the out of range (invalid data) to the score[] array..
// Lab 4
// Programmer: Naushil Mehta
// Editor(s) used: win 7 notepad
// Compiler(s) used: Visual Studio 2008
//

#include <iostream>
using std::cin;
using std::cout;
using std::endl;

int readArray(int, int[]);//implement step 2 of 5

int avg(int , const int[]); //implement step 3 of 5

int stat(int, const int[], int&, int&, int&); //implement step 4 of 5

int histogram(int,const int[], int[]);//implement step 5 of 5

int main()
{
const int MAX_SCORES = 100;
int score[MAX_SCORES];
//cout<< readArray(MAX_SCORES,score)<<endl;
int nScore = readArray(MAX_SCORES,score);
cout<<"Average of the array: "<<avg(nScore, score) <<endl;
//step 4 of 5
int avgScore, minScore, maxScore;
	
int grade[5]={0};

if (stat( nScore, score, avgScore, minScore, maxScore) == 0)
{
cout<<"Average = "<< avgScore << endl
<<"Max = "<<maxScore <<endl
<<"Min = "<<minScore <<endl;
//step 5 of 5
histogram(nScore,score,grade);
	
cout <<"As: "<< grade[0]<<endl;
cout <<"Bs: "<< grade[1]<<endl;
cout <<"Cs: "<< grade[2]<<endl;
cout <<"Ds: "<< grade[3]<<endl;
cout <<"Fs: "<< grade[4]<<endl;
}
else
{
cout << "no data"<<endl;
}
cin.get();
cin.get();
return 0;
}

int readArray(int max_score, int score[])
{
int nScores =0;
for(int i =0; i< max_score; i++)
{
cout<<"Enter element(-1 to end input): ";
cin >> score[i];
cout<<endl;

if (score[i] == -1)
{
score[i] = NULL;
break;
}
else
{
nScores++;
}
if(nScores>=max_score)
break;

cin.ignore();
}
return nScores;
}
int avg(int n, const int score[])
{
int total = 0;
if (n == 0)
return 0;

for(int i = 0; i < n; i++)
{
total += score[i];
}
return total/n;
}
int stat(int n, const int score[], int& avg, int& mn, int& mx)
{
if (n == 0)
return 1;
int total = 0;
for(int i = 0; i < n; i++)
{
total =total + score[i];
if (( i == 0) || (mn > score[i]))
mn = score[i];
if ((i == 0) || (mx < score[i]) )
mx = score[i];
}
avg = total / n;
return 0;

}
int histogram(int nScore,const int score[], int grade[])
{
if(sizeof(score)/sizeof(int)==0)
return 1;
for(int i = 0; i < nScore; i++)
{
if (score[i] >= 90)
{
grade[0]++;
}
else if(score[i] >= 80)
{
grade[1] ++;
}
else if(score[i] >= 70)
{
grade[2] ++;
}
else if(score[i] >= 60)
{
grade[3] ++;
}
else
{
grade[4] ++;
}
}
return 0;
}

 

[gwowen]

I'm not going to do you homework for you, either... but here are a few
hints on how to think about the problem.

int readArray(int max_score, int score[])
{
        int nScores =0;
        for(int i =0; i< max_score; i++)
        {
                cout<<"Enter element(-1 to end input): ";
                cin >> score;
                cout<<endl;

Here, you read straight into the array - but if the input is invalid
you don't want to store it in the array, do you?

Perhaps you could store it somewhere else temporarily, and only put it
in the array once you've determined if it is valid.

                if (score == -1)
                {
                        score = NULL;

NULL is a pointer value. You don't want to store a pointer value as if
it were an integer.

                        break;
                }
                else
                {
                        nScores++;

Here you're incrementing the number of valid scores you've stored.
Looks like a good place to check the validity of the score before
storing it.

 

[Christopher Pisz]

This is the comment from my instructor,thanks

You've already posted your homework assignment 4 times in separate
threads. You would probably get more help if you:

1) Posted one thread for one topic
2) Gave the assignment your best shot yourself
3) Asked a _specific_ question about what you are having trouble with
after #2