Чтение структуры из файла

[atenon]

Привет.
Возникла такая проблема.
При таком коде

#include "stdafx.h"
#include <iostream>
using namespace std;

struct history
	{
		char *incident;
		int number;
		int month;
		int year;
	};

int main()
{
	int n=3;
	history a[3], d[3];
	a[0].incident="1 World war"; a[0].number=28; a[0].month=7; a[0].year=1914;
	a[1].incident="Revolution"; a[1].number=7; a[1].month=11; a[1].year=1917;
	a[2].incident="2 World war"; a[2].number=1; a[2].month=9; a[2].year=1939;

	FILE *F, *G;
	F=fopen("D:\\history", "wb");
	fwrite(&n, sizeof(int), 1, F);
	fwrite(a, sizeof(history), n, F);
	fclose(F);

	G=fopen("D:\\history", "rb");
	fread(&n, sizeof(int), 1, G);
	fread(d, sizeof(history), n, G);

	for (int i=0; i<n; i++)
		cout<<d[i].incident<<"\t"<<d[i].number<<"\t"<<d[i].month<<"\t"<<d[i].year<<"\n";

	fclose(G);
	return 0;
}

Все работает нормально.
Удаляю все, что касается записи в файл (файл history на диске уже существует)

#include "stdafx.h"
#include <iostream>
using namespace std;

struct history
	{
		char *incident;
		int number;
		int month;
		int year;
	};

int main()
{
	int n=3;
	history d[3];
	
	FILE *G;
	G=fopen("D:\\history", "rb");
	fread(&n, sizeof(int), 1, G);
	fread(d, sizeof(history), n, G);
	
	for (int i=0; i<n; i++)
		cout<<d[i].incident<<"\t"<<d[i].number<<"\t"<<d[i].month<<"\t"<<d[i].year<<"\n";

	fclose(G);
	return 0;
}

После компиляции поле incident не выводится с остальным все в порядке, в чем проблема понять не могу.
Спасибо

[ozo]

use std::string, std::fstream

[atenon]

Если не трудно вставте пожалуйста в код.

[Гром]

Вы для incident память не выделили.

[atenon]

Вы для incident память не выделили.

Коды вида
char *a="csadfsdf";
cout<<a;
Всегда работали прекрасно. Не думаю, что в этом проблема. К тому же в первом варианте программы все работает без выделения памяти.

[Stilet]

Всегда работали прекрасно.

Да, но во втором коде ты считываешь строки из файла, а не инициализируешь переменные. Потому тебе просто необходимо выделить память для incident во втором примере, чтоб fread могла считать в готовый массив.

[Гром]

char *a="csadfsdf";
cout<<a;

Что тут происходит - в стеке создается константный массив символов (строковой литерал), который висит там вплоть до окончания работы программы. Затем указателю a тупо присваивается адрес висящего в стеке массива, и этот массив потом просто выводится.
В ситуации, отличной от использования строкового литерала такой номер не прокатит. Даже в таком случае:

char* s;
std::cin >> s;
std::cout << s;

у вас ничего не выйдет, т.к. под s память не выделена. Поэтому надо писать примерно так:

char* s;
s = new char[20];
std::cin >> s;
std::cout << s;
//...
delete[] s;

Или же пользоваться статическими массивами, что, впрочем, зачастую нерационально.

[atenon]

Что то никак не идет прога может быть что то с выделением памяти не то делаю. Посмотрите пожалуйста.

#include "stdafx.h"
#include <iostream>
using namespace std;

struct history
	{
		char *incident;
		int number;
		int month;
		int year;
	};

int main()
{
	int n;// Количество элементов массива записано в файле
	FILE *G;
	G=fopen("D:\\history", "rb");
	fread(&n, sizeof(int), 1, G);
	history *d=new history[n];

	for (int i=1; i<n; i++)
		d[i].incident=new char[30];

	fread(d, sizeof(history), n, G);
	
	for (int i=0; i<n; i++)
		cout<<d[i].incident<<"\t"<<d[i].number<<"\t"<<d[i].month<<"\t"<<d[i].year<<"\n";

	fclose(G);
	return 0;
}

[mrChester]

Покажи структуру записей в файле

[Stilet]

Автор, вот тебе для раздумий:http://www.programmersforum.ru/showthread.php?t=105501
Я бы именно так делал, вместо считывания сразу всего в массив и выделения памяти.