0
#include <iostream>
#include <vector>
#include <fstream>
#include <string>   
using namespace std;
/* run this program using the console pauser or add your own getch, system("pause") or input loop */

class foo
{
public:
    string Name = "Баби";
    int Age = 5;

    void show()
    {
        cout << "Name = " << Name << endl;
        cout << "Age = " << Age << endl;
    }

    void write(ostream& os) const
    {
        os.write((const char*)& Age, sizeof Age);

        size_t len = Name.length();
        os.write((const char*)& len, sizeof len);
        os.write(Name.c_str(), len);
    }

    void read(istream& in)
    {
        in.read((char*)& Age, sizeof Age);

        size_t len;
        in.read((char*)& len, sizeof len);
        Name.resize(len);
        in.read(&Name[0], len);
    }
};



int main(int argc, char** argv) {
    setlocale(LC_ALL, "Rus");
    foo* obj = new foo();
    obj->Age = 5;
    obj->Name = "Масло";

    ofstream file("file.bin", ios::binary | ios::app);
    if (file.is_open()) {
        obj->write(file);
        obj->show();
    }
    file.close();

    foo* result = new foo();
    ifstream file1("file.bin", ios::binary);
    if (file1.is_open()) {
        result->read(file1);
        file1.close();
        result->show();
    }
    file1.close();

    return 0;
}

Неправильно записывает и читает из файла. Записывает age = 5и Name = "Масло", а выводит age=0 и Name = "". В чем может быть проблема?

1
  • @AnT До сих пор не могу решить проблему. Изменил немного код, проблема остается.
    – justlead
    11 мая 2019 в 17:10

1 ответ 1

0

Файл не забывайте закрывать. Перед тем как пытаться открывать файл на чтение, надо не забывать закрыть предыдущее открытие (на запись). И не забывайте проверять, успешно ли вы у вас открылся файл - тогда бы вы сами поняли, почему "не работает" чтение.

Запись/чтение у вас сделано не так эффективно как могло бы, но тем не менее все правильно.


Более эффективно:

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

  2. При чтении строки ее можно аккуратно прочитать прямо внутрь std::string, без создания промежуточного буфера

Например

void write(ostream& os) const
{
  os.write((const char*) &Age, sizeof Age);

  size_t len = Name.length();
  os.write((const char*) &len, sizeof len);
  os.write(Name.c_str(), len);
}

void read(istream& in)
{
  in.read((char*) &Age, sizeof Age);

  size_t len;                       
  in.read((char*) &len, sizeof len);
  Name.resize(len);
  in.read(Name.data(), len); // или `in.read(&Name[0], len);`
}
18
  • Спасибо! Но у меня в строке in.read(Name.data(), len); пишет Аргумент типа const char* не совместим с типом char* Я так понимаю, это связано с тем, что Name это здесь const char*,а как его переводить в char*?
    – justlead
    10 мая 2019 в 23:43
  • @justlead: Не Name, а Name.data(). Какой компилятор вы используете? Здесь нужен С++17. Если вы получаете такую ошибку, то замените эту строчку на in.read(&Name[0], len); 11 мая 2019 в 0:06
  • Запись в файл переменных типа size_t и int приведет к веселым багам при смене разрядности. Надо бы конвертировать в типы с фиксированной битовой шириной.
    – gbg
    11 мая 2019 в 0:47
  • @gbg: Особенности мира бинарных файлов. Любой бинарный файл разрабатывается под конкретный набор требований времени жизни/переносимости: внутри сессии выполнения, между сессиями выполнения, между версиями программы, между программами данной С++ платформы, между платформами и т.п. У каждого варианта - свой набор требований. 11 мая 2019 в 0:51
  • Ну уж нет, это проявление недальновидности. Достаточно было использовать фиксированный формат, чтобы получить переносимую программу. А так получили потенциальные и очень дорогие проблемы.
    – gbg
    11 мая 2019 в 9:28

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.