Проблемы с удалением указателей

@Tony000 · Регистрация: 19.10.2023

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

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

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
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
#include <iostream>
#include <fstream>
#include <ctime>
#include <cstring>
#include <limits>
#include <iomanip>
 
using namespace std;
 
struct Employees
{
    char *name, *lastname, *patronymic;
    char gender, isRetired;
    int salary, year;
};
 
int validate();
char *get_string();
 
char *get_string()
{
    char c;
    char *word = (char *)calloc(1, sizeof(char));
    unsigned ctr = 1;
    while (cin.get(c))
    {
        if (c == '\n' || c == ' ')
            break;
        word[ctr - 1] = c;
        word = (char *)realloc(word, (ctr + 1) * sizeof(char));
        ctr++;
    }
    word[ctr - 1] = '\0';
    return word;
}
 
void write_file(Employees *&st, size_t n)
{
    ofstream out;
    out.open("data.txt");
    if (!out.is_open())
    {
        cout << "There is no file";
        return;
    }
    out << "Gender    "
        << "Name    "
        << "Last name    "
        << "Patronymic    "
        << "Salary    "
        << "Year    "
        << "Retired?" << endl;
    for (int i = 0; i < n; i++)
    {
        out << st[i].gender << "         ";
        out << st[i].name << "      ";
        out << st[i].lastname << "     ";
        out << st[i].patronymic << "      ";
        out << st[i].salary << "      ";
        out << st[i].year << "      ";
        out << st[i].isRetired << endl;
    }
    out.close();
}
 
void input_struct_fields(Employees *&group, int i)
{
    cin.ignore(1);
    int current_year = 1970 + time(NULL) / 31536000;
    cout << "Enter a gender (f/m): " << endl;
    cin >> group[i].gender;
    while (tolower(group[i].gender) != 'm' && tolower(group[i].gender) != 'f')
    {
        cout << "Wrong input data. Try again.." << endl;
        while (getchar() != '\n')
            ;
        cin >> group[i].gender;
    }
    cin.ignore(1);
    cout << "Enter a name: " << endl;
    group[i].name = get_string();
    cout << "Enter a last name: " << endl;
    group[i].lastname = get_string();
    cout << "Enter a patronymic: " << endl;
    group[i].patronymic = get_string();
    cout << "Enter a salary: " << endl;
    group[i].salary = validate();
    cout << "Enter an year: " << endl;
    group[i].year = validate();
    int age = current_year - group[i].year;
    group[i].isRetired = (tolower(group[i].gender) == 'f' &&
                          (55 - age > 0 && 55 - age <= 3)) ||
                                 ((tolower(group[i].gender) == 'm') && (60 - age > 0 && 60 - age <= 3))
                             ? 'Y'
                             : 'N';
}
 
void fill_struct(Employees *&group, size_t &n)
{
    cout << "Do you want:\n1. to rewrite data in the file\n2. to add data in the existing file" << endl;
    char op1;
    cin >> op1;
    if (op1 == '1')
    {
        cout << "How many records do you want to fill? " << endl;
        n = validate();
        group = (Employees *)calloc(n, sizeof(Employees));
        for (int i = 0; i < n; i++)
        {
            cout << "RECORD " << i + 1 << endl;
            input_struct_fields(group, i);
        }
    }
    else if (op1 == '2' && n > 0)
    {
        char op2;
        do
        {
            group = (Employees *)realloc(group, ++n * sizeof(Employees));
            input_struct_fields(group, n - 1);
            cout << "Do you want to add one more element? (y/n) ";
            cin >> op2;
        } while ((tolower(op2) == 'y'));
    }
    else
    {
        cout << "There is no such operation or array is empty" << endl;
        return;
    }
    write_file(group, n);
}
 
void print_struct(Employees *gr, const int n)
{
    cout << "№ Gender    "
         << "Name    "
         << "Last name    "
         << "Patronymic    "
         << "Salary    "
         << "Year    "
         << "Retired?" << endl;
    for (int i = 0; i < n && gr != NULL; i++)
        cout << i + 1 << " " << gr[i].gender << setw(10) << gr[i].name << setw(10) << gr[i].lastname << setw(10) << gr[i].patronymic << setw(10) << gr[i].salary << setw(5) << gr[i].year << " " << gr[i].isRetired << endl;
}
 
