// If you need a comment to explain what your function does, your function// name is not clear enough.// Also, putting 'c_' on the beginning of class names gains you exactly nothing.bool Map::readFileData() {	string line;	// Don't write comments to explain what code does; write them to	// explain why you're doing it. And again, work with clear variable	// names. Don't try to put information into the variable name that	// doesn't actually help understand the code. You only have one kind	// of map data (I'm assuming!) and there's only one data file that	// gets read by a Map (again, I'm assuming) so you don't need to say	// which file it is. (And you certainly don't need to call something	// "tmf_file" when the 'f' stands for 'file'!)	// To demonstrate the point, I've been very sparing with comments	// here - except to explain the problems with the old code.	ifstream file(filename.c_str());	if (!file.is_open()) { return false; }	// Use locals for values that you'll do a lot of work with, and only	// store them into a difficult-to-refer-to place when you're done.	// That way, you avoid repeating the "navigation instructions".	int width, length;	// Use loop constructs to loop.	while (true) {		// Notice how with modern tools, blank lines are automatically		// handled, because the attempt to scan data will fail. Although		// you could have done that with sscanf() too, by simply		// checking the return value.		if (!(getline(file, line)) { return false; }		stringstream ss(line);		if (ss >> width >> length) { break; }	}	data.width = width;	data.length = length;	// Don't store redundant information. In this case, that's	// data.n_cell.	// Indent for every for loop. If it looks bad because there's too	// much indentation, that's because your code is too complicated.	for (int y = 0; y < length; ++y) {		for (int x = 0; x < width; ++x) {			// To avoid copying overhead/logic, you can also use			// references when you do the trick with locals			// mentioned above.			Cell& c = data.cell[x][y];			while (true) {				if (!(getline(file, line)) { return false; }				stringstream ss(line);				if (ss >> c.type) { break; }			}			// In your code you didn't skip blank lines at this			// point, so I won't either. But I will continue to			// add error checking logic. ;)			if (!(getline(file, line)) { return false; }			stringstream ss(line);			if (!(ss >> c.floor_height >> c.ceiling_height) { return false; }					if (!(getline(file, line)) { return false; }			stringstream ss(line);			if (!(ss >> c.wall_textures[0] >> c.wall_textures[1] >> c.wall_textures[2] >> c.wall_textures[3]) { return false; }			if (!(getline(file, line)) { return false; }			stringstream ss(line);			if (!(ss >> floor_texture >> ceiling_texture) { return false; }			//Set player position			if (c.type == S_PLAYERSPAWN) {				set_playerPosition(-x * CELL_SIZE, c.floor_height - 1.0f, -y * CELL_SIZE);				// Hold on; where does player_pos come from?				player.set_position(data.player_pos);			}			cout << c.floor_height << ", " << c.ceiling_height << endl;		}	}	// You don't need to close the file explicitly.	return true;}// No semicolon is needed after member function implementations - just after// the class body.