Нужна помощь. Программа сложения динамических матриц на с++.

[p51x]

Ошибка все таже - нет оператора присваивания и конструктора копирования. Вы два раза освобождаете одну и туже память.

[TSwallow]

Вроде правильно всё сделал.

#include <iostream>
#include <ctime>
#include <cassert>

using namespace std;

class matrix {
	friend ostream& operator << (std::ostream&, const matrix&); // используется для вывода в main
public:
	matrix(int row, int col);								
	~matrix();
	void fill();											
	matrix(const matrix& m);
	void show(matrix m);
	matrix operator + (const matrix& m);
	matrix operator * (const matrix& m);
	matrix operator = (const matrix& m);
protected:
	int r, c;
	int** mat;
};


matrix::matrix(int row, int col) : r(row), c(col) // конструктор
{
	srand(time(0));//используется для установки начала последовательности
	mat = new int*[r];// динамическое выделение памяти под объект типа int
	assert(mat); // assert используется для проверки при исполнении программы
	for (int i = 0; i < r; i++)
	{
		mat[i] = new int[c];
		assert(mat[i]); // assert используется для проверки при исполнении программы
	}
}

void matrix::fill() //Создание рандомной матрици. Использовал для того, чтобы не вводить матрици в ручную;
{
	for (int i = 0; i < this->r; i++)
	{
		for (int j = 0; j < this->c; j++)
		{
			this->mat[i][j] = rand() % 15 + 1;//Создаёт матрица из рандомных чисел
		}
	}
}

matrix::~matrix() //Деструктор
{
	for (int i = 0; i < r; i++)
	{
		delete[] mat[i];
	}
	delete[] mat;
}

ostream & operator << (ostream & out, const matrix & m) { // Оператор << используется для вывода матриц в main
	for (int i = 0; i < m.r; i++)
	{
		for (int j = 0; j < m.c; j++) {
			out << m.mat[i][j] << "\t";
		}
		out << endl;
	}
	return out;
}


matrix matrix::operator + (const matrix& m) // оператор бинарного сложения 
{
	matrix summ(m.r, m.c);// Удерживает значение
	for (int i = 0; i < m.r; i++)
	{
		for (int j = 0; j < m.c; j++)
		{
			summ.mat[i][j] = m.mat[i][j] + mat[i][j];
		}
	}
	return summ;
}


matrix matrix::operator * (const matrix& m) // Оператор умножения
{
	matrix pr(m.r, m.c);// Удерживает значение
	for (int i = 0; i < m.r; i++)
	{
		for (int j = 0; j < m.c; j++)
		{
			pr.mat[i][j] = { };//Зануляю матрицу для вывода
			for (int n = 0; n < m.c; n++)
			{
				pr.mat[i][j] += m.mat[i][n] * mat[n][j];
			}
		}
	}
	return pr;
}

matrix matrix::operator = (const matrix& m)// Оператор присваивания
{
	c = m.c;
	r = m.r;
	mat = new int*[r]; 
	assert(mat);
	for (int i = 0; i < r; i++)
	{
		mat[i] = new int[c];
		assert(mat[i]);
	}

	for (int i = 0; i < r; i++)
	{
		for (int j = 0; j < c; j++)
		{
			mat[i][j] = m.mat[i][j];//присваивание
		}
	}
	return *this;
}

matrix::matrix(const matrix& m)// Конструктор копирования
{
	c = m.c;
	r = m.r;
	
	mat = new int*[r];
	assert(mat);
	for (int i = 0; i < r; i++)
	{
		mat[i] = new int[c];
		assert(mat[i]);
	}

	for (int i = 0; i < r; i++)
	{
		for (int j = 0; j < c; j++)
		{
			mat[i][j] = m.mat[i][j];
		}
	}

}

void matrix::show(matrix m) // show используется для вывода ответов в main
{
	for (int i = 0; i < m.r; i++)
	{
		for (int j = 0; j < m.c; j++) {
			cout << m.mat[i][j] << "\t";//вывод mat с выравниванием табуляцией 
		}
		cout << endl;
	}
}

int main()
{
	setlocale(LC_ALL, "Russian");
	int row, col;
	cout << "Введите кол-во строк: " << endl;
	cin >> row;
	cout << "Введите кол-во колонок: " << endl;
	cin >> col;
	cout << endl;
	
	cout << "Матрица A: " << endl;
	matrix m(row, col);
	m.fill();
	cout << m << endl;

	cout << "Введите кол-во строк: " << endl;
	cin >> row;
	cout << "Введите кол-во колонок: " << endl;
	cin >> col;
	cout << endl;

	cout << "Матрица B: " << endl;
	matrix m2(row, col);
	m2.fill();
	cout << m2 << endl;
	
	cout << "Сумма" << endl;//Cложение
	matrix m3(row, col);
	m3 = m + m2;
	m3.show(m3);

	cout << "Произведение" << endl;//Умножение
	matrix h3(row, col);
	h3 = m * m2;
	h3.show(h3);
}

[p51x]

Ассерты бесполезны. Обычный new кидает исключения. Либо используйте nothrow вариант, либо ловите исключения.

[TSwallow]

Окей