Освобождение памяти динамического массива. Деструктор

@Ilya2016 · Регистрация: 22.11.2015

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

Почему выдает ошибку при написании деструктора? Если его убрать, то все работает.

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
#pragma once
 
#ifndef MATRIX_H
 
#define MATRIX_H
 
#include <ctime>
#include <cstdlib>
#include <iostream>
 
using namespace std;
 
class Matrix
{
    int dimension;
    double * pointer;
public:  int counter = 0; // Для подсчета количества созданных объектов
 
public:
    Matrix(int dim = 2) {
        
        counter++;
        dimension = dim;
        pointer = new double[dim*dim];
        srand(time(NULL)); // инициализация функции rand значением функции time
        for (int i = 0; i < dim*dim; i++)
            pointer[i] = rand() % 14 + 5;
    }
    Matrix(const Matrix&);
    void point();
    ~Matrix() { delete []pointer; }
 
    void get(int, int);
    void set(int, int, double);
    Matrix operator+(const Matrix&);
    
};
 
#endif // !MATRIX_H

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
#include"Matrix.h"
 
//Matrix::Matrix
 
void Matrix::point() {
    for(int i = 0; i < dimension; i++)
    {
        for(int j = 0; j < dimension; j++)
            cout << pointer[i*dimension + j] << "   ";
        cout << endl;
    }
    
}
 
Matrix::Matrix(const Matrix& oldObject) {
    dimension = oldObject.dimension;
 
    pointer = new double[dimension*dimension];
    for (int i = 0; i < dimension; i++)
        for (int j = 0; j < dimension; j++)
            pointer[i*dimension + j] = oldObject.pointer[i*dimension + j];
    
 
}
 
 
 
void Matrix::get(int i, int j) {
    if (i > dimension || i<0 || j>dimension || j < 0)
        cout << "Столбец или строчка введена неверно";
    else cout << pointer[i*dimension + j] << "/t";
}
void Matrix::set(int i, int j, double element) {
    if (i > dimension || i<0 || j>dimension || j < 0)
        cout << "Столбец или строчка введена неверно";
    else pointer[i*dimension + j] = element;
}
 
Matrix Matrix::operator+(const Matrix& M2) {
    Matrix temp(*this);
    for (int i = 0; i < dimension; i++)
    {
        for (int j = 0; j < dimension; j++)
        {
            temp.pointer[i*dimension + j] += M2.pointer[i*dimension + j];
        }
    }
    return temp;
}

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
#include"Matrix.h"
 
int main() {
    Matrix mat;
    Matrix mat1;
    Matrix mat2;
    
    mat2.point();
    mat2 = mat + mat1;
    //Matrix mat1(mat);
    mat2.point();
    
    
    return 0;
}

@VAN0 · 05.10.2016, 20:07

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

@TheCalligrapher · 05.10.2016, 20:12

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

Почему выдает ошибку при написании деструктора?

Нарушено Правило Трех: отсутствует корректно реализованный копирующий оператор присваивания. Конструктор копирования есть, деструктор есть, а оператора присваивания нет. А в программе он используется...

P.S. В чем смысл метода void Matrix::get(int i, int j), который ничего не делает - не ясно.

@Ilya2016 · 05.10.2016, 22:28 **[ТС]**

VAN0, TheCalligrapher, Изменил. Что в это раз не так?

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
#pragma once
 
#ifndef MATRIX_H
 
#define MATRIX_H
 
#include <ctime>
#include <cstdlib>
#include <iostream>
 
using namespace std;
 
class Matrix
{
    int dimension;
    double * pointer;
public:  int counter = 0; // Для подсчета количества созданных объектов
 
public:
    Matrix(int dim = 2) {
        
        counter++;
        dimension = dim;
        pointer = new double[dim*dim];
        srand(time(NULL)); // инициализация функции rand значением функции time
        for (int i = 0; i < dim*dim; i++)
            pointer[i] = rand() % 14 + 5;
 
 
 
    
    }
    Matrix (const Matrix&);
    void point();
    ~Matrix() { delete []pointer; cout << "Деструктор" << endl;}
 
    void get(int, int);
    void set(int, int, double);
    Matrix& operator+(const Matrix&);
    Matrix& operator=(const Matrix&);
 
    //Matrix operator*(const Matrix&);
    
    // ---- Конструктор инициализации ( случайные числа от 5 до 14 включительно ) +
    // ---- Копирующий конструктор +
    // ---- Все конструкторы должны содержать код обеспечивающий корректную        // работу со счетчиком класса при создании нового объекта как на стеке так и в    // динамической памяти
    // ---- Деструктор ( работа с памятью + корректировка счетчика перед уничтожением +
    //       объекта
    // ---- Функция для получения значений матрицы +
    // ---- Функция для задания значений отдельным ячейкам матрицы +
    // ---- Форматированная печать матрицы +
    // ---- Функция для сложения двух матриц  ( перегруженная операторная функция)
    // ---- Функция умножения матрицы на число ( перегруженная операторная функция)
    // ---- Статическая функция класса для работы со статическим атрибутом
};
 
#endif // !MATRIX_H

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
#include"Matrix.h"
 
//Matrix::Matrix
 
void Matrix::point() {
    for(int i = 0; i < dimension; i++)
    {
        for(int j = 0; j < dimension; j++)
            cout << pointer[i*dimension + j] << "   ";
        cout << endl;
    }
    
}
 