void extract(ifstream &in, char *&str)
{
    int c = 1;
    char ch;
    str = (char*)calloc(c, sizeof(char));
    while (in.get(ch))
    {
        str[c - 1] = ch;
        str = (char*)realloc(str, ++c * sizeof(char));
        if (ch == ' ')
            break;
    }
    str[c - 1] = '\0';
    in >> str;
}
 
void read_data(Employees *&gr, size_t &n)
{
    if(n != 0) { 
        return;
    }
    ifstream in;
    in.open("data.txt");
    in.seekg(73);
    char ch;
    while (in.get(ch)) {
        if(ch == '\n') n++;
    }
    in.close();
    in.open("data.txt");
    in.seekg(73);
    gr = new Employees[n];
    for (int i = 0; i < n; i++)
    {
        char gender;
        in >> gender;
        char *name, *lastname, *patronymic;
        extract(in, name);
        extract(in, lastname);
        extract(in, patronymic);
        int sal, year;
        in >> sal >> year;
        char is_ret;
        in >> is_ret;
        gr[i].gender = gender; 
        gr[i].name = name;
        gr[i].lastname = lastname;
        gr[i].patronymic = patronymic;
        gr[i].salary = sal;
        gr[i].year = year;
        gr[i].isRetired = is_ret;
    }
    in.close();
}
 
int validate()
{
    int n;
    while (!(cin >> n) || n <= 0)
    {
        cin.clear();
        std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
        cout << "Enter a correct number: " << endl;
    }
    return n;
}
 
Employees *delete_record(Employees *&gr, size_t &length)
{
    if (length == 0)
        return NULL;
    cout << "Select a number of the record you want to delete: " << endl;
    print_struct(gr, length);
    int n = validate();
    if (n > length)
    {
        cout << "Length is less than a given number" << endl;
        return gr;
    }
    
    Employees *new_group = new Employees[length - 1];
    bool flag = false;
    int dyn_ind = 0;
    for (int i = 0; i < length; i++)
    {
        dyn_ind = (flag) ? i - 1 : i;
        if (i + 1 == n && length != 1)
        {
            free(gr[i].name); // тут начинаются проблемы
            free(gr[i].lastname);
            free(gr[i].patronymic);
            flag = true;
            continue;
        }
        new_group[dyn_ind].gender = gr[i].gender;
        new_group[dyn_ind].name = gr[i].name;
        new_group[dyn_ind].lastname = gr[i].lastname;
        new_group[dyn_ind].patronymic = gr[i].patronymic;
        new_group[dyn_ind].salary = gr[i].salary;
        new_group[dyn_ind].year = gr[i].year;
        new_group[dyn_ind].isRetired = gr[i].isRetired;
    }
    delete[] gr;
    length--;
    write_file(new_group, length);
    return new_group;
}
 
int main()
{
    char op;
    size_t length = 0;
    Employees *gr = NULL;
    read_data(gr, length);
    while (1)
    {
        cout << endl;
        cout << "Select an operation: " << endl;
        cout << "1. Write data in a file: " << endl;
        cout << "2. Read data from the file" << endl;
        cout << "3. Delete a record from the file" << endl;
        cout << "4. Exit" << endl;
        cin >> op;
        switch (op)
        {
        case '1':
            fill_struct(gr, length);
            break;
        case '2':
            read_data(gr, length);
            print_struct(gr, length);
            break;
        case '3':
            gr = delete_record(gr, length);
            break;
        case '4':
            exit(0);
            break;
        default:
            print_struct(gr, length);
            break;
        }
    }
    return 0;
}

@lemegeton · 18.11.2023, 13:00

У вас какая-то дикая помесь C и C++, причем совершенно непонятно, чего вы хотите достичь.
Вы надёргали откуда-то кусками код и пытаетесь в него привнести дополнительный функционал?

Очень не хочется разбираться где ошибка в таком запутанном коде.
Не должна такая простая задача исполняться в столь сложно читаемом стиле.
Моя рекомендация -- переписать всё, что связано с выделением и освобождением памяти, на С++ лад.
Желательно с использованием классов.

Какое у вас изначальное задание?

@Tony000 · 18.11.2023, 13:30 **[ТС]**

