File I/O

Code:

	case 's':
	ofstream save_file ("mudsave.txt");
	save_file<<"Test Test Test";
	save_file.close();
	ifstream load_file ("mudsave.txt");
	cout<<load_file;
	cin.get();
	break;

This is just a small part of my code where I am, of course, trying to read and write a file. These are the errors I get when I try to compile it:

Code:

/home/account/cpp/mud.cpp: In function `int main(int, char**)':
/home/account/cpp/mud.cpp:218: error: jump to case label
/home/account/cpp/mud.cpp:212: error:   crosses initialization of `
   std::ifstream load_file'
/home/account/cpp/mud.cpp:209: error:   crosses initialization of `
   std::ofstream save_file'
/home/account/cpp/mud.cpp:221: error: jump to case label
/home/account/cpp/mud.cpp:212: error:   crosses initialization of `
   std::ifstream load_file'
/home/account/cpp/mud.cpp:209: error:   crosses initialization of `
   std::ofstream save_file'
/home/account/cpp/mud.cpp:250: error: jump to case label
/home/account/cpp/mud.cpp:212: error:   crosses initialization of `
   std::ifstream load_file'
/home/account/cpp/mud.cpp:209: error:   crosses initialization of `
   std::ofstream save_file'
/home/account/cpp/mud.cpp:255: error: jump to case label
/home/account/cpp/mud.cpp:212: error:   crosses initialization of `
   std::ifstream load_file'
/home/account/cpp/mud.cpp:209: error:   crosses initialization of `
   std::ofstream save_file'
/home/account/cpp/mud.cpp:218: warning: destructor needed for `std::ifstream
   load_file'
/home/account/cpp/mud.cpp:218: warning: where case label appears here
/home/account/cpp/mud.cpp:218: warning: (enclose actions of previous case
   statements requiring destructors in their own scope.)
/home/account/cpp/mud.cpp:221: warning: destructor needed for `std::ifstream
   load_file'
/home/account/cpp/mud.cpp:221: warning: where case label appears here
/home/account/cpp/mud.cpp:250: warning: destructor needed for `std::ifstream
   load_file'
/home/account/cpp/mud.cpp:250: warning: where case label appears here
/home/account/cpp/mud.cpp:255: warning: destructor needed for `std::ifstream
   load_file'
/home/account/cpp/mud.cpp:255: warning: where case label appears here

Place the code between the case and the break in a code block (i.e. by using braces).

Code:

	case 's':{
	ofstream save_file ("mudsave.txt");
	save_file<<"Test Test Test";
	save_file.close();
	ifstream load_file ("mudsave.txt");
	load_file>>str;
	cout<<str<<"\n";
	cin.get();
	break;
	}

It kind of works now.....I can't believe I forgot the brackets, it is quite late. Instead of outputing everything in the file...like I want it to it just ouputs "Test" once.

You shouldn't need those brackets. Your error is caused because the << operator on an ostream, by default, stops on whitespace.

Open the file in your text editor, it probably contains the string "Test Test Test". You only see one "Test" because you are only reading one word into the string.

Well ya, I need to know how to read the entire file into the string.

Edit:Look at my second post. I changed the code.

You shouldn't need those brackets.

My guess is that cerin is declaring variables within a switch selection structure, and in such a case placing the code under a case label in a code block limits the variable scope as expected.

Your error is caused because the << operator on an ostream, by default, stops on whitespace.

As far as I know, this isnt true. operator>> used as an input operator stops on whitespace, but operator<< used as an output operator does not.

You shouldn't need those brackets.

My guess is that cerin is declaring variables within a switch selection structure, and in such a case placing the code under a case label in a code block limits the variable scope as expected.

I am afraid that I haven't declared it in the switch statement.

Well ya, I need to know how to read the entire file into the string.

In this case, you can use getline().

Code:

getline(load_file, str);

I am afraid that I haven't declared it in the switch statement.

You have, since you're constructing the i/o streams.

Code:

/home/account/cpp/mud.cpp: In function `int main(int, char**)':
/home/account/cpp/mud.cpp:214: error: invalid conversion from `void*' to `
   char**'
/home/account/cpp/mud.cpp:214: error: cannot convert `char*' to `size_t*' for
   argument `2' to `__ssize_t getline(char**, size_t*, FILE*)'

I changed it to getline and now I get this error.

Post your updated code. It should be the smallest and simplest code that demonstrates this error.

Code:

	case 's':{
	ofstream save_file ("mudsave.txt");
	save_file<<"Test Test Test";
	save_file.close();
	ifstream load_file ("mudsave.txt");
	getline(load_file, str);
	cout<<str<<"\n";
	cin.get();
	break;
	}

laserlight: you're probably right on that - I was reading and typing fast, and it's late. As for the brackets, I'd suspect you're right, but since we haven't seen that code, it's hard to judge. I know I've NEVER used brackets like that - just seeing it made my eyes twitch.

cerin: for getline(), it looks like you've got str declared as a char*, not a std::string (just from the error message). If not, let us see a bit more of the code, and we could help more.

I changed a few things, and now it works. Thanks for your help everyone.