Неправильно работает функция write при работе с файлом

@Ярхимед · Регистрация: 27.05.2018

Студворк — интернет-сервис помощи студентам

Добрый день. Выполняю задачу по созданию файла с произвольным доступом и взаимодействию с ним.
Имеется функция, которая должна добавлять в нужные места новые записи, но почему-то она добавляет в конец файла их.
Вот сам код функции:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
void createNewEntry(fstream& newEntry)
{
    int number, quantity;
    double price;
    string name;
 
    cout << "\n\nTo stop input, press ctrl + z\n\n";
 
    cout << "Enter the entry number: ";
 
    while (cin >> number) {
        Hardware enteredTool;
 
        cout << "Enter the name of the product: ";
        cin >> name;
        cout << "Enter the quantity in stock: ";
        cin >> quantity;
        cout << "Enter the price of the product: ";
        cin >> price;
 
        Hardware tool(number, name, quantity, price);
        newEntry.seekp((tool.getEntryNumber() - 1) * sizeof(Hardware));
        newEntry.write(reinterpret_cast<const char*> (&tool), sizeof(Hardware));
 
        cout << "Enter the entry number: ";
    }
    cin.clear();
}

Проверял посредством tellp, считыванием записи на месте, куда должна производиться новая - всё правильно расставлено, помогите, пожалуйста, разобраться в ошибке.

Если нужен, вот полный код программы

Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
#include <iostream>
#include <cstdlib>
#include <fstream>
#include <string>
#include <iomanip>
#include "Hardware.h"
 
using namespace std;
 
enum Choises { NEW = 1, EDIT, DELETE, DISPLAY, SAVE, END };
 
int enterChoise();
void createNewEntry(fstream&);
void displayMessage(fstream&);
void outputLine(ostream&, const Hardware&);
 
int main()
{
    fstream hard("Hardware.dat", ios::app | ios::in);
 
    if (hard.peek() == EOF) {
        hard.clear();
        Hardware emptyEntry;
        for (int i = 0; i < 100; i++) 
            hard.write(reinterpret_cast <const char*>(&emptyEntry), sizeof(Hardware));
    }
 
    int choise;
 
    while ((choise = enterChoise()) != END) {
        switch (choise) {
        case NEW:
            createNewEntry(hard);
            break;
        case DISPLAY:
            displayMessage(hard);
            break;
        }
 
        hard.clear();
    }
 
    getchar();
}
 
int enterChoise()
{
    cout << "\nEnter your choise\n"
        << "1 - Create a new record.\n"
        << "2 - Edit the existing entry.\n"
        << "3 - Delete an existing entry.\n"
        << "4 - Display the list on the screen.\n"
        << "5 - Save the list in the file «HardwareList.txt»\n"
        << "6 - Exit\n? ";
 
    int menuChoise;
 
    cin >> menuChoise;
 
    return menuChoise;
}
 
void createNewEntry(fstream& newEntry)
{
    int number, quantity;
    double price;
    string name;
 
    cout << "\n\nTo stop input, press ctrl + z\n\n";
 
    cout << "Enter the entry number: ";
 
    while (cin >> number) {
        Hardware enteredTool;
 
        cout << "Enter the name of the product: ";
        cin >> name;
        cout << "Enter the quantity in stock: ";
        cin >> quantity;
        cout << "Enter the price of the product: ";
        cin >> price;
 
        Hardware tool(number, name, quantity, price);
        newEntry.seekp((tool.getEntryNumber() - 1) * sizeof(Hardware));
        newEntry.write(reinterpret_cast<const char*> (&tool), sizeof(Hardware));
 
        cout << "Enter the entry number: ";
    }
    cin.clear();
}
 
void displayMessage(fstream& hardwareList)
{
    hardwareList.clear();
    hardwareList.seekg(0);
 
    Hardware tool;
    hardwareList.read(reinterpret_cast <char*> (&tool), sizeof(Hardware));
    int i = 0;
    while (!hardwareList.eof()){
        cout << i << ' ';
        outputLine(cout, tool);
        hardwareList.read(reinterpret_cast <char*> (&tool), sizeof(Hardware));
        i++;
    }
}
 