lemegeton, задание изначально должно быть сделано на C, но можно и на C++ с условием - без использования ООП и всяких продвинутых вещей (поэтому я не использую строки, векторы, классы). Мне нужно организовать ввод данных из структур в файлы, и наоборот. Я использовал realloc, calloc, чтобы мне было проще расширять массивы символов (строки)

Добавлено через 16 минут
В общем, поменял способ выделения памяти на Сишный везде, ничего не изменилось.

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
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
#include <iostream>
#include <fstream>
#include <ctime>
#include <cstring>
#include <limits>
#include <iomanip>
 
using namespace std;
 
struct Employees
{
    char *name, *lastname, *patronymic;
    char gender, isRetired;
    int salary, year;
};
 
int validate();
char *get_string();
 
char *get_string()
{
    char c;
    char *word = (char *)calloc(1, sizeof(char));
    unsigned ctr = 1;
    while (cin.get(c))
    {
        if (c == '\n' || c == ' ')
            break;
        word[ctr - 1] = c;
        word = (char *)realloc(word, (ctr + 1) * sizeof(char));
        ctr++;
    }
    word[ctr - 1] = '\0';
    return word;
}
 
void write_file(Employees *&st, size_t n)
{
    ofstream out;
    out.open("data.txt");
    if (!out.is_open())
    {
        cout << "There is no file";
        return;
    }
    out << "Gender    "
        << "Name    "
        << "Last name    "
        << "Patronymic    "
        << "Salary    "
        << "Year    "
        << "Retired?" << endl;
    for (int i = 0; i < n; i++)
    {
        out << st[i].gender << "         ";
        out << st[i].name << "      ";
        out << st[i].lastname << "     ";
        out << st[i].patronymic << "      ";
        out << st[i].salary << "      ";
        out << st[i].year << "      ";
        out << st[i].isRetired << endl;
    }
    out.close();
}
 
void input_struct_fields(Employees *&group, int i)
{
    cin.ignore(1);
    int current_year = 1970 + time(NULL) / 31536000;
    cout << "Enter a gender (f/m): " << endl;
    cin >> group[i].gender;
    while (tolower(group[i].gender) != 'm' && tolower(group[i].gender) != 'f')
    {
        cout << "Wrong input data. Try again.." << endl;
        while (getchar() != '\n')
            ;
        cin >> group[i].gender;
    }
    cin.ignore(1);
    cout << "Enter a name: " << endl;
    group[i].name = get_string();
    cout << "Enter a last name: " << endl;
    group[i].lastname = get_string();
    cout << "Enter a patronymic: " << endl;
    group[i].patronymic = get_string();
    cout << "Enter a salary: " << endl;
    group[i].salary = validate();
    cout << "Enter an year: " << endl;
    group[i].year = validate();
    int age = current_year - group[i].year;
    group[i].isRetired = (tolower(group[i].gender) == 'f' &&
                          (55 - age > 0 && 55 - age <= 3)) ||
                                 ((tolower(group[i].gender) == 'm') && (60 - age > 0 && 60 - age <= 3))
                             ? 'Y'
                             : 'N';
}
 
void fill_struct(Employees *&group, size_t &n)
{
    cout << "Do you want:\n1. to rewrite data in the file\n2. to add data in the existing file" << endl;
    char op1;
    cin >> op1;
    if (op1 == '1')
    {
        cout << "How many records do you want to fill? " << endl;
        n = validate();
        group = (Employees *)calloc(n, sizeof(Employees));
        for (int i = 0; i < n; i++)
        {
            cout << "RECORD " << i + 1 << endl;
            input_struct_fields(group, i);
        }
    }
    else if (op1 == '2' && n > 0)
    {
        char op2;
        do
        {
            group = (Employees *)realloc(group, ++n * sizeof(Employees));
            input_struct_fields(group, n - 1);
            cout << "Do you want to add one more element? (y/n) ";
            cin >> op2;
        } while ((tolower(op2) == 'y'));
    }
    else
    {
        cout << "There is no such operation or array is empty" << endl;
        return;
    }
    write_file(group, n);
}
 
void print_struct(Employees *gr, const int n)
{
    cout << "№ Gender    "
         << "Name    "
         << "Last name    "
         << "Patronymic    "
         << "Salary    "
         << "Year    "
         << "Retired?" << endl;
    for (int i = 0; i < n && gr != NULL; i++)
        cout << i + 1 << " " << gr[i].gender << setw(10) << gr[i].name << setw(10) << gr[i].lastname << setw(10) << gr[i].patronymic << setw(10) << gr[i].salary << setw(5) << gr[i].year << " " << gr[i].isRetired << endl;
}
 
