[Nela12]

It gets stuck in a loop and only seems to read the whole first line into
.city. I've never read in data with csv so any help is appreciated!

void loadData ( List<FootballTeam>& data)
{
	FootballTeam team;

	ifstream inFile ( "NFL_Coaches.csv" );

	if ( !inFile )
	{
		cout << "The file could not be found! \n";
		exit(0);
	}

		getline (inFile, team.city); 
	
		cout << team.city << endl; //used to test if code is being read in correctly

inFile.ignore();
		getline ( inFile , team.teamName);
		
inFile.ignore();
		getline (inFile, team.coach);
		

	inFile >> team.yearsofExperience;


	while ( !inFile.eof() )
	{
		data.addMember ( team );

		getline (inFile, team.city); 
		inFile.ignore();

		cout << team.city << endl;

		getline ( inFile , team.teamName);
		inFile.ignore();
		
		getline (inFile, team.coach);
		inFile.ignore();

		inFile >> team.yearsofExperience;
	}

I've tried moving around the .ignores to see if it was that, but I'm still getting the same result

[jimblumberg]

Please post a small sample of your input file.

Jim

[Nela12]

There is alot of data, but here's the first few.Each is in their own cell (from excel)

St. Louis	Rams	Jeff Fischer	17
San Francisco	49ers	Jim Harbaugh	1
Chicago	Bears	Lovie Smith	8
Cincinnati	Bengals	Marvin Lewis	9
Buffalo	Bills	Chan Gailey	4
Denver	Broncos	John Fox	10
Cleveland	Browns	Pat Schurmur	1
Tampa Bay	Buccaneers	Greg Schiano	0
Arizona	Cardinals	Ken Whisenhut	5
San Diego	Chargers	Norv Turner	16
Kansas City	Chiefs	Romeo Crennel	7

[#define]

A csv file is a text file you may be able to open it with a notepad or text editor, getline gets a whole line of text you could try printing the lines.

  string line;

  while( getline (inFile, line) )
  {
     cout << line << endl; 
  }

[jimblumberg]

This is not actually a CSV file since there are no commas separating the fields. However it does seem to be in a tab separated format so you can use this fact to process each line. I recommend reading the entire line into a std::string using getline then using a stringstream to parse that string.

std::string line;
std::string team_city, team_name, team_coach;
int coach_experiance;
while(getline(inFile,line))
{
   std::istringstream ins(line);
   getline(ins, team_city, '\t');
   getline(ins, team_name,'\t');
   getline(ins, team_coach, '\t');
   ins >> coach_experiance;
}

Of course you will need to change the variables to match your's but this should ge you going in the correct direction.


Jim

[Nela12]

If I were to manually put the commas in,how would I go about doing that? Would I do it in VS or excel?

[jimblumberg]

Quote

If I were to manually put the commas in,how would I go about doing that? Would I do it in VS or excel?

I believe Excel has the option of exporting files in the CSV format, just make sure you select the correct delimiter, the comma. However there is nothing wrong with a tab delimited file, the only difference is the delimiting character.

Jim

[Nela12]

Okay. I did that. Now the commas appear. However, it's still not reading in right. As soon as it begins reading in, it crashes. I have no idea what to do.

[jimblumberg]

Post your current code.

Jim

[Nela12]

void loadData ( List<FootballTeam>& data)
{
	FootballTeam team;

	ifstream inFile ( "NFL_Coaches.csv" );

	//Verify that file is found
	if ( !inFile )
	{
		cout << "The file could not be found! \n";
		exit(0);
	}

	//Read in data
	getline (inFile, team.city); 

	inFile.ignore();
	getline ( inFile , team.teamName);
		
	inFile.ignore();
	getline (inFile, team.coach);
		
	inFile >> team.yearsofExperience;

	while ( !inFile.eof() )
	{
		data.addMember ( team );

		getline (inFile, team.city, ','); 
		inFile.ignore();

		getline ( inFile , team.teamName, ',');
		inFile.ignore();
		
		getline (inFile, team.coach, ',');
		inFile.ignore();

		inFile >> team.yearsofExperience;
	}
	
}

[jimblumberg]

I suggest you put a cout statement after all of your extraction operations, both the getline() and extraction operator>> so you can see why your program is not working correctly.

Also you need to realize that getline() extracts and discards the delimiter, but the extraction operator>> will not. This is why I suggested using the stringstream. With the stringstream you don't need to worry about the delimiters, either the comma, or the end of line character.

Also the way you are using eof() to control your entry loop will lead to having one extra line retrieved.

Jim

[Nela12]

Added 12 cout << "hello" << endl; Outputted with 14 before it crashed.

[code]
Hello
Hello
Hello
Hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
hello
Press any key to continue . . .
[code]

I would use the string::stream but I'm still learning. I want to make sure I am doing this right before trying alternatives. PS: You have been really helpful. I really do appreciate it, thanks

[jimblumberg]

I meant to print out the value you're trying to retrieve from the file, not just hello.

I also don't see any indication of a crash. It looks more like a successful program termination.

Jim