C++ reading from file function failing to exit while cycle [duplicate]

This is one of the most common problems with reading from files. Checking f.eof() will only tell if you the previous read hit the end of the file. Hence, after the last successful read (where you got your third "a" printed), eof() returns false and your loop executes one more time, outputting an extra "a".

Instead, use your extraction line as the condition for your while loop:

while (f>>type && f>>name && f>>battery) { if (type=="Mac") { Mac r = Mac(name,battery); robots.push_back(&r); } if (type=="Eco") { Eco r = Eco(name,battery); robots.push_back(&r); } if (type=="Pro") { Pro r = Pro(name,battery); robots.push_back(&r); } } 

This works because as soon as one of your extractions hits the end of the file, the while condition will be false and the loop won't execute.

If you want to continue after badly formatted lines, I recommend that you use std::getline instead:

std::string line; while (std::getline(f, line)) { std::stringstream ss(line); if (ss >> type && ss >> name && ss >> battery) { // ... } } 

The reason for using this std::getline approach is because you would have had a problem if half of a line had been correctly formatted. Your approach would have read in perhaps type and name then failed on battery and the next iteration of the loop would have started from the same place.

You're also going to have a problem with the fact you're pushing pointers to objects that have automatic storage duration into some vectors. When the block in which the objects are created ends, the pointers will be pointing to an invalid object.

This is a FAQ. In C/C++, the end-of-file condition is triggered only after you have tried to read past the end. So, you can't assume from feof() == false that input is available (because you could be exactly at the end of the file, but not yet past it). You should always test for valid input after a read operation.