void extract(ifstream &in, char *&str)
{
    int c = 1;
    char ch;
    str = (char*)calloc(c, sizeof(char));
    while (in.get(ch))
    {
        str[c - 1] = ch;
        str = (char*)realloc(str, ++c * sizeof(char));
        if (ch == ' ')
            break;
    }
    str[c - 1] = '\0';
    in >> str;
}
 
void read_data(Employees *&gr, size_t &n)
{
    if(n != 0) { 
        return;
    }
    ifstream in;
    in.open("data.txt");
    in.seekg(73);
    char ch;
    while (in.get(ch)) {
        if(ch == '\n') n++;
    }
    in.close();
    in.open("data.txt");
    in.seekg(73);
    gr = (Employees*)calloc(n, sizeof(Employees));
    for (int i = 0; i < n; i++)
    {
        char gender;
        in >> gender;
        char *name, *lastname, *patronymic;
        extract(in, name);
        extract(in, lastname);
        extract(in, patronymic);
        int sal, year;
        in >> sal >> year;
        char is_ret;
        in >> is_ret;
        gr[i].gender = gender; 
        gr[i].name = name;
        gr[i].lastname = lastname;
        gr[i].patronymic = patronymic;
        gr[i].salary = sal;
        gr[i].year = year;
        gr[i].isRetired = is_ret;
    }
    in.close();
}
 
int validate()
{
    int n;
    while (!(cin >> n) || n <= 0)
    {
        cin.clear();
        std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
        cout << "Enter a correct number: " << endl;
    }
    return n;
}
 
Employees *delete_record(Employees *&gr, size_t &length)
{
    if (length == 0)
        return NULL;
    cout << "Select a number of the record you want to delete: " << endl;
    print_struct(gr, length);
    int n = validate();
    if (n > length)
    {
        cout << "Length is less than a given number" << endl;
        return gr;
    }
    
    Employees *new_group = (Employees*)calloc(n - 1, sizeof(Employees));
    bool flag = false;
    int dyn_ind = 0;
    for (int i = 0; i < length; i++)
    {
        dyn_ind = (flag) ? i - 1 : i;
        if (i + 1 == n && length != 1)
        {
            free(gr[i].name);
            free(gr[i].lastname);
            free(gr[i].patronymic);
            flag = true;
            continue;
        }
        new_group[dyn_ind].gender = gr[i].gender;
        new_group[dyn_ind].name = gr[i].name;
        new_group[dyn_ind].lastname = gr[i].lastname;
        new_group[dyn_ind].patronymic = gr[i].patronymic;
        new_group[dyn_ind].salary = gr[i].salary;
        new_group[dyn_ind].year = gr[i].year;
        new_group[dyn_ind].isRetired = gr[i].isRetired;
    }
    free(gr);
    length--;
    write_file(new_group, length);
    return new_group;
}
 
int main()
{
    char op;
    size_t length = 0;
    Employees *gr = NULL;
    read_data(gr, length);
    while (1)
    {
        cout << endl;
        cout << "Select an operation: " << endl;
        cout << "1. Write data in a file: " << endl;
        cout << "2. Read data from the file" << endl;
        cout << "3. Delete a record from the file" << endl;
        cout << "4. Exit" << endl;
        cin >> op;
        switch (op)
        {
        case '1':
            fill_struct(gr, length);
            break;
        case '2':
            read_data(gr, length);
            print_struct(gr, length);
            break;
        case '3':
            gr = delete_record(gr, length);
            break;
        case '4':
            exit(0);
            break;
        default:
            print_struct(gr, length);
            break;
        }
    }
    return 0;
}

@lemegeton · 18.11.2023, 13:56

Тут проблема не в самом освобождении памяти, а в логике работы кода.
Однако из-за того, что код имеет высокую связанность и низкую связность, его крайне сложно читать и пробегать глазами.

Попробуйте пройтись дебагером и посмотреть, что происходит. )

@TheCalligrapher · 19.11.2023, 11:04

