getline not working as expected - c++

Question

I'm trying to display a text file data into columns using a class structure and getline() function to read through the textfile and the dump the data into a vector class. But it seems that the program didn't even run to my 'while' loop then quitting the program. The textfile is not empty.
Below is my code:
void ScrambleWordGame::displayScoreChart() {
//open file
fstream readScoreChart("ScoreChart.txt");
string line = "";
//vector to store data in
vector<personResult> perResult;
personResult person;
//while file is open, do stuff
if(readScoreChart.is_open())
{
//check through the file
readScoreChart.seekp(0,ios::end);
//get the size of the file's data
size_t size = readScoreChart.tellg();
if(size == 0)
cout << "No results yet. Please TRY to win a game. AT LEAST~" << endl;
else
{
//create the 1st row with 4 column names
cout << left
<< setw(20) << "Player Name "
<< setw(20) << "Winning Time "
<< setw(20) << "No. Of Attempts "
<< setw(20) << "Game Level " << endl;
//fill the second line with dashes(create underline)
cout << setw(70) << setfill('-') << "-" << endl;
//read the file line by line
//push the 1st line data into 'line'
cout << getline(readScoreChart,line);
while(getline(readScoreChart,line))
{
//create stringstream n push in the entire line in
stringstream lineStream(line);
//reads the stringstream and dump the data seperated by delimiter
getline(lineStream,person.playerName,':');
getline(lineStream,person.winningTime,':');
getline(lineStream,person.noOfAttempts,':');
getline(lineStream,person.gameLvl);
//sort the results based on their timing
//sort(perResult.begin(),perResult.end(),pRes);
//display the results
cout << left
<< setfill(' ')
<< setw(25) << person.playerName
<< setw(22) << person.winningTime
<< setw(17) << person.noOfAttempts
<< setw(16) << person.gameLvl
<< endl;
}
}
}
readScoreChart.close();
}
Edit: Example of the TextFile
Joel:3:1:1
Mary:5:2:2
John:25:3:1

Answer 1

Your file pointer is at the end of the file after your first seek. You need to reposition it to the beginning of the file.
if(size == 0)
{
cout << "No results yet. Please TRY to win a game. AT LEAST~" << endl;
}
else
{
readScoreChart.seekp(0,ios::begin);
// all you other stuff
}

Answer 2

You need to seek back to the beginning of the file to be able to read. Better still, just don't seek to the end in the first place.
I'd re-structure the code a bit -- write an operator>> to read a record from a file, and an operator<< to write a record to a file.
class person {
std::string name;
std::string winning_time;
std::string num_attempts;
std::string level;
public:
bool operator<(person const &other) const {
return std::stoi(winning_time) < std::stoi(other.winning_time);
}
friend std::istream &operator>>(std::istream &in, person &p) {
std::string buffer;
std::getline(in, buffer);
std::istringstream is(buffer);
std::getline(is,p.name,':');
std::getline(is,p.winning_time,':');
std::getline(is,p.num_attempts,':');
std::getline(is,p.level);
return in;
}
friend std::ostream &operator<<(std::ostream &os, person const &p) {
return os << std::setw(25) << p.name
<< std::setw(22) << p.winning_time
<< std::setw(17) << p.num_attempts
<< std::setw(16) << p.level;
}
};
With those in place, the rest gets quite a bit simpler:
void show_header(std::ostream &os) {
cout << left
<< setw(20) << "Player Name "
<< setw(20) << "Winning Time "
<< setw(20) << "No. Of Attempts "
<< setw(20) << "Game Level " << "\n";
std::cout << std::string(70, '-');
}
void game::displayScoreChart(){
std::ifstream in("ScoreChart.txt");
// read the data:
std::vector<person> people{std::istream_iterator<person>(in),
std::istream_iterator<person>()};
if (people.empty()) {
std::cout << "No scores yet."
return;
}
// sort it by winning time:
std::sort(people.begin(), people.end());
show_header(std::cout);
// display it:
for (auto const &p : people)
std::cout << p << "\n";
return 0;
}
As a simple replacement if you don't have a C++11 compiler, an stoi can be written something like this:
int stoi(std::string in) {
return strtol(in.c_str(), NULL, 10);
}