void outputLine(ostream& output, const Hardware& tool)
{
    output << left << setw(5) << tool.getEntryNumber()
        << setw(31) << tool.getProductName() << setw(5)
        << tool.getQuantity() << setw(10) << setprecision(2)
        << right << fixed << showpoint << tool.getPrice() << endl;
}

Заранее спасибо)

@nmcf · 08.08.2018, 14:18

Ну ты же сам указал ios::app.

@Ярхимед · 08.08.2018, 14:27 **[ТС]**

nmcf, а какая разница? Разве после нельзя сменить место, куда нужно писать, с помощью seekp?

@anapshy · 08.08.2018, 14:55

C++
1
app | append | All output operations happen at the end of the file, appending to its existing contents.

http://www.cplusplus.com/refer... ream/open/

1) И если запись нужно положить в определенное место с использование write, то всё равно не получится без полной перезаписи файла. Считывай объекты в контейнер. Помещай запись в нужное место в контейнере. А потом записывай каждый объект контейнера в файл.

2) Не юзай std::string.
Считывание с двоичного файла
А если юзаешь, то перед записью самого объекта записывай размер каждой используемой в нем строки std::string.
А при считывании считывай сначала размеры строк. Потом делай resize строк std::string в объекте. А уж потом read.
Слишком много заморочек.
Лучше используй массив char фиксированной длины и работай с ним.

C++
1
2
3
4
5
6
7
8
9
class Hardware
{
    // ...
private:
    int number, quantity;
    double price;
    char name[512];
    //...
};

@Бард · 08.08.2018, 14:56

Сообщение от Ярхимед

Разве после нельзя сменить место, куда нужно писать, с помощью seekp?

Любая операция записи при наличии ios::app перепозиционируется на запись в конец файла.

@anapshy · 08.08.2018, 14:59

Вот пример как делал я когда-то подобный класс для read/write.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
class Student final
{
public:
    typedef unsigned long long ull;
    typedef unsigned UINT;
    Student(void);
    Student(short, UINT, std::string, ull, std::string, short);
    Student(const Student&);
    friend std::ostream& operator << (std::ostream&, const Student&);
    friend std::istream& operator >> (std::istream&, Student&);
    Student& operator=(const Student&);
    short GetCourse(void) const;
    UINT GetGroupCode(void) const;
    const char* GetName(void) const;
    ull GetRecordBook(void) const;
    const char* GetDiscipline(void) const;
    short GetAssesment(void) const;
private:
    short course;
    UINT group_code;
    char name[MAX_NAME + 1U]; // +1 под '/0'
    ull record_book;
    char discipline[MAX_NAME + 1U]; // +1 под '/0'
    short assessment;
};
 
Student::Student(void) : course(short()), group_code(UINT()), name("none"),
record_book(ull()), discipline("none"), assessment(short()) {};
 
Student::Student(short _c, UINT _g, std::string _na, ull _nu, std::string _d, short _a)
    : course(_c), group_code(_g), record_book(_nu), assessment(_a)
{
    std::memcpy((this->name), _na.c_str(), MAX_NAME);
    std::memcpy((this->discipline), _d.c_str(), MAX_NAME);
};
 
Student::Student(const Student &other)
{
    *this = other;
}
 
std::ostream& operator << (std::ostream &os, const Student &rhs)
{
    return os << "Курс: " << rhs.course << std::endl
        << "Код группы: " << rhs.group_code << std::endl
        << "Фамилия: " << rhs.name << std::endl
        << "Номер зачетной книжки: " << rhs.record_book << std::endl
        << "Дисциплина: " << rhs.discipline << std::endl
        << "Оценка: " << rhs.assessment << std::endl;
}
 
std::istream& operator >> (std::istream &is, Student &rhs)
{
    std::cout << "Введите 'Курс': ";             is >> rhs.course;
    std::cout << "Введите 'Код группы': ";              is >> rhs.group_code;
    while (is.get() != '\n'); // чистка потока перед getline
    std::cout << "Введите 'Фамилия': ";                   is.getline(rhs.name, MAX_NAME);
    std::cout << "Введите 'Номер зачетной книжки': "; is >> rhs.record_book;
    while (is.get() != '\n'); // чистка потока перед getline
    std::cout << "Введите 'Дисциплина': ";             is.getline(rhs.discipline, MAX_NAME);
    std::cout << "Введите 'Оценка': ";                 is >> rhs.assessment;
    return is;
}
 