Сообщение от Tony000

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
void extract(ifstream &in, char *&str)
{
    int c = 1;
    char ch;
    str = (char*)calloc(c, sizeof(char));
    while (in.get(ch))
    {
        str[c - 1] = ch;
        str = (char*)realloc(str, ++c * sizeof(char));
        if (ch == ' ')
            break;
    }
    str[c - 1] = '\0';
    in >> str;
}

Что это за загадочная функция? Что она должна делать?

Зачем она читает символы из потока по одному, а затем тут же забивает прочитанные данные, читая целое слово в последней строке. Зачем было читать данные чтобы потом их сразу затереть?

@Tony000 · 20.11.2023, 05:29 **[ТС]**

TheCalligrapher, и вправду. Бред какой-то написал.

Добавлено через 8 минут
TheCalligrapher, убрал эти строки:

C++
1
2
3
4
5
6
7
8
while (in.get(ch))
    {
        str[c - 1] = ch;
        str = (char*)realloc(str, ++c * sizeof(char));
        if (ch == ' ')
            break;
    }
    str[c - 1] = '\0';

Программа работает, несмотря на то, что я выделил память лишь под один элемент массива.

Добавлено через 59 секунд
Оператор >> автоматически выделяет необходимую память?

@TheCalligrapher · 20.11.2023, 05:33

Сообщение от Tony000

C++
1
2
3
4
5
6
7
8
while (in.get(ch))
    {
        str[c - 1] = ch;
        str = (char*)realloc(str, ++c * sizeof(char));
        if (ch == ' ')
            break;
    }
    str[c - 1] = '\0';

Программа работает, несмотря на то, что я выделил память лишь под один элемент массива.

"Лишь под один"? О чем речь? А как же realloc?

Сообщение от Tony000

Оператор >> автоматически выделяет необходимую память?

Нет, он вообще ничего не выделяет. Более того, начиная с С++20 такое применение >> к указателю невозможно.

@Tony000 · 20.11.2023, 05:36 **[ТС]**

TheCalligrapher, работает, как и прежде. Бред.

Добавлено через 1 минуту
TheCalligrapher, имею ввиду, выделил память лишь под один символ, а он записывает туда больше одного символа без проблем.

@TheCalligrapher · 20.11.2023, 05:38

Сообщение от Tony000

имею ввиду, выделил память лишь под один символ, а он записывает туда больше одного символа без проблем.

Еще раз: О чем речь? А как же realloc?

@Tony000 · 20.11.2023, 05:50 **[ТС]**

TheCalligrapher, я и пишу, что всё работает без realloc. Вот немного исправленный код:

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
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
#include <iostream>
#include <fstream>
#include <ctime>
#include <cstring>
#include <limits>
#include <iomanip>
 
using namespace std;
 
struct Employees
{
    char *name, *lastname, *patronymic;
    char gender, isRetired;
    int salary, year;
};
 
int validate();
void get_string(char *&word);
 
void get_string(char *&word)
{
    char c;
    word = (char *)calloc(1, sizeof(char));
    unsigned ctr = 1;
    while (cin.get(c))
    {
        if (c == '\n' || c == ' ')
            break;
        word[ctr - 1] = c;
        word = (char *)realloc(word, ++ctr * sizeof(char));
    }
    word[ctr - 1] = '\0';
}
 
void write_file(Employees *&st, size_t n)
{
    ofstream out;
    out.open("data.txt");
    if (!out.is_open())
    {
        cout << "There is no file";
        return;
    }
    out << "Gender    "
        << "Name    "
        << "Last name    "
        << "Patronymic    "
        << "Salary    "
        << "Year    "
        << "Retired?" << endl;
    for (int i = 0; i < n; i++)
    {
        out << st[i].gender << "         ";
        out << st[i].name << "      ";
        out << st[i].lastname << "     ";
        out << st[i].patronymic << "      ";
        out << st[i].salary << "      ";
        out << st[i].year << "      ";
        out << st[i].isRetired << endl;
    }
    out.close();
}
 
