Умножение матриц: перемножается только первая строка

@_karnaval_ · Регистрация: 22.12.2018

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

Функция перемножает только первую строку, а остальные нет

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
Matrix operator*(const Matrix& a, const Matrix& b) // ф-ция для перемножения матриц
{
    setlocale(LC_ALL, "rus");
    cout << "Произведение матриц равно:" << endl;
    Matrix result(a.row, a.column);
    for (int i = 0; i < a.row; i++)
    {
        for (int j = 0; j < a.column; j++)
        {
            for (int t = 0; t < b.row; t++) {
                result.matrix[i][j] += a.matrix[i][t] * b.matrix[t][j];
            }
        }
        return result;
    }
}

Вот полный код:

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
//5. Создайте класс Matrix на базе вектора vector<vector<int>>. Определите операторную функцию ostream& operator<<(ostream&, const Matrix&) для вывода матрицы.
//Создайте следующие бинарные операторные функции, по возможности используя объекты функций STL:
//operator+(const Matrix&, const Matrix&) для сложения матриц, operator*(const Matrix&, const vector<int>&)
//для умножения матрицы на вектор, operator*(const Matrix&, const Matrix&) для умножения матриц.
#include <iostream>
#include <cstdlib>
#include <vector>
 
using namespace std;
 
class Matrix
{
private:
    vector<vector<int> > matrix;
    int row;
    int column;
public:
    Matrix() : row(0), column(0) {}
    Matrix(int row, int column) : row(row), column(column)
    {
        matrix.resize(row, vector<int>(column));
    }
    void EnterMatrix() //ввод матрицы
    {
        int element;
        for (int i = 0; i < row; i++)
            for (int j = 0; j < column; j++)
            {
                cin >> element;
                matrix[i][j] = element;
            }
    }
    friend Matrix operator+(const Matrix& a, const Matrix& b); //ф-ция для сложения матриц
    friend ostream& operator<<(ostream& out, const Matrix& m); //ф-ция для вывода матрицы
    friend Matrix operator*(const Matrix& a, const Matrix& b); // ф-ция для перемножения матриц
};
 
Matrix operator*(const Matrix& a, const Matrix& b) // ф-ция для перемножения матриц
{
    setlocale(LC_ALL, "rus");
    cout << "Произведение матриц равно:" << endl;
    Matrix result(a.row, a.column);
    for (int i = 0; i < a.row; i++)
    {
        for (int j = 0; j < a.column; j++)
        {
            for (int t = 0; t < b.row; t++) {
                result.matrix[i][j] += a.matrix[i][t] * b.matrix[t][j];
            }
        }
        return result;
    }
}
 
Matrix operator+(const Matrix& a, const Matrix& b) //ф-ция для сложения матриц
{
    setlocale(LC_ALL, "rus");
    if (a.row != b.row || a.column != b.column || a.row <= 0 || b.row <= 0 || a.column <= 0 || b.column <= 0)
    {
        cout << "Операция не может быть выполнена. Количество строк и столбцов обеих матриц должно совпадать, а также не должно быть равным нулю." << endl;
        system("pause");
        exit(1);
    }
    else
    {
        cout << "Сумма матриц равна:" << endl;
        Matrix result(a.row, a.column);
        for (int i = 0; i < a.row; i++)
        {
            for (int j = 0; j < a.column; j++)
            {
                result.matrix[i][j] = a.matrix[i][j] + b.matrix[i][j];
            }
        }
        return result;
    }
}
 
ostream& operator<<(ostream& out, const Matrix& m) //ф-ция для вывода матрицы
{
    for (int i = 0; i < m.row; i++)
    {
        for (int j = 0; j < m.column; j++)
            out << m.matrix[i][j] << " ";
        out << endl;
    }
    return out;
}
 
