@Гром

Нужно вызывать srand всего один раз за время работы программы, где-нибудь в начале функции main.

P.S.
1. Деструктор должен освобождать выделенную память:
2. Функция подсчета определителя должна, во-первых, называть det или get_det (вовсе не set_det); во-вторых, должна возвращать целое значение (как произведение и разность целых; это же касается и всех внутренних переменных этой функции); в-третьих, вообще его возвращать - через return det; в-четвертых, насколько я понял, у вас он считается как сумма произведений элементов на линиях, параллельных главной диагонали минус то же для побочной - это на самом деле верно только для матриц 2*2 и 3*3, в более общем случае определитель (т.е. минор n-го порядка) - это линейная комбинация миноров n-1 порядка (т.е. определителей матриц размером на 1 меньше) и далее рекурсивно. Обычно он раскладывается по первой строке или столбцу, если только нет более удобного варианта (строки или столбца с большим количеством нулей).

3. Матрица вообще не должна знать, что вы там с ней делали - вместо двух функций типа showMatrix (одна из которых с параметром вообще никак не относящемся к матрице) вам нужна одна:
4. В названиях функций не обязательно везде писать слово Matrix. Вызовы функций типа show, get_size или size, get_element, set_element и т.п. гораздо короче и не менее понятны.

5. Функции, которые не изменяют значения переменных настоятельно рекомендуется помечать как const:
И еще довольно много замечаний по вашему коду, но для начала хотя бы это.

Вернуться к обсуждению:
Наследование, нахождения произведения и сложения матриц C++

0