void input_struct_fields(Employees *&group, int i)
{
    cin.ignore(1);
    int current_year = 1970 + time(NULL) / 31536000;
    cout << "Enter a gender (f/m): " << endl;
    cin >> group[i].gender;
    while (tolower(group[i].gender) != 'm' && tolower(group[i].gender) != 'f')
    {
        cout << "Wrong input data. Try again.." << endl;
        while (getchar() != '\n')
            ;
        cin >> group[i].gender;
    }
    cin.ignore(1);
    cout << "Enter a name: " << endl;
    get_string(group[i].name);
    cout << "Enter a last name: " << endl;
    get_string(group[i].lastname);
    cout << "Enter a patronymic: " << endl;
    get_string(group[i].patronymic);
    cout << "Enter a salary: " << endl;
    group[i].salary = validate();
    cout << "Enter an year: " << endl;
    group[i].year = validate();
    int age = current_year - group[i].year;
    group[i].isRetired = (tolower(group[i].gender) == 'f' &&
                          (55 - age > 0 && 55 - age <= 3)) ||
                                 ((tolower(group[i].gender) == 'm') && (60 - age > 0 && 60 - age <= 3))
                             ? 'Y'
                             : 'N';
}
 
void fill_struct(Employees *&group, size_t &n)
{
    cout << "Do you want:\n1. to rewrite data in the file\n2. to add data in the existing file" << endl;
    char op1;
    cin >> op1;
    if (op1 == '1')
    {
        cout << "How many records do you want to fill? " << endl;
        n = validate();
        group = (Employees *)calloc(n, sizeof(Employees));
        for (int i = 0; i < n; i++)
        {
            cout << "RECORD " << i + 1 << endl;
            input_struct_fields(group, i);
        }
    }
    else if (op1 == '2' && n > 0)
    {
        char op2;
        do
        {
            group = (Employees *)realloc(group, ++n * sizeof(Employees));
            input_struct_fields(group, n - 1);
            cout << "Do you want to add one more element? (y/n) ";
            cin >> op2;
        } while ((tolower(op2) == 'y'));
    }
    else
    {
        cout << "There is no such operation or array is empty" << endl;
        return;
    }
    write_file(group, n);
}
 
void print_struct(Employees *gr, const int n)
{
    cout << "№ Gender    "
         << "Name    "
         << "Last name    "
         << "Patronymic    "
         << "Salary    "
         << "Year    "
         << "Retired?" << endl;
    for (int i = 0; i < n && gr != NULL; i++)
        cout << i + 1 << " " << gr[i].gender << setw(10) << gr[i].name << setw(10) << gr[i].lastname << setw(10) << gr[i].patronymic << setw(10) << gr[i].salary << setw(5) << gr[i].year << " " << gr[i].isRetired << endl;
}
 
void extract(ifstream &in, char *&str)
{
    int c = 1;
    char ch;
    str = (char*)calloc(c, sizeof(char));
    in >> str;
}
 
void read_data(Employees *&gr, size_t &n)
{
    if(n != 0) { 
        return;
    }
    ifstream in;
    in.open("data.txt");
    in.seekg(73);
    char ch;
    while (in.get(ch)) {
        if(ch == '\n') n++;
    }
    in.close();
    in.open("data.txt");
    in.seekg(73);
    gr = (Employees*)calloc(n, sizeof(Employees));
    for (int i = 0; i < n; i++)
    {
        in >> gr[i].gender;
        extract(in, gr[i].name);
        extract(in, gr[i].lastname);
        extract(in, gr[i].patronymic);
        in >> gr[i].salary >> gr[i].year >> gr[i].isRetired;
    }
    in.close();
}
 
int validate()
{
    int n;
    while (!(cin >> n) || n <= 0)
    {
        cin.clear();
        std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
        cout << "Enter a correct number: " << endl;
    }
    return n;
}
 
Employees *delete_record(Employees *&gr, size_t &length)
{
    if (length == 0)
        return NULL;
    cout << "Select a number of the record you want to delete: " << endl;
    print_struct(gr, length);
    int n = validate();
    if (n > length)
    {
        cout << "Length is less than a given number" << endl;
        return gr;
    }
    
    Employees *new_group = (Employees*)calloc(n - 1, sizeof(Employees));
    bool flag = false;
    int dyn_ind = 0;
    for (int i = 0; i < length; i++)
    {
        dyn_ind = (flag) ? i - 1 : i;
        if (i + 1 == n && length != 1)
        {
            free(gr[i].name);
            free(gr[i].lastname);
            free(gr[i].patronymic);
            flag = true;
            continue;
        }
        new_group[dyn_ind].gender = gr[i].gender;
        new_group[dyn_ind].name = gr[i].name;
        new_group[dyn_ind].lastname = gr[i].lastname;
        new_group[dyn_ind].patronymic = gr[i].patronymic;
        new_group[dyn_ind].salary = gr[i].salary;
        new_group[dyn_ind].year = gr[i].year;
        new_group[dyn_ind].isRetired = gr[i].isRetired;
    }
    free(gr);
    length--;
    write_file(new_group, length);
    return new_group;
}
 
