Quote:Original post by DarkAnima
The suggestion was made to remove the spaces, but that would involve creating new variable sets for storing the values without spaces and then turning those into values with spaces after loading, which would take a long time and a lot of code.

string s; // So our routine doesn't affect states = Player1.name;replace(s.begin(),s.end(),' ','_');fout << s << endl;s = Player2.name;replace(s.begin(),s.end(),' ','_');fout << s << endl;s = Player3.name;replace(s.begin(),s.end(),' ','_');fout << s << endl;// etc

// No need for intermediates herefin >> Player1.name;replace(Player1.name.begin(),Player1,name.end(),'_',' ');fin >> Player2.name;replace(Player2.name.begin(),Player2,name.end(),'_',' ');fin >> Player3.name;replace(Player3.name.begin(),Player3,name.end(),'_',' ');// etc

Quote:Original post by DarkAnima

if (choice=!1) goto donesave;	}	fout << Player1.name;	fout << "\n" << Player2.name;	fout << "\n" << Player3.name;       //etc      	fout << "\n" << web.owned;	fout << flush;	fout.close();donesave:	;

Quote:Original post by erissian

Quote:Original post by DarkAnima

if (choice=!1) goto donesave;	}	fout << Player1.name;	fout << "\n" << Player2.name;	fout << "\n" << Player3.name;       //etc      	fout << "\n" << web.owned;	fout << flush;	fout.close();donesave:	;

Oh no you didn't!

Quote:Original post by DarkAnima
Another thing I was looking into and just thought of again, was getting definitions for the monsters and items put into a header file or something I can #include at the start of the main game file so that I can keep my code separate from everything else. I tried doing it with my monsters, but at first it told me that I hadn't defined the variables, and when I returned the variable initiation to my code (just the definition of the monster class, really) it told me that I was redefining the variables. After spending a couple hours trying to figure out what was going wrong, I gave up, but since there's going to be ~300 items and ~150 monsters I really would like to store all that data away from the program to keep things less cluttered. I'm sure I'm just forgetting something silly, or my inexperience is showing.

Quote:Original post by DarkAnimaI still would like to know why the cin function is unhappy with char*. I have plenty of good workarounds, so if I can't figure out how to make cin and char* be friends, I'll go back to strings and use one of the workarounds, but I'd rather fix the problem than find a complicated way around it, if possible.

char* text = NULL;fin >> text; // will crash, fin does not allocate space for your array

char* text = "bob"; // allocated 4 chars ("bob" + null terminator)// the file contains "bobIsAJollyGoodFellow"fin >> text; // will crash, "bob" will fit, "IsAJollyGoodFellow" overflows the 4 chars allocated

char buffer[1024];char* text;cin >> buffer; // will work as long as the line is not longer than 1024 charactersint len = strlen(buffer);text = new char[len + 1]; // + null terminatorstrcpy(text, buffer);

Quote:Original post by Structural

Quote:Original post by DarkAnimaI still would like to know why the cin function is unhappy with char*. I have plenty of good workarounds, so if I can't figure out how to make cin and char* be friends, I'll go back to strings and use one of the workarounds, but I'd rather fix the problem than find a complicated way around it, if possible.

I took a peek at the other code you posted, and the char* in the character class triggers me. I can't give you more than an educated guess because I never work with ifstreams and ostreams in C++. I think you failed to allocate (enough) space for the reading the string.

char buffer[BIG];fin.getline(buffer,BIG);

#include <string>using namespace std;string stuff;getline(fin,stuff);