Преждевременно срабатывает деструктор

@proggamer12 · Регистрация: 06.07.2012

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

Здравствуйте!

Не могли бы вы помочь со следующей проблемой:
реализую класс матрицы (здесь часть исходников):
// Matrix.h

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
class Matrix
{
public:
    double **a;  // Массив удобнее хранить через двойной указатель.
    int m, n;    // m - количество столбцов, n - количество строк
 
    Matrix(int m, int n); // Конструктор, принимающий размеры массива
 
    ~Matrix();                              // Деструктор (используются динамические массивы, которые нужно удалять)
 
    void consoleRead();                     // Чтение массива из консоли
        void consoleWrite();
 
                                            // Перегрузка опреторов
    Matrix operator +(Matrix m2);
};

// 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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
Matrix::Matrix(int m, int n)
{
    this->m = m;
    this->n = n;
 
    a = new double *[m];
 
    for (int i = 0; i < m; i++)
        a[i] = new double[n];
}
 
 
//Деструктор (используется динамическое создание переменных, поэтому нужно проводить "Сборку мусора")
Matrix::~Matrix()
{
    for (int i = 0; i < m; i++)
        delete a[i];
    delete a;
}
 
 
void Matrix::consoleWrite()
{
    for (int i = 0; i < m; i++)
    {
        for (int j = 0; j < n; j++)
            cout << a[i][j] << " ";
 
        cout << endl;
    }
}
 
void Matrix::consoleRead()
{
    cout << "Введите элементы матрицы (размеры матрицы " << m << "x" << n << ")" << endl;
 
    for (int i = 0; i < m; i++)
        for (int j = 0; j < n; j++)
            cin >> a[i][j];
}
 
Matrix Matrix::operator +(Matrix m2)
{
    Matrix temp(this->m, this->n); // Создаем временную матрицу, в которой будет храниться результатирующая матрица
 
    for (int i = 0; i < m; i++)
        for (int j = 0; j < n; j++)
            temp.a[i][j] = this->a[i][j] + m2.a[i][j];
 
    return temp;
}

//main.cpp

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
int m, n;
 
    setlocale(LC_ALL, "Russian");   
    cin >> m >> n;
 
    Matrix m1(m, n);
    Matrix m2(m, n);
    Matrix temp(m, n);
 
    m1.consoleRead();
 
    m2.consoleRead();
 
    (m1 + m2).consoleWrite(); // ОШИБКА

Проверил через отладчик - срабатывает деструктор, данные удаляются, consoleWrite не может ничего вывести...

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

Заранее благодарен,
proggamer12

@castaway · 06.09.2015, 15:07

DrOffset, проще будет сказать что данное правило следует применять ко всем бинарным операторам.

@DrOffset · 06.09.2015, 15:16

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

проще будет сказать что данное правило следует применять ко всем бинарным операторам.

Ну, возможно и проще. Только что тогда делать вот с этими?
+= , -= , *= , /= , %= , <<= , >>= , &= , |= , ^=
Они же тоже бинарные. Но ссылку здесь возвращать можно (даже нужно).
А еще есть операторы >>, << для потока. Которые по совместительству операторы ввода\вывода. Там тоже ссылка.

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

@castaway · 06.09.2015, 15:32

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

Они же тоже бинарные. Но ссылку здесь возвращать можно (даже нужно).

Всегда считал (и до сих пор считаю) эти операторы унарными.

@DrOffset · 06.09.2015, 15:43

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

Всегда считал (и до сих пор считаю) эти операторы унарными.

Разрешены обе формы.
Унарная - как член класса и бинарная - как свободная функция.

@castaway · 06.09.2015, 15:46

Не по теме:

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

Разрешены обе формы.
Унарная - как член класса и бинарная - как свободная функция.

С этого и надо было начинать.

@proggamer12 · 06.09.2015, 16:48 **[ТС]**

Внес изменения, как вы писали - все (почти) работает
Теперь еще одна ошибка - после return 0 появляется сообщение: "MatrixClass.Exe вызвал срабатывание точки останова"

Ошибка возникает во время вызова деструктора

@DrOffset · 06.09.2015, 16:50

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

Ошибка возникает во время вызова деструктора

Покажи тот код, который получился после исправлений.

@proggamer12 · 06.09.2015, 16:54 **[ТС]**

main.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
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
#include <iostream>
#include "Matrix.h"
 
