Heap corruption detected

@sicknessed · Регистрация: 20.03.2013

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

Добрый день! Проблема вот в чём: когда выполнение кода доходит до 163 строки, появляется ошибка: " HEAP CORRUPTION DETECTED: after Normal block (#752) at 0x0038A5D8. CRT detected that the application wrote to memory after end of heap buffer". Получается, что это ошибка происходит при вызове деструктора моего класса. Но, вот, в чём именно там ошибка, я, увы, разобраться не могу.

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
#include <iostream>
#include <string>
#include <fstream>
#include <conio.h>
 
class Matr {
    public:
        Matr(const int row, const int column);
        ~Matr();
        void printMatr(int count);
        bool addStr(char* str, int number);
        void setNumberOfColumn(int i);
        int getNumberOfColumn();
        int getNumberOfRow();
        void sortBubble();
        void sortSelection();
        char** body;
    private:
        int numberOfColumn;
        int numberOfRow;
        static int numberOfBubbleCompare;
        static int numberOfBubbleChange;
        static int numberOfSelectionCompare;
        static int numberOfSelectionChange;
};
 
Matr::Matr(const int row = 1, const int column = 0){
    body = new char*[row];
    for(int i = 0; i < column; i++){
        body[i] = new char[column];
    }
    numberOfColumn = column;
    numberOfRow = row;
}
 
Matr::~Matr(){
    for(int i = 0; i < numberOfRow; i++){
        delete body[i];
    }
    delete[] body;
}
 
void Matr::setNumberOfColumn(int i){
    numberOfColumn = i;
}
 
int Matr::getNumberOfColumn(){
    return numberOfColumn;
}
 
int Matr::getNumberOfRow(){
    return numberOfRow;
}
 
bool Matr::addStr(char* str, int number){
    using namespace std;
    /*if (sizeof(str)/sizeof(char*) != numberOfColumn){
        cout << "Неверное количество столбцов! Это не может быть матрицей!" << endl;
        return false;
    }*/
    for(int i = 0; i <= numberOfColumn; i++){
        this->body[number][i] = str[i];
    }
    return true;
}
 
void Matr::printMatr(int count){
    using namespace std;
    for(int i = 0; i <= count; i++){
        for(int k = 0; k < numberOfColumn; k++){
            cout << body[i][k] << " ";
        }
        cout << endl;
    }
}
 
void Matr::sortBubble(){
    for(int j = 0; j < numberOfRow; j++){
        for(int i = 1; i < numberOfColumn; ++i){
            for(int k = numberOfColumn-1; k >= i; --k) { 
                if(body[j][k-1] > body[j][k]) { 
                    char t = body[j][k-1]; 
                    body[j][k-1] = body[j][k]; 
                    body[j][k] = t; 
                    numberOfBubbleChange++;
                }
                numberOfBubbleCompare++;
            } 
        }
    }
}
 
void Matr::sortSelection(){
    for(int k = 0; k < numberOfRow; k++){
        for(int i = 0; i < numberOfColumn-1; ++i) { 
            bool flag = false; 
            int c = i; 
            char t = body[k][i]; 
            for(int b = i+1; b < numberOfColumn; ++b) { 
                if(body[k][b] < t) { 
                    c = b; 
                    t = body[k][b]; 
                    flag = true; 
                }
                numberOfSelectionCompare++;
            } 
            if(flag) { 
                body[k][c] = body[k][i]; 
                body[k][i] = t; 
                numberOfSelectionChange;
            } 
        } 
    }
}
 
 
int Matr::numberOfBubbleCompare = 0;
int Matr::numberOfBubbleChange = 0;
int Matr::numberOfSelectionCompare = 0;
int Matr::numberOfSelectionChange = 0;
 
void deleteChar(char *string, int number){
    for(int k = number; k < strlen(string); k++){
        string[k] = string[k+1];
    }
}
 
int deleteSpaces(char* str){
    int i = 0;
    while(str[i] != '\n'){
        if(isspace(str[i])){
            deleteChar(str, i);
        }else{
            i++;
        }
    }
    return i;
}
int main(){
    using namespace std;
    setlocale(0,"");
    bool flagOfRepeat = false;
    do{
        cout << "Введите имя файла, из которого нужно считать матрицу: ";
        string nameOfFile;
        cin >> nameOfFile;
        ifstream f(nameOfFile);
        if(f && !f.eof()){
            char* p = new char[255];
            f.getline(p, 255);
            p[strlen(p)] = '\n';
            Matr matr(10,deleteSpaces(p));                  //Создание матрицы и удаление из р пробелов
            int countOfRow = 0;
            matr.addStr(p, countOfRow);
            while(!f.eof()){
                countOfRow++;
                f.getline(p,255);
                p[strlen(p)] = '\n';
                deleteSpaces(p);
                matr.addStr(p, countOfRow);
            }
            matr.printMatr(countOfRow);         
        }else{  
            cout << "Ошибка открытия файла!" << endl;
        }
        cout << "Хотите считать данные из другого файла? Если да, введите 1, нет - любую другую клавишу: ";
        char i;
        cin >> i;
        if(i != '1'){
            flagOfRepeat = false;
        }
    }while(flagOfRepeat);
    _getch();
return 0;
}