Matrix::Matrix(const Matrix& oldObject) {
    cout << "Конструктор копирования" << endl;
    dimension = oldObject.dimension;
    pointer = new double[dimension];
    for (int i = 0; i < dimension; i++)
        for (int j = 0; j < dimension; j++)
            pointer[i*dimension + j] = oldObject.pointer[i*dimension + j];
}
 
Matrix& Matrix::operator=(const Matrix& oldObject) {
    if (this != &oldObject)
    {
        cout << "Перегруженный оператор присваивания" << endl;
        delete[]pointer;
        dimension = oldObject.dimension;
        pointer = new double[dimension];
        for (int i = 0; i < dimension; i++)
            for (int j = 0; j < dimension; j++)
                pointer[i*dimension + j] = oldObject.pointer[i*dimension + j];
    }
    else cout << "Самоприсваивание" << endl;
    return *this;
}
 
void Matrix::get(int i, int j) {
    if (i > dimension || i<0 || j>dimension || j < 0)
        cout << "Столбец или строчка введена неверно";
    else cout << pointer[i*dimension + j] << "/t";
}
void Matrix::set(int i, int j, double element) {
    if (i > dimension || i<0 || j>dimension || j < 0)
        cout << "Столбец или строчка введена неверно";
    else pointer[i*dimension + j] = element;
}
 
Matrix& Matrix::operator+(const Matrix& M2) {
    //Matrix temp(*this);
    for (int i = 0; i < dimension; i++)
    {
        for (int j = 0; j < dimension; j++)
        {
            pointer[i*dimension + j] += M2.pointer[i*dimension + j];
        }
    }
    return *this;
}
    
 
/*Matrix Matrix::operator*(const Matrix& M2) {
 
    Matrix temp(*this);
    for (int i = 0; i < dimension; i++) {
        for (int j = 0; j < dimension; j++) {
            temp.pointer[i*dimension + j] = 0;
            for (int k = 0; k < dimension; k++)
                temp.pointer[i*dimension + j] += pointer[i*dimension + k] * M2.pointer[k*dimension + j];
 
 
        }
 
 
    }
    
 
    return temp;
}*/

@TheCalligrapher · 05.10.2016, 23:07

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

Что в это раз не так?

В операторе присваивания памяти выделяется недостаточно. Судя по конструктору Matrix:Matrix() вы сами знаете, что выделять массив надо размера dimension*dimension, а вы в операторе присваивания выделяете только dimension. Та же проблема с конструктором копирования. Может надо как-то самому учиться отлавливать такие "детские" баги?

И теперь вы зачем-то искорежили свой оператор +. В первой версии было все нормально (по крайней мере внешне), а теперь вы из оператора + вдруг ни стого ни с сего сделали некое подобие оператора +=. Зачем? Почему?

@Ilya2016 · 05.10.2016, 23:47 **[ТС]**

TheCalligrapher, Почему-то в функцию

C++
1
Matrix& Matrix::operator=(const Matrix& oldObject) {

oldObject передается со значением 0x00bdfda8 {dimension=-858993460 pointer=0xcccccccc {???} counter=-858993460 } Ив роде бы после

C++
1
2
3
4
5
6
7
8
9
10
11
Matrix Matrix::operator+(const Matrix& M2) {
    Matrix temp(*this);
    for (int i = 0; i < dimension; i++)
    {
        for (int j = 0; j < dimension; j++)
        {
            temp.pointer[i*dimension + j] += M2.pointer[i*dimension + j];
        }
    }
    return temp;
}

вызывается деструктор для this и удаляет temp

@TheCalligrapher · 06.10.2016, 00:04

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

вызывается деструктор для this и удаляет temp

Мне трудно представить, о чем идет речь, не видя финальной версии кода, в т.ч. вызывающего кода. Как выглядит код?

@Ilya2016 · 06.10.2016, 00:05 **[ТС]**

TheCalligrapher,

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
#include"Matrix.h"
 
int main() {
    Matrix mat;
    Matrix mat1;
    Matrix mat2;
    
    mat2.point();
    mat2 = mat + mat1;
    //Matrix mat1(mat);
    mat2.point();
    
    
    return 0;
}

@TheCalligrapher · 06.10.2016, 00:11

Никаких проблем с oldObject на входе в operator = я не вижу.

После выхода из operator + (подразумевая, что вы вернули его к прежнему виду) вызывается деструктор временного объекта - это тоже нормально. Что такое "вызывается деструктор для this" я не понял.

Новые блоги и статьи Все статьи Все блоги /
Изучаю kubernetes lagorue 13.01.2026 А пригодятся-ли мне знания kubernetes в России?	Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .
Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .	Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .

@VAN0 59 / 59 / 53 Регистрация: 05.05.2013 Сообщений: 150
	05.10.2016, 20:07
	Сообщение было отмечено Ilya2016 как решение Решение При копировании экземпляров класса с динамически выделенной памятью необходимо явно определить конструктор копирования и оператор = У вас фактически происходит двойное освобождение выделенной памяти, т. к. конструктор копирования по умолчанию осуществляет поверхностное копирование 1

@TheCalligrapher Вездепух 12931 / 6799 / 1820 Регистрация: 18.10.2014 Сообщений: 17,210
	06.10.2016, 00:11
	Никаких проблем с `oldObject` на входе в `operator =` я не вижу. После выхода из `operator +` (подразумевая, что вы вернули его к прежнему виду) вызывается деструктор временного объекта - это тоже нормально. Что такое "вызывается деструктор для this" я не понял. 0

Освобождение памяти динамического массива. Деструктор

Решение