Деструктор для массива матриц...

@Vladd · Регистрация: 25.10.2009

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

Помогите написать деструктор для массива матриц.. Деструктор для матрицы вроде бы вот так пишется:

C++
1
2
3
4
5
{
    for (int z = 0; z < _rows; ++z)
        delete[] _matrix[z];
    delete[] _matrix;
}

А вот для массива матриц я не знаю

Была вот такая идея:

C++
1
2
3
4
{
    for (int i = 0; i < _masDimension; ++i)
    _arrayOfMatrix[i].~Matrix();
}

Но компилятор ужасно злится, а потом программа вылетает..

@niXman · 25.10.2009, 17:58

Vladd, Для начала, покажи кто такой

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

Matrix

@M128K145 · 25.10.2009, 18:38

попробуй

так

C++
1
2
3
4
5
6
7
for (int i = 0; i < _masDimension; ++i)
{
    for (int z = 0; z < _rows; ++z)
        delete[] _matrix[z][i];
    delete[] _matrix[i];
}
delete[] _arrayOfMatrix;

@Vladd · 26.10.2009, 19:42 **[ТС]**

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

попробуй

так

C++
1
2
3
4
5
6
7
for (int i = 0; i < _masDimension; ++i)
{
    for (int z = 0; z < _rows; ++z)
        delete[] _matrix[z][i];
    delete[] _matrix[i];
}
delete[] _arrayOfMatrix;

Попробовал, пишет что _matrix необъявленный идентификатор... Я подумал, что может у меня создание неправильно реализовано.. В общем вот код:

Matrix.h

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
class Matrix
{
private:
    int** _matrix;
    int _rows, _cols;
 
    void Create();
    void Filling();
 
public:
    Matrix();
    Matrix(int i);
    Matrix(int i, int j);
    Matrix(const Matrix&);
 
    ~Matrix();
};

Matrix.cpp

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
void Matrix::Create()
{
    _matrix = new int*[_rows];
    for (int z = 0; z < _rows; z++)
    _matrix[z] = new int[_cols];
}
 
void Matrix::Filling()
{
    srand(time(NULL));
    for (int i = 0; i < _rows; ++i)
    for (int j = 0; j < _cols; ++j)
        _matrix[i][j] = rand()%10;
}
 
Matrix::Matrix():_rows(5), _cols(5) { Create(), Filling(); }
 
Matrix::Matrix(int i): _rows(i), _cols(i) { Create(); Filling(); }
 
Matrix::Matrix(int rows, int cols): _rows(rows), _cols(cols) { Create(); Filling(); }
 
Matrix::Matrix(const Matrix& m)         
{
    _rows = m._rows;
    _cols = m._cols;
    Create();
    for (int i = 0; i < _rows; ++i)
        for (int j = 0; j < _cols; ++j)
            _matrix[i][j] = m._matrix[i][j];
}
 
Matrix::~Matrix()
{
    for (int z = 0; z < _rows; z++)
        delete[] _matrix[z];
    delete[] _matrix;
}

ArrayOfMatrix.h

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
class ArrayOfMatrix
{
    private:
    Matrix* _arrayOfMatrix;
    int _masDimension;
    int _rows, _cols;
    void Create ();
    Matrix Creator();
        
    public:
    ArrayOfMatrix(int, int, int);           // 1ое число: размер массива, 2ое и 3ье: размеры матрицы
 
    ~ArrayOfMatrix();
}

ArrayOfMatrix.cpp

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
void ArrayOfMatrix::Create()
{
    _arrayOfMatrix = new Matrix[_masDimension];
    for (int i = 0; i < _masDimension; ++i)
    _arrayOfMatrix[i] = Creator();
}
 
Matrix ArrayOfMatrix::Creator()
{
    Matrix obj;
    return obj;
}
 
ArrayOfMatrix::ArrayOfMatrix(int masDimension, int rows, int cols) 
{   
    _rows = rows;
    _cols = cols;
    _masDimension = masDimension;
    Create();
}
 