Student& Student::operator=(const Student &rhs)
{
    if (this != &rhs)
    {
        this->course = rhs.course;
        this->group_code = rhs.group_code;
        std::strcpy(this->name, rhs.name);
        this->record_book = rhs.record_book;
        std::strcpy(this->discipline, rhs.discipline);
        this->assessment = rhs.assessment;
    }
    return *this;
}
 
short Student::GetCourse(void) const
{
    return this->course;
}
 
Student::UINT Student::GetGroupCode(void) const
{
    return this->group_code;
}
 
// если есть поддержка С++17, то можно возвращать std::string_view для удобства
const char* Student::GetName(void) const
{
    return this->name;
}
 
Student::ull Student::GetRecordBook(void) const
{
    return this->record_book;
}
 
const char* Student::GetDiscipline(void) const
{
    return this->discipline;
}
 
short Student::GetAssesment(void) const
{
    return this->assessment;
}

@obivan · 08.08.2018, 14:59

Сообщение от Ярхимед

а какая разница? Разве после нельзя сменить место, куда нужно писать, с помощью seekp?

при ios::app при каждом очищении потока по моему курсор будет помещаться в конец файла, а при ios::ate только при первом открытии. Вкратце юзни ios::ate и все поидее

@anapshy · 08.08.2018, 15:10

obivan, после записи кажется идет пометка конца файла.
Подай на вход: 1 1 2 1 1 1
После введи 2 и по идее должно вывести 1,а потом 2. А выводится только 1.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
#include <iostream>
#include <fstream>
#include <string>
 
class MyStruct
{
    int m_Number;
public:
    MyStruct() : MyStruct({}) {}
    MyStruct(int number)
        : m_Number{ number } {}
    int& GetNumber() { return m_Number; }
    int GetNumber() const { return m_Number; }
};
 
int main()
{
    int key{ -1 };
    while (key)
    {
        std::cout << "Enter key: ";
        std::cin >> key;
        switch (key)
        {
        case 1: // добавить
        {
            std::cout << "Enter place: ";
            int place{};
            std::cin >> place;
            std::fstream file{ "text.bin", std::ios_base::binary | std::ios_base::out };
            MyStruct record{};
            std::cin >> record.GetNumber();
            file.seekp(sizeof(MyStruct) * (place - 1), std::ios_base::beg);
            file.write(reinterpret_cast<const char*>(&record), sizeof(MyStruct));
            file.close();
            break;
        }
        case 2: // считать
        {
            std::fstream file{ "text.bin", std::ios_base::binary | std::ios_base::in };
            file.seekg(0, std::ios_base::end);
            auto endPos = file.tellp();
            file.seekg(0, std::ios_base::beg);
            while (file.tellg() < endPos)
            {
                std::cout << "ok" << std::endl;
                MyStruct obj{};
                file.read(reinterpret_cast<char*>(&obj), sizeof(obj));
                std::cout << obj.GetNumber() << std::endl;
            }
            file.close();
            break;
        }
        }
    }
}

@QuakerRUS · 08.08.2018, 15:24

Сообщение от Ярхимед

fstream hard("Hardware.dat", ios::app | ios::in);

C++
1
fstream hard("Hardware.dat", ios::out | ios::in | ios::binary);

Если вы используете файлы с бинарной записью или чтением, не забывайте указывать флаг ios::binary, иначе у вас символы разметки текстовых строк будут автоматически преобразованы и структура файла собьется, так как поток будет считать себя текстовым. Вместо ios::app используйте ios::out или ios::ate, если хотите, чтобы отсчет seekp шел с начала файла. Если вы одновременно открываете файл и на запись, и на чтение, вы должны быть уверены что файл с таким именем уже существует. Если не уверены, откройте сначала только на запись и создайте его, потом закройте, потом откройте в режиме запись/чтение. Или открывайте одновременно только в одном режиме: запись или чтение.

Новые блоги и статьи Все статьи Все блоги /
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .

@nmcf 7804 / 6568 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	08.08.2018, 14:18
	Ну ты же сам указал `ios::app`. 0

@Ярхимед 0 / 0 / 0 Регистрация: 27.05.2018 Сообщений: 8
	08.08.2018, 14:27 [ТС]
	nmcf, а какая разница? Разве после нельзя сменить место, куда нужно писать, с помощью seekp? 0