Как исправить ошибку с построчным чтением из бинарного файла C++?

Question

[Андрей Загородний]

Есть функция введения (input):

void input() {
	ofstream In("AutoBase.txt", ios::binary);
	char k = '+';
	AutoBase a;
	string Read;
	while (k == '+') {
		cout << "Enter name of car: "; getline(cin, a.Name);
		cout << "Enter Release Date with a point '10.10.2010': "; getline(cin, Read);
		a = Divide(a, Read, 0);
		cout << "Enter Sale Release Date with a point '10.10.2010': "; getline(cin, Read);
		a = Divide(a, Read, 1);
		cout << "Enter '+' in case you want to contnue: "; cin >> k;
                In.write((char*)&a, sizeof(AutoBase));
		cin.ignore();
	}
	In.close();
}
AutoBase Divide(AutoBase a, string Read, bool b) {
	int pos1 = Read.find('.');
	int pos2 = Read.rfind('.');
	if (!b) {
		a.Release.day = stoi(Read.substr(0, pos1));
		a.Release.mounth = stoi(Read.substr(pos1 + 1, pos2));
		a.Release.year = stoi(Read.substr(pos2 + 1));
	}
	else {
		a.SaleRelease.day = stoi(Read.substr(0, pos1));
		a.SaleRelease.mounth = stoi(Read.substr(pos1 + 1, pos2));
		a.SaleRelease.year = stoi(Read.substr(pos2 + 1));
	}
	return a;
}

Есть сама структура:

struct Data {
	int year;
	int mounth;
	int day;
};
struct AutoBase {
	string Name;
	Data Release;
	Data SaleRelease;
};

И я пытаюсь написать функцию вывода(output):

void output(string name) {
	ifstream Out(name, ios::binary);
	AutoBase a;
	while (Out.read((char*)&a, sizeof(AutoBase))) {
		cout << endl << a.Name << "   " << a.Release.day << "." << a.Release.mounth << "." << a.Release.year << "   " << 
                a.SaleRelease.day << "." << a.SaleRelease.mounth << "." << a.SaleRelease.year;
	}
	Out.close();
}

Так вот: после вывода текста из файла вылазит ошибка

Answer 1

[res2001]

1. Хорошо бы код спрятать под тег code (есть кнопка в панели инструментов) и вернуть разметку, не возможно читать.
   
2. Не видна последовательность вызовов input() и output().
   
3. В input() файл открываете на запись, но ничего туда не пишете, в итоге файл пустой.
   
4. В output вы не правильно читаете:
   

Суть в том, что тип string хранит саму строку в динамической памяти, а не в самом объекте (в объекте только указатель на динамическую память). string у вас используется в AutoBase.
Тем способом, которым вы пытаетесь читать из файла вы прочитаете какой-то указатель и скорее всего будет обращение к не выделенной памяти и программа упадет. Подозреваю, что запись вы бы реализовали так же и в файл записался бы указатель, а не строка.
Вам надо читать и писать объект string по другому. Есть 2 варианта:

• Сначала пишем/читаем размер строки, затем пишем/читаем саму строку заданного размера.
  
• Пишем всю строку, в конце дописываем какой-то разделитель. В качестве разделителя можно использовать символ '\0', так же как в сырых Си строках или то что вам больше нравится, лишь бы разделитель не мог содержаться в самой строке. При чтении надо читать побайтно, до тех пор пока не встретится символ разделитель.
  

Какой вариант реализовывать решать вам.

Comments

[Андрей Загородний]

Сначала вызывается Input, потом Output. Сорри, думал это интуитивно понятно. Строчку записи не скопировал сюда, уже исправил. Насчет записи - то в файл записывается все правильно, но я не могу корректно достать оттуда ничего. То есть оно читается и выводится в консольку, но после выдает ошибку нарушения прав доступа.

[res2001]

Андрей Загородний, Это как раз то о чем я писал - string нельзя так сохранять/загружать.
Вы просто можете посмотреть файл, который у вас записался на диск - там не будет строки AutoBase.Name.
Вообще это относится не только к string, но и к любым сложно устроенным объектам - требуются отдельная процедура сериализации/десериализации.
В вашем случае AutoBase.Name вы должны сохранять примерно так:

uint32_t name_len = a.Name.length();
In.write(&name_len, sizeof(name_len));
In.write(a.Name.c_str(), name_len);

Процесс чтения - аналогичный, сначала читаете размер, а затем заданное количество символов. После чтения размера можно подготовить строку - выделить память с помощью string.reserve() для последующего чтения.
Такой подход будет работать с обычным строками, каждый символ которых представлен 1 байтом. Для многобайтных строк (все варианты Unicode) этот вариант уже не пройдет, т.к. string.length() - возвращает количество символов в строке, а не количество байт, а пишете в файл вы байты, а не символы.