Reading from file and outputting to console through vector - c++

Question

I'm trying to read from this file file.txt which contains contents of the contestants who participated in the long jump event during the Olympic games.
The file is in the format of [First Name] [Last Name] [Nationality] [Distance]
There are 40 contestants in this file. I'm trying to organize them such that there is a vector of pointers of athletes, 40 to be precise. Then dynamically allocate them to the heap. Each athlete is one object in the vector.
Once each athlete object is entered into the vector, I wish to output all the contents of the vector onto the console through a for loop.
However, as it currently stands, in my code I do have 40 objects allocated to the vector, but its the same one being repeated 40 times. The last object is also being repeated twice for some reason.
Any help would be greatly appreciated! Thanks in advance.
Test.cpp
#include <iostream>
#include<fstream>
#include<vector>
#include<string>
#include "Person.h"
#include "Athlete.h"
using namespace std;
vector<Athlete*> athletesList(40);
//overload the operator << to be used for printing the record Objects
ostream& operator<<(ostream& out, Athlete& a) {
out << a.getFirstName() << " " << a.getLastName() << " " << a.getNationality() << " " << a.getDistance() << "\n";
return out;
}
void readAthletesFromFile() {
fstream athlethesFile;
athlethesFile.open("file.txt");
string tmpFirstName;
string tmpLastName;
string tmpNationality;
string tmpDoubleDistance;
while (!athlethesFile.eof()) {
athlethesFile >> tmpFirstName;
athlethesFile >> tmpLastName;
athlethesFile >> tmpNationality;
athlethesFile >> tmpDoubleDistance;
double tmpDistance = stod(tmpDoubleDistance);
for (int i = 0; i < 40; i++) {
athletesList[i] = new Athlete(tmpFirstName, tmpLastName, tmpNationality, tmpDistance);
}
cout << *athletesList[0];
}
}
int main()
{
readAthletesFromFile();
system("pause");
}
File.txt
Aleksandr Menkov Russia 8.09
Aleksandr Petrov Russia 7.89
Alyn Camara Germany 7.72
Arsen Sargsyan Armenia 7.62
Boleslav Skhirtladze Georgia 7.26
Christian Reif Germany 7.92
Christopher Tomlinson Great_Britain 8.06
Damar Forbes Jamaica 7.79
Eusebio Caceres Spain 7.92
George Kitchens United_States 6.84
Godfrey Khotso-Mokoena South_Africa 8.02
Greg Rutherford Great_Britain 8.08
Henry Frayne Australia 7.95
Ignisious Gaisah Ghana 7.79
Li Jinzhe China 7.77
Lin Ching-Hsuan-Taipei China 7.38
Louis Tsatoumas Greece 7.53
Luis Rivera Mexico 7.42
Marcos Chuva Portugal 7.55
Marquise Goodwin United_States 8.11
Mauro-Vinicius da-Silva Brazil 8.11
Michel Torneus Sweden 8.03
Mitchell Watt Australia 7.99
Mohamed Fathalla-Difallah Egypt 7.08
Mohammad Arzandeh Iran 7.84
Ndiss Kaba-Badji Senegal 7.66
Povilas Mykolaitis Lithuania 7.61
Raymond Higgs Bahamas 7.76
Roman Novotny Czech-Republic 6.96
Salim Sdiri France 7.71
Sebastian Bayer Germany 7.92
Sergey Morgunov Russia 7.87
Stanley Gbagbeke Nigeria 7.59
Stepan Wagner Czech-Republic 7.5
Supanara Sukhasvasti Thailand 7.38
Tyrone Smith Bermuda 7.97
Vardan Pahlevanyan Armenia 6.55
Viktor Kuznyetsov Ukraine 7.5
Will Claye United_States 7.99
Zhang Xiaoyi China 7.25
Expected Output
Ex: *athletesList[0] = Aleksandr Menkov Russia 8.09
*athletesList[10]= Godfrey Khotso-Mokoena South_Africa 8.02
Current Output
Ex: *athletesList[0] =
Aleksandr Menkov Russia 8.09
Aleksandr Petrov Russia 7.89
Alyn Camara Germany 7.72
Arsen Sargsyan Armenia 7.62
Boleslav Skhirtladze Georgia 7.26
Christian Reif Germany 7.92
Christopher Tomlinson Great_Britain 8.06
Damar Forbes Jamaica 7.79
Eusebio Caceres Spain 7.92
George Kitchens United_States 6.84
Godfrey Khotso-Mokoena South_Africa 8.02
Greg Rutherford Great_Britain 8.08
Henry Frayne Australia 7.95
Ignisious Gaisah Ghana 7.79
Li Jinzhe China 7.77
Lin Ching-Hsuan-Taipei China 7.38
Louis Tsatoumas Greece 7.53
Luis Rivera Mexico 7.42
Marcos Chuva Portugal 7.55
Marquise Goodwin United_States 8.11
Mauro-Vinicius da-Silva Brazil 8.11
Michel Torneus Sweden 8.03
Mitchell Watt Australia 7.99
Mohamed Fathalla-Difallah Egypt 7.08
Mohammad Arzandeh Iran 7.84
Ndiss Kaba-Badji Senegal 7.66
Povilas Mykolaitis Lithuania 7.61
Raymond Higgs Bahamas 7.76
Roman Novotny Czech-Republic 6.96
Salim Sdiri France 7.71
Sebastian Bayer Germany 7.92
Sergey Morgunov Russia 7.87
Stanley Gbagbeke Nigeria 7.59
Stepan Wagner Czech-Republic 7.5
Supanara Sukhasvasti Thailand 7.38
Tyrone Smith Bermuda 7.97
Vardan Pahlevanyan Armenia 6.55
Viktor Kuznyetsov Ukraine 7.5
Will Claye United_States 7.99
Zhang Xiaoyi China 7.25
Zhang Xiaoyi China 7.25

