Thread: Help with some data writing - I can't make it more speciefic then that

Ok, How are you guys doing? the reason why i'm posting is because i seem to run into some "techincal difficuties" when writing data from a buffer, mainly this:

Player Name: Gordon
Player Health: 100
Player Shields: 100
Player Position:
X: 300
Y: 100
Z: 100

now it looks good but the buffer is 256 and when i save the file and open it, there's data written to file that's a bunch of "i"s so i'm wondering how can you make a char buffer that will know exactly how much the size has to be for the data ro be written to file and not have that left over garbage? maybe i'm not explainging my self good.......:-p anyways, if u don't get it i'll just post the file or the picture on photobucket.

One thing i should mention that those are part of a struct, i didn't write them directly buit they are variables, but i'll try the C++ way, thanks

Code:

struct Data
{
	string PlayerName;
	unsigned int Health;
	unsigned int Shields;
	
	float Xpos;
	float Ypos;
	float Zpos;
};

bool SaveData(Data &data);

bool LoadData(Data &data);

Data player;



int _tmain(int argc, _TCHAR* argv[])
{

	player.PlayerName = "Gordon";
	player.Health    = 100;
	player.Shields   = 100;
	player.Xpos		 = 300;
	player.Ypos		 = 100;
	player.Zpos		 = 100;

	if(SaveData(player))
	{
		cout << "Data Saved" << endl;
	}

	cout << "Would u like to load it back? " << endl;
	char in;
	cin >> in;
	if(in =='y')
	{
		LoadData(player);
	}

	cout << "Size of: " << sizeof(Data) << endl;
	while(1)
	{}
	return 0;
}

bool SaveData(Data &data)
{
	char DataBuffer[256];

	std::ofstream DataFile("Player.data");

	DataFile.setf( ios_base::binary );
	if(DataFile.is_open())
	{
		sprintf(DataBuffer,"Player Name: %s\n"
			"Player Health: %d\n"
			"Player Shields: %d\n"
			"Player Position: \n"
			"X: %.f\n"
			"Y: %.f\n"
			"Z: %.f\n",data.PlayerName.c_str(),data.Health,data.Shields,
			data.Xpos,data.Ypos,data.Zpos);

		DataFile.write((char*)&DataBuffer,sizeof(DataBuffer));

		if(DataFile.is_open())
		{
			DataFile.close();
		}
	}

	return true;
}

bool LoadData(Data &data)
{
	char BufferData[256];

	ifstream PlayerData("Player.data");

	sprintf(BufferData,"Player Name: %s\n"
			"Player Health: %d\n"
			"Player Shields: %d\n"
			"Player Position: \n"
			"X: %.f\n"
			"Y: %.f\n"
			"Z: %.f\n",data.PlayerName.c_str(), data.Health,data.Shields,data.Xpos,
			data.Ypos,
			data.Zpos);

	if(PlayerData.is_open())
	{
		PlayerData.read((char*)BufferData,sizeof(BufferData));

		if(PlayerData.is_open())
		{
			PlayerData.close();
		}
	}

	cout << "Player Name: " << data.PlayerName << endl;
	cout << "Player Health: " << data.Health << endl;
	cout << "Player Shileds: " << data.Shields << endl;
	cout << "Player Position: " << endl;
	cout << "X: " << data.Xpos << endl;
	cout << "Y: " << data.Ypos << endl;
	cout << "Z: " << data.Zpos << endl;

	return true;
}

As you can see i used the write/read functions of the streams to write files now that seems to be better then using << but i could be wrong, are there any advantages to using write/read? or should i avoid them like the plague?

There is one reason not to use write() and read() right now. One reason is because you're using text files, and because you're using a text file, you can treat ifstreams like cin, and ofstreams like cout, instead. If write() is better than operator<< I wouldn't know why: if it's better because you care about performance, you're just guessing. File I/O is a bottleneck anyway. That is why files do their own buffering: to reduce the number of reads and writes performed. For the future, the write() and read() functions are quite useful if you're trying to use a binary file, though.