int main()
{
    setlocale(LC_ALL, "rus");
    
    cout << "Выберите операцию: \n 1) Сложение матриц \n 2) Умножение матриц \n 3) Умножение матрицы на вектор" << endl;
    int n;
    cin >> n;
 
    switch (n)
    {
    case 1: // ввод для сложения
    {
        int r1, c1, r2, c2;
        cout << "Введите количество строк для первой матрицы: ";
        cin >> r1;
        cout << "Введите количество столбцов для первой матрицы: ";
        cin >> c1;
        cout << "Введите элементы 1 матрицы: ";
 
        Matrix mat(r1, c1);
        mat.EnterMatrix();
        cout << mat;
 
        cout << "Введите количество строк для второй матрицы: ";
        cin >> r2;
        cout << "Введите количество столбцов для второй матрицы: ";
        cin >> c2;
        cout << "Введите элементы 2 матрицы: ";
 
        Matrix mat2(r2, c2);
        mat2.EnterMatrix();
        cout << mat2;
 
        cout << endl;
 
        cout << mat + mat2;
        break;
    }
    case 2: // ввод для умножения
    {
        int n, m, q;
        cout << "Введите количество строк для 1 матрицы: ";
        cin >> m;
        cout << "Введите количество строк для 2 матрицы и количество столбцов для 1 (это одно число): ";
        cin >> n;
 
        cout << "Введите элементы 1 матрицы: ";
 
        Matrix mat(m, n);
        mat.EnterMatrix();
        cout << mat;
 
        cout << "Введите количество столбцов для 2 матрицы: ";
        cin >> q;
        cout << "Введите элементы 2 матрицы: ";
 
        Matrix mat2(n, q);
        mat2.EnterMatrix();
        cout << mat2;
 
        cout << endl;
 
        cout << mat * mat2;
        break;
    }
    }
    
    system("pause");
    return 0;
}

@Pvt · 05.07.2019, 22:15

result.matrix[i][j] += a.matrix[i][t] * b.matrix[t][j];

Байт · 05.07.2019, 22:43

Я бы перед строкой 10 написал

C++
1
result.matrix[i][j] = 0;

@zss · 06.07.2019, 06:53

_karnaval_, почему не читаете предупреждения компилятора?

warning C4715: operator*: значение возвращается не при всех путях выполнения

Плюс к этому

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

Функция перемножает только первую строку, а остальные нет

А теперь догадайтесь с 3 раз, что не так!

@_karnaval_ · 06.07.2019, 10:01 **[ТС]**

Нашел ошибку,

C++
1
return result;

был поставлен не в том месте, надо было его ставить после цикла for i.

Добавлено через 21 минуту
Теперь у меня программа вылетает, при умножении матрицы 2x3 на 3x2

@zss · 06.07.2019, 10:13

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

программа вылетает,

Немного перепутаны индексы

C++
1
2
3
4
5
6
Matrix result(a.row,b.column);  //  !!!!!!
    for (int i = 0; i < a.row; i++)
    {
        for (int j = 0; j < b.column; j++)   //  !!!!!!
        {
            for (int t = 0; t < b.row; t++) {

Ну и не стоит пренебрегать правилом ТРЕХ - у Вас нет копиконструктора, деструктора и operator=
В данном конкретном случае повезло - у класса vector все это есть.

@zayats80888 · 06.07.2019, 10:18

_karnaval_,

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
Matrix operator*(const Matrix& a, const Matrix& b) // ф-ция для перемножения матриц
{
    // необходимо проверить условие a.column == b.row
    if (a.column != b.row)
    {
        // TO DO
    }
    setlocale(LC_ALL, "rus");
    cout << "Произведение матриц равно:" << endl;
    Matrix result(a.row, b.column);
    for (int i = 0; i < a.row; i++)
    {
        for (int j = 0; j < b.column; j++)
        {
            result.matrix[i][j] = 0;
            for (int t = 0; t < a.column; t++) {
                result.matrix[i][j] += a.matrix[i][t] * b.matrix[t][j];
            }
        }
    }
    return result;
}

@_karnaval_ · 06.07.2019, 12:00 **[ТС]**

у меня есть operator<<

C++
1
2
3
4
5
6
7
8
9
10
ostream& operator<<(ostream& out, const Matrix& m) //ф-ция для вывода матрицы
{
    for (int i = 0; i < m.row; i++)
    {
        for (int j = 0; j < m.column; j++)
            out << m.matrix[i][j] << " ";
        out << endl;
    }
    return out;
}

он не подойдет?

@zayats80888 · 06.07.2019, 12:06

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

он не подойдет?

для чего?

@_karnaval_ · 06.07.2019, 12:08 **[ТС]**

для вывода

@zayats80888 · 06.07.2019, 12:10

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

для вывода

а при чем здесь вывод, вопрос про умножение был, я исправил ваши ошибки в операторе умножения и указал, что нужно обработать исключительную ситуацию, хотя бы как в операторе сложения это сделано

Новые блоги и статьи Все статьи Все блоги /
Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .	Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .
SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .

@Pvt Заблокирован
	05.07.2019, 22:15
	result.matrix[i][j] += a.matrix[i][t] * b.matrix[t][j]; 0

@_karnaval_ 1 / 1 / 0 Регистрация: 22.12.2018 Сообщений: 58
	06.07.2019, 12:08 [ТС]
	для вывода 0