Thread: Need help with array

shouldn't the while loop like:

Code:

while(!inFile.eof()){
    
    // ...
}

The way you have it looks like an infinite loop because:

Code:

bool end = false;

while( end == false ) // will always be true.

Also, what's up with all the global variables?

The prototype for studentData() is okay to be located there, but I would put everything else within main().

Then you need to pass the inFile by reference to studentData() like this:

Code:

studentData( &inFile );

I didn't see this:

Code:

if(inFile >> tempGrade) //priming read
end = true; // if eof set boolean to true

that looks kinda fishy...

BTW, I don't know what your exact assignment is, but judging by what you have posted I would reconsider your design, specifically all the arrays you have:

Code:

char studentName[24][40],letterGrade[24];
int classData[24][4], tempGrade;

letterGrade[] is unused at present, though I don't know if you'll need that later.

Consider using an array of structs for this particular assignment, if you're allowed to do so.

Originally Posted by dudeomanodude

shouldn't the while loop like:

Code:

while(!inFile.eof()){
    
    // ...
}

Typically no, eof() should not be used to control a loop.

Originally Posted by dudeomanodude

The way you have it looks like an infinite loop because:

Code:

bool end = false;

while( end == false ) // will always be true.

Also, what's up with all the global variables?

Actually, I suspect it is not an infinite loop precisely because end is a global variable, but you were misled because of that. This is anecdotal evidence why global variables should be avoided.

It looks like reading the file might be almost correct as far as controlling the end is concerned (whereas using inFile.eof() definitely isn't), although it seems to suffer from the exact same drawback.

The usage of globals in this code is terrible indeed, so it is no wonder it confuses the hell out of a casual reader. No portion of code, neither main nor StudentData, makes much sense when read separately.

As to the problem, you have 4 numbers followed by a string on a line, but it appears you are trying to read 24*4 + 1 numbers in a row (in the inner loop).


Edit: Sorry, that's not quite right, although it seems that you might be trying to read 5 numbers on the first iteration.

Can someone point me in the right direction?

I think that at the moment the right direction is to change your global variables into local variables. Try that first, and then post your attempt if you still need more help. If you do ask for help again, remember to post your code with consistent indentation.

Sorry, the above post doesn't make much sense. (I mean OP's).

Your file input fails because of

Code:

if(inFile >> tempGrade) //priming read

This reads one number, leaving 3 numbers unread on the first line. You then attempt to read 4 more integers. However, on the 4th time the file stream encounters a string. As it cannot convert it to a number, it goes into a failed state and all subsequent reads fail.

You might probably need to check for potential eof at each input statement. Currently your code seems to be meant for reading the file in 24 line batches. What if the count of lines is not a multiple of 24?