DaniWeb Forum Index > Software Development > C++

Noob: inputing 5 grades and averaging them

Please support our C++ advertiser: Programming Forums - DaniWeb Sister Site

•

Join Date: Oct 2009

Posts: 12

Reputation: kadji.kahn is an unknown quantity at this point

Solved Threads: 0

kadji.kahn kadji.kahn is offline

Offline

Newbie Poster

Noob: inputing 5 grades and averaging them

Oct 26th, 2009

I was asked "This program asks for a series of five test scores and calculates the average score in a function. The function should receive the total of the five scores as an argument and return the average of the five scores. The program should not accept scores less than zero or greater than one hundred. The output should look like the example below."

Here's my code. It works, but I think I could do WAY better. Could anobody nudge me in the right direction? I really need to check each input as is comes in, but can't figure out a way to do that.

BTW I'm really green with C++ so be gentle.

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
#include<iostream>
#include<string>
 
 
using namespace std;
 
 
int average(int a, int b, int c, int d, int e); // prototype average
 
 
 
int main()
{
 
 
	// declare variables
 
	int testscore = 0;
	int iaverage = 0;
	int f = 0;
	int g = 0;
	int h = 0;
	int i = 0;
	int j = 0;
 
	// get input
 
		cout << "Please enter 5 test scores ";
		cin >> f >> g >> h >> i >> j;
 
        if(f <= 0 || f > 100)
        {      
           cout << "Please enter a value between 0 - 100";
           cout << endl;       
        }         
        else
        {          
                   iaverage = average(f, g, h, i, j);
                   testscore = (f+g+h+i+j);
                   cout << "Total of all scores: " << testscore << endl;                  
                   cout << "Average score: " << iaverage << endl;
        }//end if-else
 
 
 return 0;
 
}//end main
 
int average(int a, int b, int c, int d, int e)
{
       int daverage;
       daverage = (a+b+c+d+e) / 5;
       return daverage;
} //end average
#include<iostream>
#include<string>


using namespace std;


int average(int a, int b, int c, int d, int e); // prototype average



int main()
{
  
	
	// declare variables
	
	int testscore = 0;
	int iaverage = 0;
	int f = 0;
	int g = 0;
	int h = 0;
	int i = 0;
	int j = 0;
	
	// get input
		
		cout << "Please enter 5 test scores ";
		cin >> f >> g >> h >> i >> j;

        if(f <= 0 || f > 100)
        {      
           cout << "Please enter a value between 0 - 100";
           cout << endl;       
        }         
        else
        {          
                   iaverage = average(f, g, h, i, j);
                   testscore = (f+g+h+i+j);
                   cout << "Total of all scores: " << testscore << endl;                  
                   cout << "Average score: " << iaverage << endl;
        }//end if-else
 
 
 return 0;
 
}//end main

int average(int a, int b, int c, int d, int e)
{
       int daverage;
       daverage = (a+b+c+d+e) / 5;
       return daverage;
} //end average

•

Join Date: Mar 2008

Posts: 678

Reputation: Sky Diploma is a splendid one to behold

Solved Threads: 101

Sky Diploma Sky Diploma is offline

Offline

Practically a Master Poster

Oct 26th, 2009

Do you know how to use arrays? If you are comfortable with arrays, try using them to replace the different mark variables.

Btw, this program doesn't check if the other variables apart from f, have been initialised to a value greater than 100 or lesser 0. Examine that you only are checking that for the variable 'f'.

the #include <string> is completely un-necessary!

1. Please Mark Your Thread as Solved After Getting Your Answers.
2. Please Use CODE TAGS .

•

Join Date: Aug 2009

Posts: 73

Reputation: dgr231 is on a distinguished road

Solved Threads: 7

dgr231

Offline

Junior Poster in Training

Oct 26th, 2009

Hello kadji.kahn,

I agree with everything that Sky Diploma said. In addition, you can shorten up your function a bit. This is what you currently have:

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
int average(int a, int b, int c, int d, int e)
{
       int daverage;
       daverage = (a+b+c+d+e) / 5;
       return daverage;
} //end average
int average(int a, int b, int c, int d, int e)
{
       int daverage;
       daverage = (a+b+c+d+e) / 5;
       return daverage;
} //end average

I would first suggest that you make your return type a double, unless you are okay with rounding to the nearest whole number. You can get rid of the daverage variable and simplify the function to the following:

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
double average(int a, int b, int c, int d, int e)
{
     return (a+b+c+d+e) / 5.0;
}
double average(int a, int b, int c, int d, int e)
{
     return (a+b+c+d+e) / 5.0;
}

If you decide to go the route that Sky Diploma suggested with arrays, the function wouldn't change drastically, you would just replace your arguements with an array of the correct size and replace a, b, c, d, and e with the appropriate array index.

I hope that helps a little.

-D

•

Join Date: Oct 2009

Posts: 12

Reputation: kadji.kahn is an unknown quantity at this point

Solved Threads: 0

kadji.kahn kadji.kahn is offline

Offline

Newbie Poster

Oct 26th, 2009

Thanks for the quick reply!

I haven't go to arrays yet, so I can't do that.

Any idea on how to analyze each input w/o excessive "if" statements?