Буду благодарен за помощь!

Убежденный · 12.09.2013, 18:07

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

Получается, что это ошибка происходит при вызове деструктора моего класса.

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

@Kuzia domovenok · 12.09.2013, 18:17

где конструктор копий, где оператор =
?
ну просто типичнейшая ошибка.

@sicknessed · 12.09.2013, 18:57 **[ТС]**

Сообщение от Kuzia domovenok

где конструктор копий, где оператор =
?
ну просто типичнейшая ошибка.

Зачем они, если я их не использую в своей программе? Или их отсутствие может вызвать ошибку?

@coloc · 12.09.2013, 19:01

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

если я их не использую

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

@sicknessed · 12.09.2013, 20:22 **[ТС]**

Дописал и конструктор копий, и присваивание:

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
Matr::Matr(const Matr &M){
    numberOfColumn = M.numberOfColumn;
    numberOfRow = M.numberOfRow;
    body = new char*[numberOfRow];
    for(int i = 0; i < numberOfRow; i++){
        body[i] = new char[numberOfColumn];
    }
    for(int i = 0; i < numberOfRow; i++){
        for(int k = 0; k < numberOfColumn; k++){
            body[i][k] = M.body[i][k];
        }
    }
}
 
const Matr& Matr::operator = (const Matr& M){
    using namespace std;
    if(numberOfRow == M.numberOfRow && numberOfColumn == M.numberOfColumn){
        for(int i = 0; i < numberOfRow; i++){
            for(int k = 0; k < numberOfColumn; k++){
                body[i][k] = M.body[i][k];
            }
        }
        return *this;
    }else{
        cout << "Невозможно выполнить присваивание! Размеры не совпадают!" << endl; 
    }
}

Но от этого ошибка не исчезла.

alsav22 · 12.09.2013, 20:32

Содержимое файла покажите.

Убежденный · 12.09.2013, 20:34

sicknessed, учитесь сводить задачу к минимально воспроизводимому примеру:

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
#include <iostream>
 
 
 
class Matr {
    public:
        Matr(const int row, const int column);
        ~Matr();
        char** body;
    private:
        int numberOfColumn;
        int numberOfRow;
};
 
 
 
Matr::Matr(const int row = 1, const int column = 0){
    body = new char*[row];
    for(int i = 0; i < column; i++){
        body[i] = new char[column];
    }
    numberOfColumn = column;
    numberOfRow = row;
}
 
 
 
Matr::~Matr(){
    for(int i = 0; i < numberOfRow; i++){
        delete body[i];
    }
    delete[] body;
}
 
 
 
 
int main(){
    Matr matr(10,5);
return 0;
}

Вылетает с segmentation fault.
По-прежнему нет идей, где искать ошибку ?

@Kuzia domovenok · 12.09.2013, 20:39

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

for(int i = 0; i <= numberOfColumn; i++)

Почему тут неравенство нестрогое???

alsav22 · 12.09.2013, 20:42

C++
1
2
3
4
5
6
7
8
Matr::Matr(const int row = 1, const int column = 0){
    body = new char*[row];
    for(int i = 0; i < column; i++){ // !!! i < row
        body[i] = new char[column];
    }
    numberOfColumn = column;
    numberOfRow = row;
}

@sicknessed · 12.09.2013, 21:29 **[ТС]**

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

C++
1
2
3
4
5
6
7
8
Matr::Matr(const int row = 1, const int column = 0){
    body = new char*[row];
    for(int i = 0; i < column; i++){ // !!! i < row
        body[i] = new char[column];
    }
    numberOfColumn = column;
    numberOfRow = row;
}

Спасибо большое. Заработало!

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование 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 с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

@Kuzia domovenok 4268 / 3327 / 926 Регистрация: 25.03.2012 Сообщений: 12,532 Записей в блоге: 1
	12.09.2013, 18:17
	где конструктор копий, где оператор = ? ну просто типичнейшая ошибка. 0

alsav22 5500 / 4895 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	12.09.2013, 20:32
	Содержимое файла покажите. 0