Hi,
You just needed to include the summary statements IN the for loop. That and you should initialize your variables because not all compilers will initialize them to zero or what not. Also you might want to think about checking for divide by zero errors if they ever arise. But the main problem was not including the lines where you summarize in the for loop.
CODE
#include<iostream>
#include<string>
#include<iomanip>
using namespace std;
int main()
{
string name,team,posi;
int j;
// Always make sure you initialize your variables.
// In VC++ this will be flagged as an error for not initializing before using.
double avg = 0.0,atbats = 0.0,hits = 0.0,teamavg = 0.0,teamatbats = 0.0,teamhits = 0.0;
const int LOOP_LIMIT=3;
cout<<setiosflags(ios::fixed|ios::showpoint)<<setprecision(3);
cout<<"Enter the team name"<<endl;
getline(cin, team);
for(j=1;j<=LOOP_LIMIT; ++j)
{
cout<<"Enter the player name"<<endl;
getline(cin, name);
cout<<"Enter the position played"<<endl;
getline(cin, posi);
cout<<"Enter the player's at bats:"<<endl;
cin>>atbats;
cout<<"Enter the player's hits:"<<endl;
cin>>hits;
cout<<endl;
cout<<name<<endl;
avg=(hits/atbats);
cout<<avg<<endl;
cout<<endl;
cout<<"Player # "<<j<<endl;
cin.ignore();
// These have to be inside the for loop to accumulate.
teamatbats=(teamatbats+atbats);
teamhits=(teamhits+hits);
}
teamavg=(teamhits/teamatbats);
cout<<team<<endl;
cout<<endl;
cout<<"The team's batting average is: "<<teamavg<<endl;
return 0;
}
Enjoy!
"At DIC we be baseball averaging code ninjas!"
This post has been edited by Martyr2: 6 Dec, 2007 - 12:45 PM
| 