using namespace std;
 
int main()
{
    int m, n;
 
    setlocale(LC_ALL, "Russian");
    cout << "Введите через пробел количество строк и столбцов: ";
    cin >> m >> n;
 
    Matrix m1(m, n);
    Matrix m2(m, n);
    Matrix temp(m, n);
 
    cout << "Заполните массив значениями: переход на следующий элемент в строке пробелом, на следующую строку - Enter:" << endl;
    m1.consoleRead();
 
    cout << "Для дальнейшей работы программы, заполните еще один массив размерности " << m << "x" << n << ": ";
    m2.consoleRead();
 
    cout << "В результате считывания, в первый массив были записаны следующие данные:" << endl;
    m1.consoleWrite();
    
    cout << "Во второй массив: " << endl;
    m2.consoleWrite();
 
    cout << "Начинаем проверять функциональность класса: " << endl;
 
    cout << "Создание дубликата матрицы: " << endl;
 
    temp = m1;
    temp.consoleWrite();
    
    cout << "Сумма двух массивов:" << endl;
    (m1 + m2).consoleWrite();
 
    cout << "Разность двух массивов:" << endl;
    (m1 - m2).consoleWrite();
 
    if (m == n)
    {
        cout << "Произведение двух массивов:";
        (m1 * m2).consoleWrite();
    }
    else
        cout << "Массивы нельзя умножить из-за их размеров" << endl;
 
    double num = (rand() % 10000000) / 100000;
    cout << "Умножим массив на некоторое число, например на " << num << endl;
 
    (m1*num).consoleWrite();
 
    cout << "Вычислим определитель первой матрицы: он равен " << m1.determinant() << endl;
    
    cout << "Получим минор (2,2) первой матрицы" << endl;
    (m1.getMinor(1, 1)).consoleWrite();
 
    cout << "Демонстрация функций данного класса закончена, нажмите Enter для продолжения.";
 
    cin.get();
    cin.get();
 
    return 0;
}

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
#pragma once
class Matrix
{
public:
    double **a;  // Массив удобнее хранить через двойной указатель.
    int m, n;    // m - количество столбцов, n - количество строк
 
 
 
    Matrix(unsigned int m, unsigned int n); // Конструктор, принимающий размеры массива
 
    Matrix(Matrix const& copy);             // Конструктор, принимающий другой массив
 
    ~Matrix();                              // Деструктор (используются динамические массивы, которые нужно удалять)
 
    void consoleRead();                     // Чтение массива из консоли
 
    void consoleWrite();                    // Выввод массива на консоль
 
                                            // Перегрузка опреторов
    Matrix& operator =(const Matrix& m2);
 
    Matrix operator +(const Matrix& m2);
 
    Matrix operator -(const Matrix& m2);
 
    Matrix operator * (double num);
 
    Matrix operator * (const Matrix& m2);
 
    // Для реализации вычисления детерминанта, добавим функцию, возвращающую матрицу без i-й строки и j-го столбца (минор). Далее предполагается, что массив является квадратным
    Matrix getMinor(int i, int j);
 
    // Детерминант будет вычислять рекурсивно, поэтому создадим функцию, принимающую матрцу как аргумент
    double determinant(Matrix m);
 
    double determinant();
    
private:
    void swap(Matrix & x);
 
 
};

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
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
178
179
180
181
182
183
184
185
186
187
#include "Matrix.h"
#include <iostream>
using namespace std;
 
 
Matrix::Matrix(unsigned int m, unsigned int n)
{
    this->m = m;
    this->n = n;
 
    a = new double *[m];
 
    for (int i = 0; i < m; i++)
        a[i] = new double[n];
 
    for (int i = 0; i < m; i++)
        for (int j = 0; j < n; j++)
            a[i][j] = 0;
}
 
Matrix::Matrix(Matrix const & x)
    : a(new double *[x.m])
    , m(x.m), n(x.n)
{
    for (int i = 0; i < m; i++)
    {
        a[i] = new double[n];
        for (int j = 0; j < n; j++)
        {
            a[i][j] = x.a[i][j];
        }
    }
}
 
//Деструктор (используется динамическое создание переменных, поэтому нужно проводить "Сборку мусора")
Matrix::~Matrix()
{
    for (int i = 0; i < m; i++)
        delete[] a[i];
    delete[] a;
}
 
void Matrix::consoleRead()
{
    cout << "Введите элементы матрицы (размеры матрицы " << m << "x" << n << ")" << endl;
 
    for (int i = 0; i < m; i++)
        for (int j = 0; j < n; j++)
            cin >> a[i][j];
}
 
void Matrix::consoleWrite()
{
    for (int i = 0; i < m; i++)
    {
        for (int j = 0; j < n; j++)
            cout << a[i][j] << " ";
 
        cout << endl;
    }
}
 
Matrix& Matrix::operator =(const Matrix& m2)
{   
    if (this != &m2)
    {
        (Matrix(m2)).swap(*this);
    }
    return *this;
}
 
Matrix Matrix::operator +(const Matrix &m2)
{
    Matrix temp(this->m, this->n); // Создаем временную матрицу, в которой будет храниться результатирующая матрица
 
    for (int i = 0; i < m; i++)
        for (int j = 0; j < n; j++)
            temp.a[i][j] = this->a[i][j] + m2.a[i][j];
 
    return temp;
}
 
Matrix Matrix::operator -(const Matrix &m2)
{
    Matrix temp(this->m, this->n); // Создаем временную матрицу, в которой будет храниться результатирующая матрица
 
    for (int i = 0; i < m; i++)
        for (int j = 0; j < n; j++)
            temp.a[i][j] = this->a[i][j] - m2.a[i][j];
 
    return temp;
}
 
Matrix Matrix::operator * (double num)
{
    Matrix temp(this->m, this->n); //В данном случае неважно - использовать m1 или m2, т. к. при корректном использовании, размеры матриц равны
 
    for (int i = 0; i < this->m; i++)
        for (int j = 0; j < this->n; j++)
            temp.a[i][j] = this->a[i][j] * num;
 
    return temp;
}
 
Matrix Matrix::operator * (const Matrix &m2)
{
    Matrix temp(this->m, m2.n); //В данном случае неважно - использовать m1 или m2, т. к. при корректном использовании, размеры матриц равны
 
    for (int i = 0; i < this->m; i++)
        for (int j = 0; j < this->n; j++)
        {
            for (int k = 0; j < this->n; j++)
                temp.a[i][j] += this->a[i][k] * m2.a[k][j];
        }
 
    return temp;
 
}
 
double Matrix::determinant(Matrix m)
{
    int k = 1;
    double sum = 0;
 
    if (m.m == 2)
        sum = m.a[0][0] * m.a[1][1] - m.a[1][0] * m.a[0][1];
 
    if (m.m > 2)
    {
        for (int i = 0; i < m.m; i++)
        {
            Matrix temp = m.getMinor(0, 0);
 
            // используем теорему о разложении определителя по строке/столбцу: удаляем первую сверху строку
            sum += m.a[0][0] + k * m.a[i][0] * determinant(temp);
        }
    }
 
    return sum;
}
 
double Matrix::determinant()
{
    return determinant(*this);
}
void Matrix::swap(Matrix & x)
{
    std::swap(x.a, a);
    std::swap(x.m, m);
    std::swap(x.n, n);
}
 
 
// Для реализации вычисления детерминанта, добавим функцию, возвращающую матрицу без i-й строки и j-го столбца (минор). Далее предполагается, что массив является квадратным
Matrix Matrix::getMinor(int i, int j)
{
    unsigned int arr_size = this->m;
    unsigned int minus_i = 0, minus_j = 0;
 
    Matrix temp(arr_size - 1, arr_size - 1);
 
    for (int ii = 0; i < arr_size; i++)
    {
        for (int jj = 0; j < arr_size; j++)
        {
            if (i == ii || j == jj)
                continue;
 
            // Т. к. при заходе за "удаленную" строку индекс сдвигается, нужно вычитать единицу из переменной цикла в результатирующем массиве
            minus_i = 0;
            minus_j = 0;
            // Возможны 3 случая
            // 1. Индекс строки заходит за "удаленную" строку, тогда нужно в результатирующем массиве из индекса вычитать 1
            if (ii > i)
                minus_i = 1;
            // 2. Индекс столбца заходит за "удаленную" строку, тогда нужно в результатрующем массиве из индекса вычитать 1
            if (jj > j)
                minus_j = 1;
            // 3. Индекс и строки и столбца заходит за "удаленную" строку, ничего делать не надо, т. к. эта проблема является объединением пунктов 1, 2
 
            temp.a[ii - minus_i, jj - minus_j] = this->a[ii, jj];
        }
 
    }
 
    return temp;
}

@DrOffset · 06.09.2015, 17:16

Индексация:

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

C++
1
temp.a[ii - minus_i, jj - minus_j] = this->a[ii, jj];

Надо так:

C++
1
temp.a[ii - minus_i][jj - minus_j] = this->a[ii][jj];

Корректность алгоритма при этом я не проверял.

@proggamer12 · 06.09.2015, 17:18 **[ТС]**

Спасибо!
Почему компилятор на это не жаловался?

@DrOffset · 06.09.2015, 17:22

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

Почему компилятор на это не жаловался?

Потому что это корректная операция. Просто она обозначает другое действие.
Вот, почитай.

Новичок · 06.09.2015, 20:09

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

public:
double **a;

Может не очень в тему но мне бросилось это в глаза. В private же такое надо.

Croessmah · 06.09.2015, 20:13

Не по теме:

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

а ты дальновидный

экстрасекс :)

Новые блоги и статьи Все статьи Все блоги /
GitHub Copilot в JetBrains: модель по умолчанию, Fix и контроль MCP and_y87 12.10.2026 10 октября 2026 года GitHub обновил Copilot для JetBrains: добавил настройки стартовой модели, исправления из диагностики и управление автозапуском MCP. Поддержка IDE 2025. 1 прекращена — требуется. . .	GitHub Copilot в JetBrains: модель по умолчанию, Fix и контроль MCP and_y87 12.10.2026 10 октября 2026 года GitHub обновил Copilot для JetBrains: добавил настройки стартовой модели, исправления из диагностики и управление автозапуском MCP. Поддержка IDE 2025. 1 прекращена — требуется. . .	День 5 из 28: Codex подсказывает следующий запрос and_y87 12.10.2026 9 октября 2026 года OpenAI запустила бету Composer predictions — подсказок следующего сообщения в текущем диалоге Codex. Это помощь с формулировкой запроса, а не автоматическое продолжение задачи. . . .	Запустил конкурс "тем и промптов для текстовых квестов созданных почти чисто ИИ" Adler 06.10.2026 Всем привет! За последние три-четыре дня я создал более 16 текстовых квестовых игр используя преимущественно по одному запросу к ИИ на игру. Мне так понравилось смотреть все ветки/ сцены во всех. . .
ИИ не может найти нужный язык в списке Supersumestria 05.10.2026 Я ему даю вот такое изображение и прошу найти и подчеркнуть немецкий язык. Возвращает он вот это: https:/ / i. **********/ vqBWLe2. png Нужную строчку в 3й колонке просто выдумал. . Это. . .	Новая последняя моя музыка в SUNO zorxor 05.10.2026 Здравствуйте, дорогие мои друзья! С большой радостью я хотел бы представить вам свою новую последнею музыку, которую сгенерировала мне по моей просьбе нейросеть SUNO. С уважением, zorxor. Это. . .	Программный домашний кинотеатр russiannick 27.09.2026 Сподобился на программный домашний кинотеатр. В качестве ЯВУ по традиции выбрал js. В помощники взял Яндекс-Алису. Было создано три зала на разные интересы. исторические и ретро сериал Хичкок. . .	Беседа с ИИ о программистах, недопускающих к созданию и правке кода генеративные ИИ и причины этого zorxor 21.09.2026 Раньше я радовался или получал некоторые эмоции, пусть небольшие, но всё же, от самого процесса написания кода, рекомпиляции и запуска, видя постепенное развитие программы и прочее. А теперь лень. . .

@castaway 4986 / 3093 / 456 Регистрация: 10.11.2010 Сообщений: 11,170 Записей в блоге: 10
	06.09.2015, 15:07
	DrOffset, проще будет сказать что данное правило следует применять ко всем бинарным операторам. 0

@proggamer12 17 / 17 / 4 Регистрация: 06.07.2012 Сообщений: 510
	06.09.2015, 16:48 [ТС]
	Внес изменения, как вы писали - все (почти) работает Теперь еще одна ошибка - после return 0 появляется сообщение: "MatrixClass.Exe вызвал срабатывание точки останова" Ошибка возникает во время вызова деструктора 0

@proggamer12 17 / 17 / 4 Регистрация: 06.07.2012 Сообщений: 510
	06.09.2015, 17:18 [ТС]
	Спасибо! Почему компилятор на это не жаловался? 0