ArrayOfMatrix::~ArrayOfMatrix()
{
    for (int i = 0; i < _masDimension; ++i)
    {
        for (int z = 0; z < _rows; ++z)
        delete[] _matrix[z][i];
        delete[] _matrix[i];
    }
    delete[] _arrayOfMatrix;
}

Не спрашивайте почему я так модно создаю массив матриц

Зато сам придумал

@Vladd · 27.10.2009, 16:57 **[ТС]**

Скажите пож-та на что вообще указывает указатель массива матриц? На элемент [0][0]?

Evg · 27.10.2009, 19:32

Ели ты про поле _arrayOfMatrix, то на _arrayOfMatrix[0], которое ты выделил через new

@M128K145 · 27.10.2009, 23:12

C++
1
2
3
4
5
6
7
ArrayOfMatrix::~ArrayOfMatrix()
{
    int i;
    for(i = 0; i < _masDimension; ++i)
        _arrayOfMatrix[i].~Matrix();
    delete[] _arrayOfMatrix;
}

Evg · 27.10.2009, 23:21

Конструктор и деструктор - это НЕ функции, а потому вызывать их нельзя

@Даемоник · 27.10.2009, 23:33

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

C++
1
2
3
4
5
6
7
8
9
ArrayOfMatrix::~ArrayOfMatrix()
{
 for (int i = 0; i < _masDimension; ++i)
 {
 for (int z = 0; z < _rows; ++z)
 delete[] _matrix[z][i];
 delete[] _matrix[i];
 }
 delete[] _arrayOfMatrix;

По-моему в этом куске надо поменять местами i и z(в строке delete[] _matrix[z][i]), потому что получается ты удаляешь из каждой строки элемент соответствующий значению i, а не z.

@M128K145 · 27.10.2009, 23:57

Evg, я тоже слышал такое утверждение, но все же объясни тогда пожалуйста такое явление

код

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
#include <iostream>
class test
{
    int a;
    public:
    test();
    ~test();
};
test::test()
{
    a = 5;
    std::cout<<"создан\na = "<<a<<std::endl;
}
test::~test()
{
    a = NULL;
    std::cout<<"удален\na = "<<a<<std::endl;
}
int main()
{
    setlocale(LC_ALL, "Russian");
 
    test* ts1 = new test();
    delete ts1;
 
    test* ts2 = new test();
    ts2->~test();
 
    system("pause");
    return 0;
}

результат

Evg · 28.10.2009, 00:10

Я фигею... на gcc работает, на борланде работает.... Более того, на деструктор разрешается брать адрес (на конструктор - нет). Интересная штука, надо этот вопрос поизучать

@Vladd · 28.10.2009, 06:47 **[ТС]**

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

C++
1
2
3
4
5
6
7
ArrayOfMatrix::~ArrayOfMatrix()
{
    int i;
    for(i = 0; i < _masDimension; ++i)
        _arrayOfMatrix[i].~Matrix();
    delete[] _arrayOfMatrix;
}

Выдаёт вот это и программа вылетает

Необработанное исключение типа "System.AccessViolationException" произошло в ArrayOfMatrix.exe
Дополнительные сведения: Попытка чтения или записи в защищенную память. Это часто свидетельствует о том, что другая память повреждена.

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

Сообщение от Даемоник

По-моему в этом куске надо поменять местами i и z(в строке delete[] _matrix[z][i]), потому что получается ты удаляешь из каждой строки элемент соответствующий значению i, а не z.

Пишет: error C2065: _matrix: необъявленный идентификатор

@M128K145 · 28.10.2009, 07:40

Vladd, скинь код мейна

@Vladd · 28.10.2009, 08:39 **[ТС]**

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

Vladd, скинь код мейна

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
#include <iostream>
#include <conio.h>
#include <time.h>
#include <stdlib.h>
#include "ArrayOfMatrix.h"
#include "Matrix.h"
 
using namespace std;
 
void main()
{
    ArrayOfMatrix m1(3,5,5);
    cout << m1[1] << endl;
    getch();
}

Без деструктора всё нормально работает, но мне надо учителю доказать что здесь будет композиция, т.е. времена жизни объектов Матрикс и АррОфМатрикс одинаковы.. Вот я и парюсь с деструктором... Оператор << перегружен и выдаёт всё нормально..

C++
1
2
3
4
5
6
7
8
9
10
 std::ostream& operator<<(std::ostream& out, const Matrix& object )
 {
    for( int i=0; i<object._rows; ++i )
    {
        for( int j=0; j<object._cols; ++j )
            out << object._matrix[i][j] << " ";
        out << std::endl;
    }
    return out;
 }

Мне кажется что я коряво как-то создаю массив матриц...

C++
1
2
3
4
5
6
7
8
9
10
11
12
void ArrayOfMatrix::Create()
{
    _arrayOfMatrix = new Matrix[_masDimension];
    for (int i = 0; i < _masDimension; ++i)
        _arrayOfMatrix[i] = Creator();
}
 
Matrix ArrayOfMatrix::Creator()
{
    Matrix obj;
    return obj;
}

Может есть другой метод?

Evg · 28.10.2009, 16:22

С деструктораи ситуация следующая. В стандарте действительно их можно вызывать. При этом в момент удаления объекта он всё равно будет вызван

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <iostream.h>
 
class T
{
  public:
    T() { cout << "constr\n"; }
    ~T() { cout << "destr\n"; }
};
 
int main (void)
{
  T t;
  t.~T();
}

Code
1
2
3
constr
destr
destr

В стандарте описывается единственный случай, когда это нужно делать - это если мы вызывали new с указанием места в памяти, где выделать объект. Ибо в этом случае получается, что нельзя вызывать delete на объект, поэтому оставили способ, чтобы вызвать деструктор. Во всех остальных случаях поведение неопределено. Поэтому в данном случае так вызывать деструктор нельзя

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
#include <iostream.h>
 
class T
{
    int x;
  public:
    T() { cout << "constr\n"; }
    ~T() { cout << "destr\n"; }
};
 
int main (void)
{
  char *buf = new char[sizeof(T)];
  T *t = new(buf) T;
  
  t->~T();
}

@Vladd · 28.10.2009, 19:33 **[ТС]**

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

С деструктораи ситуация следующая. В стандарте действительно их можно вызывать. При этом в момент удаления объекта он всё равно будет вызван

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <iostream.h>
 
class T
{
  public:
    T() { cout << "constr\n"; }
    ~T() { cout << "destr\n"; }
};
 
int main (void)
{
  T t;
  t.~T();
}

Code
1
2
3
constr
destr
destr

В стандарте описывается единственный случай, когда это нужно делать - это если мы вызывали new с указанием места в памяти, где выделать объект. Ибо в этом случае получается, что нельзя вызывать delete на объект, поэтому оставили способ, чтобы вызвать деструктор. Во всех остальных случаях поведение неопределено. Поэтому в данном случае так вызывать деструктор нельзя

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
#include <iostream.h>
 
class T
{
    int x;
  public:
    T() { cout << "constr\n"; }
    ~T() { cout << "destr\n"; }
};
 
int main (void)
{
  char *buf = new char[sizeof(T)];
  T *t = new(buf) T;
  
  t->~T();
}

Эм.. А мне то что делать?

У меня есть new... Как деструктор правильно написать?

Evg · 28.10.2009, 20:44

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

@Vladd · 29.10.2009, 05:15 **[ТС]**

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

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

В программе меня волнует только деструктор, у меня его пока что нету... Вся программа это Matrix.h, Matrix.cpp, ArrayOfMatrix.h, ArrayOfMatrix.cpp, Main.cpp. 1ые 4 файла я написал в одном сообщении, а мэйн пойзже... Просто не вижу смысла снова всё вставлять. Зачем засорять тему?

Вот самые популярные предложения по поводу того как организовать деструктор:
Вот так

C++
1
2
3
4
5
6
7
8
9
10
ArrayOfMatrix::~ArrayOfMatrix()
{
    for (int i = 0; i < _masDimension; ++i)
    {
        for (int z = 0; z < _rows; ++z)
            delete[] _matrix[i][z];
        delete[] _matrix[i];
    }
    delete[] _arrayOfMatrix;
}

Но тогда выдаёт error C2065: _matrix: необъявленный идентификатор.

Либо вот так

C++
1
2
3
4
5
6
7
ArrayOfMatrix::~ArrayOfMatrix()
{
    int i;
    for(i = 0; i < _masDimension; ++i)
            _arrayOfMatrix[i].~Matrix();
    delete[] _arrayOfMatrix;
}

А в этом случае программа вылетает

"Необработанное исключение типа "System.AccessViolationException" произошло в ArrayOfMatrix.exe
Дополнительные сведения: Попытка чтения или записи в защищенную память. Это часто свидетельствует о том, что другая память повреждена."

Evg · 29.10.2009, 10:16

Всё, увидел исходник, из-за CUT'ов что-то проморгал. Деструктор пишется симметрично с конструктором (в том плане, что delete пишется симметрично с new)

C++
1
2
3
4
5
6
void ArrayOfMatrix::Create()
{
    _arrayOfMatrix = new Matrix[_masDimension]; // <--- вот new
    for (int i = 0; i < _masDimension; ++i)
        _arrayOfMatrix[i] = Creator();  // <--- здесь никаких new нет, т.к. Creator НЕ создаёт динамических объектов
}

Правда мне не очень понятно, для чего было сделана процедура Creator. В момент выделения массива Matrix'ов для каждого элемента и так вызывается конструктор

Деструктор получается вроде бы вот такой

C++
1
2
3
4
void ArrayOfMatrix::~ArrayOfMatrix()
{
    delete[] _arrayOfMatrix;
}

Для порядку во всех конструкторах и деструкторах влепи печать, чтобы понимать сколько и чего у тебя создаётся/инициализируется

@Vladd · 29.10.2009, 20:45 **[ТС]**

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

Правда мне не очень понятно, для чего было сделана процедура Creator. В момент выделения массива Matrix'ов для каждого элемента и так вызывается конструктор

Я думал, что у меня массивы не создаются... Т.е. можно вообще оставить вот эту строчку:
_arrayOfMatrix = new Matrix[_masDimension];?
Да, кстати тут же вызывается конструктор по умлочанию, поэтому мне пришлось его сделать таким Matrix::Matrix() _rows(5), _cols(5) { Create() } А как можно избежать этой проблемы? Т.е. можно ли в строчке _arrayOfMatrix = new Matrix[_masDimension]; сделать вызов конструктора с какими-либо параметрами? Ведь по идее конструктор по умолчанию должен иметь матрицу без эл-ов...

И ещё меня волнует серьёзный вопрос.. Почему на каждую матрицу выделяется всего 12 байт??

C++
1
2
3
4
5
6
void main()
{
    ArrayOfMatrix m1(3,5,5);
    cout << &m1[0] << " " << &m1[1] << endl;
    getch();
}

Выдаёт: 003В7DC4 и 003В7DD0.. А где выделяется память на матрицу размером 5х5?? Должно же быть 25 эл-ов типа инт.. Инт = 4 байта, то как минимум надо 60 байт.. Или я ошибаюсь?

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

Деструктор получается вроде бы вот такой:

C++
1
2
3
4
void ArrayOfMatrix::~ArrayOfMatrix()
{
    delete[] _arrayOfMatrix;
}

Работает! Чёрт побери! Но как же всякие сложные циклы? Почему всё намного проще?

Новые блоги и статьи Все статьи Все блоги /
Вывод данных через динамический список в справочнике Maks 01.04.2026 Реализация из решения ниже выполнена на примере нетипового справочника "Спецтехника" разработанного в конфигурации КА2. Задача: вывести данные из ТЧ нетипового документа. . .	Функция заполнения текстового поля в реквизите формы документа Maks 01.04.2026 Алгоритм из решения ниже реализован на нетиповом документе "ВыдачаОборудованияНаСпецтехнику" разработанного в конфигурации КА2, в дополнении к предыдущему решению. На форме документа создается. . .	К слову об оптимизации kumehtar 01.04.2026 Вспоминаю начало 2000-х, университет, когда я писал на Delphi. Тогда среди программистов на форумах активно обсуждали аккуратную работу с памятью: нужно было следить за переменными, вовремя. . .	Идея фильтра интернета (сервер = слой+фильтр). Hrethgir 31.03.2026 Суть идеи заключается в том, чтобы запустить свой сервер, о чём я если честно мечтал давно и давно приобрёл книгу как это сделать. Но не было причин его запускать. Очумелые учёные напечатали на. . .
Модель здравосоХранения 6. ESG-повестка и устойчивое развитие; углублённый анализ кадрового бренда anaschu 31.03.2026 В прикрепленном документе раздумья о том, как можно поменять модель в будущем	10 пpимет, которые всегда сбываются Maks 31.03.2026 1. Чтобы, наконец, пришла маршрутка, надо закурить. Если сигарета последняя, маршрутка придет еще до второй затяжки даже вопреки расписанию. 2. Нaдоели зима и снег? Не надо переезжать. Достаточно. . .	Перемещение выделенных строк ТЧ из одного документа в другой Maks 31.03.2026 Реализация из решения ниже выполнена на примере нетипового документа "ВыдачаОборудованияНаСпецтехнику" с единственной табличной частью "ОборудованиеИКомплектующие" разработанного в конфигурации КА2. . . .	Functional First Web Framework Suave DevAlt 30.03.2026 Sauve. IO Апнулись до NET10. Из зависимостей один пакет, работает одинаково хорошо как в режиме проекта так и в интерактивном режиме. из сложностей - чисто функциональный подход. Решил. . .

@Vladd 0 / 0 / 0 Регистрация: 25.10.2009 Сообщений: 11
	27.10.2009, 16:57 [ТС]
	Скажите пож-та на что вообще указывает указатель массива матриц? На элемент [0][0]? 0

Evg 21281 / 8305 / 637 Регистрация: 30.03.2009 Сообщений: 22,660 Записей в блоге: 30
	27.10.2009, 19:32
	Ели ты про поле _arrayOfMatrix, то на _arrayOfMatrix[0], которое ты выделил через new 0

Evg 21281 / 8305 / 637 Регистрация: 30.03.2009 Сообщений: 22,660 Записей в блоге: 30
	27.10.2009, 23:21
	Конструктор и деструктор - это НЕ функции, а потому вызывать их нельзя 0

Evg 21281 / 8305 / 637 Регистрация: 30.03.2009 Сообщений: 22,660 Записей в блоге: 30
	28.10.2009, 00:10
	Я фигею... на gcc работает, на борланде работает.... Более того, на деструктор разрешается брать адрес (на конструктор - нет). Интересная штука, надо этот вопрос поизучать 0

@M128K145 8384 / 3617 / 419 Регистрация: 03.07.2009 Сообщений: 10,709
	28.10.2009, 07:40
	Vladd, скинь код мейна 0

Evg 21281 / 8305 / 637 Регистрация: 30.03.2009 Сообщений: 22,660 Записей в блоге: 30
	28.10.2009, 20:44
	Ты хоть покажи всю программу целиком, а не куски-огрызки. И ткни пальцем в место, где возникает вопрос. Из прошлых объяснений я, например, не понял ничего 0