Thread: Adding Scores from a string array

I have to do the read a string, accumulate the scores, and get the average from records.dat.

The string includes name and test scores.ie John Smith 99 89 88 91.

My problem is that I can't figure out how to pull the test scores while reading the string so that I can accummulate the scores and find the average.

Here's my code so far:

Code:

#include <conio.h>
#include <iostream>
using namespace std;
#include <fstream>
#include <string>
 
int main(){
	string str[20];
	int i = 0, j = 0, last;
	string str1 = "Name		Score1	Score2	Score3	Score4	Average";
	cout << str1 << endl;
	ifstream InFile("Records.dat");
	if(! InFile)
		{
		 cout << "Error opening output file" << endl;
		 getch();
		 return -1;
		}
		while(! InFile.eof()){
			for(int I = 0; i < 5; i++){
				getline(InFile, str[i]);
				cout << str[i] << endl;
			}
		}
 
	InFile.close();
	//To make console show
	getch();
	//return '0' to show that the process ran ok.
	return 0;
}

I got the error isdigit() cannot convert parameter 1 from std::string to int.

In the example, line is a string, so line[i] is a character (that converts to an int automatically). Make sure you are using the [i] to get the ith character from the string.

Originally Posted by 7stud

Where does the string come from? A file? Will it always be the same format, i.e.: first, last, score1, score2, score3, score4?

You should always look to ''>>" when reading in data that is separated by spaces. If you know the data is going to be in that format every time, it's simple matter of declaring two string variables and an int array to read in the data. The ">>" operator reads in data sequentially one "word" at a time, where a word is a sequence of adjacent characters. Here is an example:

dataSource>>first>>last>>score1>>score2>>score3>>s core4;

Additionally, the general rule is to make your read statement a while loop conditional. You may not realize it, but a statement like:

dataSource>>first;

calls a function. The function reads data from the source and puts the data into the variable first. The return value of the function is whatever is on the left side of the ">>" symbol, e.g. dataSource. In a loop conditional, the dataSource object will evaluate to false if for any reason you can't read from the source anymore--like when hitting the end of the file or an error occurs while reading in data.

If you can't read in any data because of an error, your loop should terminate immediately otherwise you may enter an infinite loop. Checking for eof() isn't the answer. If there's an error while reading in data before hitting eof(), then you won't be able to read in anymore data. Your loop will continue looping because you haven't hit eof yet, but you can't read in data because of the error, so you will never hit eof, and the loop will be infinite.

Data comes from a .dat file.

first last test1 test2 test3 test4

I think I hit the .eof() loop so I changed to a for() loop.

I think I hit the .eof() loop so I changed to a for() loop.

Wrong approach. Read my post again:

Additionally, the general rule is to make your read statement a while loop conditional.

For instance,

Code:

while(inFile>>data1>>data2)
{


}