Heres a function thats intended to append a string to a text file:
I want it to seek the end of the file then write from there on, which I thought I was doing with seekp(). The problem is it overwrites the contents of the file. What am I doing wrong here?Code:bool FString::Append(const char* filename) { ofstream out; out.open(filename); if(!out.is_open())return false; out.seekp(0, ios::end); out.write(str_ptr, (int)strlen(str_ptr)); return true; }
Cheers.
Code:out.open(filename, ios::app);
Thanks that works, and I don't even need the seekp() anymore. Oh and I added out.close() at the end before anyone points that out. Cheers
I'd do this instead though:
Considering that I changed str_ptr (char*) to str (std::string).Code:bool Append(const std::string& filename) { std::ofstream out(filename.c_str(), std::ios::app); if(out.is_open( )) { out << str; return true; } return false; }
Edit: It's a bit more C++ ish and I find it much cleaner.
The explicit close() is not necessary. That's the nice thing about RAII.
All the buzzt!
CornedBee
"There is not now, nor has there ever been, nor will there ever be, any programming language in which it is the least bit difficult to write bad code."
- Flon's Law
Thanks for the tips.