Answer 1

Avoid calling new and delete explicitly to dynamically allocate memory. You don't need it in combination with STL containers. A container allocates and manages the memory on the heap.
The for loop is not necessary. For each element you read you overwrite all elements with the last read element.
You print the first element from the vector for each element you read:
Change
cout << athletesList[0];
to
cout << athletesList.back();
You can't
while (!athlethesFile.eof()) {
because eof is set after you tried to read and it failed. You have to check if the read was successful after the read. Why is iostream::eof inside a loop condition (i.e. while (!stream.eof())) considered wrong?
Don't include Person.h if you don't use it.
You should avoid system(). Usually it's not portable.
In your current code you read 40 elements in a loop. In each loop iteration you allocate 40 elements for that vector. That means that you allocate memory for 1600 elements. You do not clean that up. That is a serious memory leak.
#include "Athlete.h"
#include <fstream>
#include <iostream>
#include <string>
#include <vector>
using std::cerr;
using std::cout;
using std::fstream;
using std::ostream;
using std::string;
using std::vector;
//overload the operator << to be used for printing the record Objects
ostream& operator<<(ostream& out, Athlete& a) {
out << a.getFirstName() << " " << a.getLastName() << " " << a.getNationality() << " " << a.getDistance() << "\n";
return out;
}
auto readAthletesFromFile() {
vector<Athlete> athletesList;
athletesList.reserve(40);
fstream athlethesFile("file.txt");
if (!athlethesFile) {
cerr << "Could not open file\n";
return athletesList;
}
string tmpFirstName;
string tmpLastName;
string tmpNationality;
string tmpDoubleDistance;
while (true) {
athlethesFile >> tmpFirstName;
athlethesFile >> tmpLastName;
athlethesFile >> tmpNationality;
athlethesFile >> tmpDoubleDistance;
if (!athlethesFile) break;
auto tmpDistance = stod(tmpDoubleDistance);
athletesList.emplace_back(tmpFirstName, tmpLastName, tmpNationality, tmpDistance);
//cout << athletesList.back();
}
return athletesList;
}
int main() {
auto athletesList = readAthletesFromFile();
cout << athletesList[9];
}

Answer 2

I'm going to posit that #Thomas Sablik started out in the right direction, but then at least in my opinion, he kind of dropped the ball, so to speak. In particular, I think this part:
//overload the operator << to be used for printing the record Objects
ostream& operator<<(ostream& out, Athlete& a) {
out << a.getFirstName() << " " << a.getLastName() << " " << a.getNationality() << " " << a.getDistance() << "\n";
return out;
}
...is absolutely great! Even if I really wanted to say bad things about it, just about the worst I could probably come up with would be that the Athlete should be passed by const reference (and being honest, that's a pretty minor quibble).
So we have great code to write an Athlete object. But for some reason, our code for reading Athlete objects isn't the same at all. Why not?
I believe the code for reading should be almost a mirror image of the code for writing.
std::istream &operator>>(std::istream &in, Athlete &a) {
in >> a.firstName >> a.lastName >> a.nationality >> a.distance;
return in;
}
Using that, we can leave almost all the memory management to the standard library. In fact, there's probably no need for the readAthletesFromFile at all. Reading in a file becomes something like this:
std::ifstream input("file.txt");
std::vector<Athlete> list{std::istream_iterator<Athlete>(input), {}};
That's it. That reads in the entire file. Well, it could stop sooner if it encountered bad input, but it'll read as much as it can anyway.
So let's consider an example: we'll read in the data and print out the top 5 distances (and the information for each of those records):
class Athlete {
public:
friend std::ostream& operator<<(std::ostream& out, Athlete const& a) {
out << a.firstName << " " << a.lastName << " " << a.nationality << " " << a.distance << "\n";
return out;
}
friend std::istream &operator>>(std::istream &in, Athlete &a) {
in >> a.firstName >> a.lastName >> a.nationality >> a.distance;
return in;
}
bool operator<(Athlete const &other) const {
return distance < other.distance;
}
private:
std::string firstName, lastName, nationality;
double distance;
};
int main() {
std::ifstream in("file.txt");
std::vector<Athlete> list{std::istream_iterator<Athlete>(in), {}};
std::partial_sort(list.begin(), list.end()-5, list.end());
std::copy(list.end()-5, list.end(), std::ostream_iterator<Athlete>(std::cout, ""));
}
Result:
Mauro-Vinicius da-Silva Brazil 8.11
Marquise Goodwin United_States 8.11
Aleksandr Menkov Russia 8.09
Greg Rutherford Great_Britain 8.08
Christopher Tomlinson Great_Britain 8.06