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

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

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

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

C++

#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++

#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++

#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++

#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++

#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++

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++

#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" я не понял.

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

@TheCalligrapher Вездепух 12792 / 6669 / 1795 Регистрация: 18.10.2014 Сообщений: 16,877
	05.10.2016, 20:12	3
	Сообщение от Ilya2016 Почему выдает ошибку при написании деструктора? Нарушено Правило Трех: отсутствует корректно реализованный копирующий оператор присваивания. Конструктор копирования есть, деструктор есть, а оператора присваивания нет. А в программе он используется... P.S. В чем смысл метода `void Matrix::get(int i, int j)`, который ничего не делает - не ясно. 1

@TheCalligrapher Вездепух 12792 / 6669 / 1795 Регистрация: 18.10.2014 Сообщений: 16,877
	05.10.2016, 23:07	5
	Сообщение от Ilya2016 Что в это раз не так? В операторе присваивания памяти выделяется недостаточно. Судя по конструктору `Matrix:Matrix()` вы сами знаете, что выделять массив надо размера `dimension*dimension`, а вы в операторе присваивания выделяете только `dimension`. Та же проблема с конструктором копирования. Может надо как-то самому учиться отлавливать такие "детские" баги? И теперь вы зачем-то искорежили свой оператор `+`. В первой версии было все нормально (по крайней мере внешне), а теперь вы из оператора `+` вдруг ни стого ни с сего сделали некое подобие оператора `+=`. Зачем? Почему? 1

@TheCalligrapher Вездепух 12792 / 6669 / 1795 Регистрация: 18.10.2014 Сообщений: 16,877
	06.10.2016, 00:04	7
	Сообщение от Ilya2016 вызывается деструктор для this и удаляет temp Мне трудно представить, о чем идет речь, не видя финальной версии кода, в т.ч. вызывающего кода. Как выглядит код? 0

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

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

Решение