0

下面是我用于 2 个矩阵相乘的 C++ 代码。第一个输入是维度：矩阵 1 的行和列以及矩阵 2 的行和列。然后您需要输入两个矩阵的所有元素。完成后，应将 2 个矩阵相乘并显示结果矩阵。但是由于一个原因，在我输入了所有元素之后，它就卡住了。你们中有人明白我做错了什么吗？

#include <iostream>
#include "Matrix_functions.hpp"

using namespace std;    

int read_matrix(int** matrix, int rows, int cols)
{
    for(int i = 0; i < rows; i++)
    {
        matrix[i] = new int[cols];
        for(int j = 0; j < cols; j++)
        {
            cin >> matrix[i][j];
        }
    }

    return **matrix;
} 

int print_matrix(int** result, int rows, int cols)
{
    for(int i = 0; i < rows; i++)
    {
        for(int j = 0; j < cols; j++)
        {
            cout << result[i][j];
        }
    }

    cout << endl;

    return **result;
}

int multiply_matrix(int** matrix, int rows, int cols, int** matrix1, int rows1, int    cols1, int** result)
{
    for(int i = 0; i < rows; i++)
    {
        for(int j = 0; j < cols1; j++)
        {
            result[i][j] = 0;

            for(int k = 0; k < cols; k++)
            {
                result[i][j] = result[i][j] + (matrix[i][k] * matrix1[k][j]);
            }
        }
    }

    return **result;
 }

int main ()
{
    //matrices and dimensions
    int rows1, cols1, rows2, cols2;
    int **matrix1, **matrix2, **result = 0;

    cout << "Enter matrix dimensions" << endl;
    cin >> rows1 >> cols1 >> rows2 >> cols2;

    cout << "Enter a matrix" << endl;

    matrix1 = new int*[rows1];

    // Read values from the command line into a matrix
    read_matrix(matrix1, rows1, cols1);

    cout << "Enter a matrix" << endl;

     matrix2 = new int*[rows2];

    read_matrix(matrix2, rows2, cols2);

    // Multiply matrix1 one and matrix2, and put the result in matrix result
    multiply_matrix(matrix1, rows1, cols1, matrix2, rows2, cols2, result);

    print_matrix(result, rows1, cols2);

    //TODO: free memory holding the matrices

    return 0;
}

另外，现在，我的矩阵被打印为一维水平向量 (1 2 3 4) 而不是矩阵 (1 2 和 3 4 下面)。任何人都可以解释我如何解决这个问题？

4

2 回答 2

0

如果要将矩阵打印在不同的行上，请打印换行符。执行此操作的 C++ 方法是使用 std::endl。

for(int i = 0; i < rows; i++)
{
    for(int j = 0; j < cols; j++)
    {
        cout << result[i][j];
    }
    cout << endl; ///< ADD THIS LINE
}
于 2013-09-15T16:14:10.870 回答
0 
multiply_matrix(matrix1, rows1, cols1, matrix2, rows2, cols2, result); 
`new` for result? Forgot?

像这样的东西：

result = new int*[rows1];
for(int i = 0; i < rows1; i++)
{
   result[i] = new int[cols2];
}
multiply_matrix(matrix1, rows1, cols1, matrix2, rows2, cols2, result);

您还没有为result矩阵分配内存。
因此，您是未定义行为的受害者。

关于以矩阵形式打印，您需要打印换行符。

无需从 read 和 multiply 函数中返回元素。

此外，检查两个矩阵相乘的有效性，即cols1 == rows2在相乘之前。

于 2013-09-15T16:19:43.543 回答