int main()
{
    char op;
    size_t length = 0;
    Employees *gr = NULL;
    read_data(gr, length);
    while (1)
    {
        cout << endl;
        cout << "Select an operation: " << endl;
        cout << "1. Write data in a file: " << endl;
        cout << "2. Read data from the file" << endl;
        cout << "3. Delete a record from the file" << endl;
        cout << "4. Exit" << endl;
        cin >> op;
        switch (op)
        {
        case '1':
            fill_struct(gr, length);
            break;
        case '2':
            read_data(gr, length);
            print_struct(gr, length);
            break;
        case '3':
            gr = delete_record(gr, length);
            cout << gr[0].name;
            break;
        case '4':
            exit(0);
            break;
        }
    }
    return 0;
}

Добавлено через 9 минут
TheCalligrapher, ещё хочу добавить, что такое поведение характерно только для чтения из файла. При вводе данных с клавиатуры (функция get_string) без realloc программа работает некорректно.

@DrOffset · 20.11.2023, 08:45

Сообщение от Tony000

всё работает без realloc

Это просто UB.

@Tony000 · 21.11.2023, 15:26 **[ТС]**

C++
1
2
3
4
5
6
7
8
while (in.get(ch))
    {
        str[c - 1] = ch;
        str = (char*)realloc(str, ++c * sizeof(char));
        if (ch == ' ')
            break;
    }
    str[c - 1] = '\0';

Вернул эти строки кода. Оказывается, что in.get() ничего не считывает, соответственно память не выделяется должным образом. Я не знаю, что делать.

@nmcf · 21.11.2023, 16:40

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

Добавлено через 8 минут

Не по теме:

И почему гендеров так мало? :)

@Tony000 · 22.11.2023, 12:42 **[ТС]**

nmcf, файл открывается.

Сообщение от nmcf

И зачем тебе извлекать побайтно что-то, если сразу же перезаписываешь, но без учёта размера?

Это я уже исправил. Ничего не изменилось.

Новые блоги и статьи Все статьи Все блоги /
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Рецензия / Мнение Это мой обзор планшета X220 с точки зрения школьника. Недавно я решила попытаться уменьшить свой. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .

@lemegeton 4903 / 2696 / 921 Регистрация: 29.11.2010 Сообщений: 5,783
	18.11.2023, 13:00
	У вас какая-то дикая помесь C и C++, причем совершенно непонятно, чего вы хотите достичь. Вы надёргали откуда-то кусками код и пытаетесь в него привнести дополнительный функционал? Очень не хочется разбираться где ошибка в таком запутанном коде. Не должна такая простая задача исполняться в столь сложно читаемом стиле. Моя рекомендация -- переписать всё, что связано с выделением и освобождением памяти, на С++ лад. Желательно с использованием классов. Какое у вас изначальное задание? 1

@lemegeton 4903 / 2696 / 921 Регистрация: 29.11.2010 Сообщений: 5,783
	18.11.2023, 13:56
	Тут проблема не в самом освобождении памяти, а в логике работы кода. Однако из-за того, что код имеет высокую связанность и низкую связность, его крайне сложно читать и пробегать глазами. Попробуйте пройтись дебагером и посмотреть, что происходит. ) 0

@Tony000 1 / 1 / 1 Регистрация: 19.10.2023 Сообщений: 22
	20.11.2023, 05:36 [ТС]
	TheCalligrapher, работает, как и прежде. Бред. Добавлено через 1 минуту TheCalligrapher, имею ввиду, выделил память лишь под один символ, а он записывает туда больше одного символа без проблем. 0

@nmcf 7804 / 6568 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	21.11.2023, 16:40
	Файл вообще открывается? Проверяй. И зачем тебе извлекать побайтно что-то, если сразу же перезаписываешь, но без учёта размера? Добавлено через 8 минут Не по теме: И почему гендеров так мало? :) 0