Thread: trouble reading binary data file

Hi,

I was hoping to get some help to understand why I have coded this so poorly. I have re written the code below to demonstrate the problem I face. A really good indicator that something is wrong is the fact that it crashes 50% during run time.
I would like to know why it crashes half the time ? what conditions had changed to make it execute okay the other times.

My main problem is that I had thought this had successfully written to file. As i have tested, by reading it back.
But if I remove my output to file code and only try to import my file (previously created) it recognizes there are two elements but can't output its data.
I am using code::blocks 10.05.
btw, I was only wanting to do this as a learning experience with binary files I do understand I could use a text file. Should I not be using binary files with string.

Many thanks in advance for your help.

Code:

#include <iostream>
#include <vector>
#include <fstream>

// Demonstrating my problem with reading from binary from file

using namespace std;

int main()
{
vector<string> newBuffer;    // will output file from here
vector<string> tempBuffer;    // will input file into here

string testing = "hello world\n";
tempBuffer.push_back(testing);
tempBuffer.push_back(testing);


// output to file
///*
        ofstream fout ("file1", ios::binary);

        if(!fout)
        {
            cout << "unable to open file\n";
        }
        fout.write( (char*) &tempBuffer,sizeof tempBuffer);          // casting char*
        fout.close();

        tempBuffer.clear();
//*/

// read from file put into new vector

    ifstream fin ("file1", ios::binary);
    if(!fin)
    {
        cout << "unable to open file\n";
    }
    fin.read( (char*) &newBuffer,sizeof newBuffer);
    fin.close();


// reading new vector


    cout << "newBuffer size " << newBuffer.size() << "\n";

    for (unsigned short x =0;x<newBuffer.size();x++)
            cout << (string) newBuffer[x];

    newBuffer.clear();

    return 0;
}

Thanks guys, obviously bit off more than I could chew. My tutorials had explained how to save a simple object to binary file using write/read. My class contains linked lists of objects and vectors of strings which I thought I might be able to apply the same theory to. Once that failed I did try to simplify it as iMalc suggested and thought I had proved to myself containers populated with int would work (no class) using my method. But I was obviously mislead by the first iteration of output that suggested I had successfully written and read back from file.
Oh well, I think have probably been adventurous enough and should return back to appending to text files. I am using strings after all.
p.s
In regards to Salem remarks regarding casting. Should I let the commands that I want to use (say read) dictate to me what my class member variable types should be?
Obviously strings are easier to work with then char *. Is it normal to go back and rewrite your code as you amend the type to avoid casting ? Or do you tend to learn what types are better used with what commands so tend to choose wisely from the beginning?

Originally Posted by Elysia

No, you should focus on using the correct types and choosing the best methods to manipulate the data from those.

okay I think I understand that. For example if using c style char* I should stay with c libraries to manipulate it?

The serialize a vector worked great. But my strings contained multiple words "hello big world" for example and I couldn't work out how to stop it from truncating the white spaces.