Ошибки работы с динамической памятью

@_-_VLADoS_-_ · Регистрация: 27.12.2019

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

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
#include<iostream>
#include<cstdlib>
#include<string>
 
using namespace std;
 
class Human
{
public:
    Human(string name = "UNKNOW", int year=0000, char gender='A');
    Human(Human& other);
    void print();
    void input();
    ~Human();
private:
    string name;
    int year;
    char gender;
};
 
Human::Human(string name, int year, char gender) :name(name),year(year),gender(gender) {}
 
Human::Human(Human& other): name(other.name), year(other.year), gender(other.gender) {}
 
void Human::print()
{
    cout << "Name: " << this->name << endl
         << "Year of birth: " << this->year << endl
         << "Gender: " << this->gender << endl;
}
 
void Human::input()
{
    cout << "Name: "; cin >> this->name;
    cout << "Year of birth: "; cin >> this->year;
    cout << "Gender: "; cin >> this->gender;
}
 
Human::~Human() = default;
 
class Student : public Human
{
public:
 
    Student(string name = "UNKNOW", int year = 0000, char gender = 'A', int university_entrance=0000,int book_of_registration=0000, int amount_subject=0,int avg_marks=0.0);
    Student(Student& other);
    void _print();
    void _input();
    float avg_mark_func();
    void add_new(string subject,float mark);
     ~Student();
private:
 
    int university_entrance;
    int book_of_registration;
    int amount_subject;
    string* subjects;
    float* marks;
    float avg_marks;
};
 
Student::Student(string name, int year, char gender, int university_entrance, int book_of_registration, int amount_subject, int avg_marks) : Human(name,year,gender),
university_entrance(university_entrance), book_of_registration(book_of_registration), amount_subject(amount_subject),subjects(new string[amount_subject]),marks(new float[amount_subject]),avg_marks(avg_marks)
{
    for (int i = 0; i < this->amount_subject; i++)
    {
        this->subjects[i] = "UNKNOW";
        this->marks[i] = 0.0;
    }
}
 
Student::Student(Student& other):Human(other), university_entrance(other.university_entrance), book_of_registration(other.book_of_registration), amount_subject(other.amount_subject), subjects(new string[other.amount_subject]), marks(new float[other.amount_subject]), avg_marks(other.avg_marks)
{
    for (int i = 0; i < this->amount_subject; i++)
    {
        this->subjects[i] = other.subjects[i];
        this->marks[i] = other.marks[i];
    }
}
 
void Student::_print()
{
    print();
 
    cout << "University entrance: " << this->university_entrance << endl
         << "Book of registration: " << this->book_of_registration << endl
         << "Amount subject: " << this->amount_subject << endl;
    for (int i = 0; i < this->amount_subject; i++)
    {
        cout << "Subject: " << this->subjects[i] << " mark: " << this->marks[i] << endl;
    }
    cout << "Avg. mark: " << this->avg_marks<< endl;
}
 
void Student::_input()
{
    input();
    cout << "University entrance: "; cin >> this->university_entrance;
    cout << "Book of registration: "; cin >> this->book_of_registration;
    cout << "Amount subject: "; cin >> this->amount_subject;
    for (int i = 0; i < this->amount_subject; i++)
    {
        cout << "Subject: "; cin >> this->subjects[i];  cout << "mark: "; cin>> this->marks[i];
    }
}
 
float Student::avg_mark_func()
{
 
    for (int i = 0; i < this->amount_subject; i++)
    {
        this->avg_marks += this->marks[i];
    }
    this->avg_marks /= this->amount_subject;
    return this->avg_marks;
}
 
void Student::add_new(string subject, float mark)
{
    Student temp;
    if (this->amount_subject != 0)
    {
        temp.marks = new float[amount_subject + 1];
        temp.subjects = new string[amount_subject + 1];
        for (int i = 0; i < this->amount_subject; i++)
        {
            temp.marks[i] = this->marks[i];
            temp.subjects[i] = this->subjects[i];
        }
        temp.marks[amount_subject] = mark;
        temp.subjects[amount_subject] = subject;
        delete[] this->marks;
        delete[] this->subjects;
        this->amount_subject++;
        this->marks = new float[this->amount_subject];
        for (int i = 0; i < this->amount_subject; i++)
        {
            this->marks[i] = temp.marks[i];
            this->subjects[i] = temp.subjects[i];
        }
        delete[]temp.marks;
        delete[]temp.subjects;
    }
    else
    {
        delete[]this->marks;
        this->amount_subject++;
        this->marks = new float[amount_subject];
        this->subjects = new string[amount_subject];
        marks[0] = mark;
        subjects[0] = subject;
    }
 
    
}
 
Student::~Student()
{
    delete[] this->subjects;
    delete[] this->marks;
    subjects = nullptr; marks = nullptr;
}
 
 
int main(void)
{
    Human a;
    a.print();
    Student b;
    b.add_new("math", 5);
    b.add_new("biology", 3);
    b.add_new("reading", 4);
    b._print();
 
    system("pause > nul");
    return EXIT_SUCCESS;
}

Тут ошибка!!!!

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
void Student::add_new(string subject, float mark)
{
    Student temp;
    if (this->amount_subject != 0)
    {
        temp.marks = new float[amount_subject + 1];
        temp.subjects = new string[amount_subject + 1];
        for (int i = 0; i < this->amount_subject; i++)
        {
            temp.marks[i] = this->marks[i];
            temp.subjects[i] = this->subjects[i];
        }
        temp.marks[amount_subject] = mark;
        temp.subjects[amount_subject] = subject;
        delete[] this->marks;
        delete[] this->subjects;
        this->amount_subject++;
        this->marks = new float[this->amount_subject];
        for (int i = 0; i < this->amount_subject; i++)
        {
            this->marks[i] = temp.marks[i];
            this->subjects[i] = temp.subjects[i];
        }
        delete[]temp.marks;
        delete[]temp.subjects;
    }
    else
    {
        delete[]this->marks;
        this->amount_subject++;
        this->marks = new float[amount_subject];
        this->subjects = new string[amount_subject];
        marks[0] = mark;
        subjects[0] = subject;
    }

@ivan37 · 22.02.2020, 23:10

В 18 строчке выделяется память только для marks, но не для subjects. Необходимо добавить

C++
1
this->subjects = new string[this->amount_subject];

Ну еще память кораптится при освобождении в 24 и 25 строчках, хотя я не понял из-за чего (если их закоментить, то падения не будет).
Вообще лучше не выполнять каждый раз перевыделение памяти подобным образом, поскольку при увеличении числа записей увеличивается время добавления новой записи, лучше использовать линейные списки или стандартные контейнеры, такие как list или vector.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
include <vector>
// ...
class Student
{
    // ...
    vectors<float> marks;
 
    void add_new(string subject, float mark)
    {
        //...
        this->marks.push_back(mark);
    }
}

Добавлено через 4 минуты
а, понял, почему падение возникает, вы сначала освобождаете память для marks и subjects в 24, 25 строчках, а затем при выходе из функции вызывается деструктор для Students и таким образом выполняется повторное освобождение памяти, которое приводит к ошибке.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .	SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm 8Observer8 30.01.2026 Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .	SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .	SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .