Почему деструктор крашится на чистке и-ого элемента?

Question

[Алексей]

Создать класс для работы с матрицами, память притоп вводить динамически

void massiv::sum(massiv obj1, massiv obj2)
{
cout << "Summary of matrixes: ";
for (int i = 0; i < size; i++) {
for (int j = 0; j < size; j++) {
mas[i][j] = obj1.mas[i][j] + obj2.mas[i][j];
}
}
return;
}

void massiv::multiply(massiv obj1, massiv obj2)
{
cout << "Multiplied matrix: ";
for (int i = 0; i < size; i++) {
for (int j = 0; j < size; j++) {
mas[i][j] = 0;
for (int k = 0; k < size; k++)
mas[i][j] += obj1.mas[
Скриншот ошибки:

Answer 1

[jcmvbkbc]

У тебя в классе massiv нет конструктора копирования, а аргументы методам sum и multiply ты передаёшь по значению. Это значит, что при создании копий объектов класса massiv память не выделяется, но при их уничтожении она освобождается. А потом освобождается повторно при удалении оригинальных объектов. Поэтому не работает.
Самый простой фикс -- поменять

void massiv::sum(massiv obj1, massiv obj2)
void massiv::multiply(massiv obj1, massiv obj2)

на

void massiv::sum(const massiv& obj1, const massiv& obj2)
void massiv::multiply(const massiv& obj1, const massiv& obj2)

Comments

[Алексей]

Спасибо)

[Roman]

Алексей,

for (int i = 0; i < size; i++) 
{
  delete mas[i]; // delete[]
}
delete[] mas;