Problems writing an integer to binary file C++

Question

I am using fstream to write an integer to binary file.

int main(){

fstream f1;
int num = 2, num2 = 0;

f1.open("dat1", ios::app | ios::out | ios::in | ios::binary);

f1.write((char *)num, sizeof(int));
f1.seekp(0);
f1.read((char *)num2, sizeof(int));

cout << num2;

}

The problem is on line f1.write. I can write to binary file an array, but when I try to write just one block of int it gives me an error:

Unhandled exception at 0x522C7EA6 (msvcp120d.dll) in Project.exe: 0xC0000005: Access violation reading location 0x00000002.

I don't understand what the problem is.

Bathsheba · Accepted Answer · 2016-01-05 12:52:49Z

5

You need to cast the address of num to char*, not num itself. In fact, the behaviour of using the result of (char*)num is undefined which explains the crash.

Use f1.write((char *)&num, sizeof(num)); instead. Change f1.read similarly.

I've also changed the sizeof argument: I prefer that style as it future-proofs you against type changes. Moving on, you'll need to consider endianness if you're writing and reading in different platforms (e.g. Windows and Linux).

answered Jan 5, 2016 at 12:52

Bathsheba

235k35 gold badges376 silver badges504 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Danny_ds · Accepted Answer · 2016-01-05 13:26:54Z

You need this:

f1.write((char *)&num, sizeof(int));
f1.seekp(0);
f1.read((char *)&num2, sizeof(int));

write() and read() want a pointer to where your data is.

int num = 2;

Using (char *)num converts num to a pointer pointing at address 0x00000002.

Also, when working with binary files (other than testing) it is always best to include a header, which should at least contain the following:

A Magic Number (usually 4 bytes), so the exact file type can be determined or checked regardless of what the extention might be.
A version number, which will allow you to extend the format and structure of your file in the future without breaking your current format.
As others already stated, a Byte Order Mark which will allow your data to be read in correctly on all platforms (Little-endian / Big-endian).

Eutherpy · Accepted Answer · 2016-01-05 12:49:53Z

2

The line should actually be:

f1.write((char *)&num, sizeof(int));

You need an address as an argument to the write function. Check out http://www.cplusplus.com/reference/ostream/ostream/write/.

answered Jan 5, 2016 at 12:49

Eutherpy

4,5818 gold badges47 silver badges75 bronze badges

1 Comment

frast Over a year ago

Also one needs to consider the byte oder in order to use the file in a platform independent manner if needed.

Alex44 · Accepted Answer · 2016-01-05 12:50:53Z

0

write want's a array. This means the start address. (char *)num this converts the number to a address and will point to a point you don't want. Use &num to get the address of num. Maybe make a cast with (char*) too.

answered Jan 5, 2016 at 12:50

Alex44

3,9858 gold badges44 silver badges62 bronze badges

Collectives™ on Stack Overflow

Problems writing an integer to binary file C++

4 Answers 4

Comments

Comments

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related