Как записать сразу всю структуру (содержащую string) в бинарный файл?

Question

[SUkI3]

Пример структуры

struct anketa
{
    int year;
    string fio;
};

Структуру необходимо записать таким методом, чтобы потом запустить отдельно другую функцию, которая сможет прочитать файл, но перед этим не добавлять записи в файл(ну то есть чисто чтение)
Пробовала так: после того как пользователь введет string посчитать string.size() + 1 и приплюсовать sizeof(int), но после чтение из файла выдает какой-то бред

Comments

[justAnotherCluelessUser]

"но после чтение из файла выдает какой-то бред"
Почему так много людей верят в ясновидящих??

Тут их особо нет... вроде....
Так что весь код на текстообменник(типа пастбин) и ссылку.

[SUkI3]

justAnotherCluelessUser,
Воть

anketa ar;
	int size = 0;
	int k = 0;
	fstream fout ("test.dat", ios::binary | ios::app);
	if(!fout)
{
	cout<<"ERROR!!!!!123";
	return 1;
}
	cout<<"Vvedite kol-vo struct"<<endl;
	cin >> k;
	for(int i = 0; i<k; i++){
		cout<< i+1 <<" Struct : " << endl;
		cout << "Year\n";
		cin >> ar.year;
    	cin.ignore();
    	cout << endl << "FIO \n";
    	getline(cin, ar.fio, '\n');
        cin.clear();
        size = sizeof(ar.year) + ar.fio.length() + 1;
		fout.write((char*)&ar,sizeof(size));
		size = 0;
}
fout.close();

Потом вот это запускаю отдельно

fstream file("test.dat", ios::binary | ios::in);
	if(!file)
{
	cout<<"ERROR!!!!!11";
	return 404;
}	
	if (getline( file, ar.fio, '\0' ))
	do
	{
		cout << endl << " Struct  String: " << ar.fio << " Year " << ar.year << endl;
	}
	while(getline( file, ar.fio, '\n' ));
	file.close();

[RabraBabr]

Эмм....

Вы понимаете, что тут происходит?

size = sizeof(ar.year) + ar.fio.length() + 1;
    fout.write((char*)&ar,sizeof(size));

[SUkI3]

RabraBabr, передаем байты и размер, который получился после как пользователь ввел данные

[RabraBabr]

SUkI3, какие байты? И размер чего? Внимательно подумайте.

[SUkI3]

RabraBabr, байты которые преобразовали адрес указателя структуры, а размер, ну просто размер цифра, или похоже я не правильно понимаю работу записи в бинарник...(

[Алексей Сергей]

SUkI3, почитайте про POD типы. В приведенном коде std::string не является POD типом и соответсвенно вся anketa тоже. При fout.write(&ar, sizeof(ar)) в файл попадут внутренние поля std::string (включая указатель (адрес) на строку), но не сама строка!

[SUkI3]

Алексей Сергей, да я знаю, что структура не POD, если б было все так просто...
Поэтому и вопрос на тостере
Так возможно создать алгоритм который записывает структуру со string так, как если б там не было string, а были только POD типы?

[Алексей Сергей]

SUkI3, смотрите в сторону сериализаторов.
Есть много либ на гитхабе. Например я такую пару раз использовал: https://github.com/USCiLab/cereal
Есть еще либа в бусте, но не все хотят буст в зависимости тащить. https://www.boost.org/doc/libs/1_70_0/libs/seriali...

[Алексей Сергей]

SUkI3, забыл еще упомянуть, что сериализаторы обычно предоставляют несколько форматов для выходного файла. Легко делается как-минимум: бинарный, бинарный-кроссплатформа, текст, json и xml.

[SUkI3]

Алексей Сергей, сериализаторы какое страшное слово, спасибо, почитаем)

[RabraBabr]

SUkI3, https://thispointer.com/c-how-to-read-or-write-obj...

[Roman]

SUkI3, в чем смысл бинарной записи обычной строки. Байт на байт, шило на мыло.

[SUkI3]

Roman, у string методы, модификаторы есть уже готовые, вроде как легче работаться должно

[Roman]

SUkI3,
при чем тут сейчас методы и модификаторы?
вы данные (data) сохраняете, данные в std::string это динамический массив char.
если использовать

struct anketa
{
    int year;
    char fio[64];
};

то все гораздо проще и с чтением и с записью.

Со string как раз таки мороки больше.

#include<iostream>
#include<fstream>
#include<string>
#include<cassert>
using namespace std;

struct Data
{
  int value = 0;
  string str;
};

ofstream& operator<<(ofstream& ofs, Data& d)
{
  size_t sz = d.str.size();
  ofs.write(reinterpret_cast<char*>(&d.value), sizeof(d.value));
  ofs.write(reinterpret_cast<char*>(&sz), sizeof(sz));
  ofs.write(d.str.c_str(), sz);
  return ofs;
}

ifstream& operator>>(ifstream& ifs, Data& d)
{
  ifs.read(reinterpret_cast<char*>(&d.value), sizeof(d.value));
  size_t sz;
  ifs.read(reinterpret_cast<char*>(&sz), sizeof(sz));
  d.str.resize(sz);
  ifs.read(reinterpret_cast<char*>(&*d.str.begin()), sz);
  return ifs;
}


int main()
{
  Data d = {42, "John Doe"};
  ofstream ofs("D:\\file.bin", ios_base::binary);
  ofs << d;
  ofs.close();

  Data d2;
  ifstream ifs("D:\\file.bin", ios_base::binary);
  ifs >> d2;

  assert(d.value == d2.value && d.str == d2.str);

  cin.get();
}

Answer 1

[vanyamba-electronics]

#include <iostream>
#include <fstream>      // std::fstream

using namespace std;

typedef struct
{
    int year;
    string fio;
} Anketa;

int main () 
{
  Anketa anketa;
  anketa.year = 1643;
  anketa.fio = "Isaac John Newton";
  
  // *** Writing ***
  fstream fs;
  fs.open ("anketa.dat", std::fstream::out | std::fstream::binary);

  fs.write((char*) &anketa.year, sizeof(int));
  size_t len = anketa.fio.length();
  fs.write((char*) &len, sizeof(size_t));
  fs.write(anketa.fio.data(), len);

  fs.close();

  // *** Reading *** 
  fs.open ("anketa.dat", std::fstream::in | std::fstream::binary);

  fs.read((char*) &anketa.year, sizeof(int));
  fs.read((char*) &len, sizeof(size_t));
  char buf[len+1];
  fs.read(buf, len);
  buf[len] = 0;
  anketa.fio = buf;

  fs.close();

  cout << anketa.year << " " << anketa.fio << endl;

  return 0;
}

Comments

[SUkI3]

Суть в том, что запись, а потом соответственно и считывание из бинарного файла должно происходить в одну строчку, если б например все элементы структуры были простыми

Answer 2

[Vitaly]

Ну для POD должно как-то так работать:

anketa a;

std::ofstream out(file_name);
out.write(reinterpret_cast<char*>(&a), sizeof(a));

// ...

std::ifstream in(file_name);
in.read(reinterpret_cast<char*>(&a), sizeof(a));

Но у вас там std::string есть, а его надо в char* преобразовать чтобы так записать. Это больше чем одна строчка. Поэтому, такие чтение/запись надо выносить в отдельные функции.