•

Join Date: Oct 2009

Posts: 12

Reputation: kadji.kahn is an unknown quantity at this point

Solved Threads: 0

kadji.kahn kadji.kahn is offline

Offline

Newbie Poster

Oct 26th, 2009

I appreciate the info on shorting up my function! Thats great, I see why the other code is excessive.

•

Join Date: Mar 2008

Posts: 678

Reputation: Sky Diploma is a splendid one to behold

Solved Threads: 101

Sky Diploma Sky Diploma is offline

Offline

Practically a Master Poster

Oct 26th, 2009

well you could use the logical && to join up checking for all !! and use one if statement only!

1. Please Mark Your Thread as Solved After Getting Your Answers.
2. Please Use CODE TAGS .

•

Join Date: Oct 2009

Posts: 12

Reputation: kadji.kahn is an unknown quantity at this point

Solved Threads: 0

kadji.kahn kadji.kahn is offline

Offline

Newbie Poster

Oct 26th, 2009

Ha! While its a nasty looking if statement that does work.

Cheers!

I appreciate the info.

•

Join Date: Aug 2009

Posts: 73

Reputation: dgr231 is on a distinguished road

Solved Threads: 7

dgr231

Offline

Junior Poster in Training

Oct 26th, 2009

Also, keep in mind that an if statement will only check the condition once (unless it is in a loop). If you have studied while loops, they may serve your purpose a little better as a while loop will allow the user to continue to input incorrect data until the data meets the specifications.

•

Join Date: Oct 2009

Posts: 12

Reputation: kadji.kahn is an unknown quantity at this point

Solved Threads: 0

kadji.kahn kadji.kahn is offline

Offline

Newbie Poster

Oct 26th, 2009

you're input was great, I was at least able to clean some things up, making the program a little better.

Have a great monday!

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
#include<iostream>
 
using namespace std;
 
 
int average(int a, int b, int c, int d, int e); // prototype average
 
 
 
int main()
{
 
 
	// declare variables
 
	int testscore = 0;
	int iaverage = 0;
	int f = 0;
	int g = 0;
	int h = 0;
	int i = 0;
	int j = 0;
 
	// get input
 
 
 
     while(cin)
	 {
		cout << "Please enter 5 test scores ";
		cin >> f >> g >> h >> i >> j;
		cout << endl;
 
		 if(f <= 0 || f > 100 || g <= 0 || g > 100 || h <= 0 || h > 100 || i <= 0 || i > 100 || j <= 0 || j > 100)
        {      
           cout << "Please enter a value between 0 - 100";
           cout << endl;       
        }         
        else
        {          
                   iaverage = average(f, g, h, i, j);
                   testscore = (f+g+h+i+j);
                   cout << "Total of all scores: " << testscore << endl;                  
                   cout << "Average score: " << iaverage << endl;
			    return 0;
        }//end if-else
	 }
 
 
 
}//end main
 
int average(int a, int b, int c, int d, int e)
{
       return (a+b+c+d+e) / 5.0;
} //end average
#include<iostream>

using namespace std;


int average(int a, int b, int c, int d, int e); // prototype average



int main()
{
  
	
	// declare variables
	
	int testscore = 0;
	int iaverage = 0;
	int f = 0;
	int g = 0;
	int h = 0;
	int i = 0;
	int j = 0;
	
	// get input
		
		

     while(cin)
	 {
		cout << "Please enter 5 test scores ";
		cin >> f >> g >> h >> i >> j;
		cout << endl;

		 if(f <= 0 || f > 100 || g <= 0 || g > 100 || h <= 0 || h > 100 || i <= 0 || i > 100 || j <= 0 || j > 100)
        {      
           cout << "Please enter a value between 0 - 100";
           cout << endl;       
        }         
        else
        {          
                   iaverage = average(f, g, h, i, j);
                   testscore = (f+g+h+i+j);
                   cout << "Total of all scores: " << testscore << endl;                  
                   cout << "Average score: " << iaverage << endl;
			    return 0;
        }//end if-else
	 }
 

 
}//end main

int average(int a, int b, int c, int d, int e)
{
       return (a+b+c+d+e) / 5.0;
} //end average

•

Join Date: Oct 2008

Posts: 103

Reputation: cgeier is an unknown quantity at this point

Solved Threads: 14

cgeier

Offline

Junior Poster

#10

Oct 26th, 2009

It's undersirable (in my opinion) to make the user re-enter all five scores just because ONE of them was wrong. Ask for each score individually using a for (or while) loop. You can also use "switch-case" statements in place of "if/else if" statements.

Some considerations when writing a program include: execution time (not so important when first learning to program; more important later)--fewer lines of code do not necessarily mean less execution time, user experience --how easy is the program to use, what are it's limitations (what user input could cause it to crash), what are it's features, readability and maintainability--how easy is it for someone else to read, understand, and modify your code (ex: use descriptive variable names: score1,score2, score3 instead of: a,b,c). I'm sure you can come up with more, but these are a few to get you started.

You might consider making all your variables of type "double" to allow for non-integer values. ex: 94.5

Last edited by cgeier; Oct 26th, 2009 at 1:58 pm.