Question

嗨，请有人帮我弄清楚我做错了什么。编写函数来执行矩阵乘法。我没有显示所有值。我真的很困惑。谢谢（非常新的C ++）

    int MultiplyTwoMatrices(int **MatrixA, int rowA, int ColumnA, int **MatrixB, int rowB,    int columnB);
int **Matrixmultiply;       //I have allocated and freed it's memory
int rowA=4;
int rowB=4;
int columnA=4;
int columnB=4;

int main()
{
for ( x = 0; x < rowA; x++)
{
    for (y = 0; y < columnB; y++)
    {
    Matrixmultiply[x][y] = MultiplyTwoMatrices(MatrixA,rowA,columnA,MatrixB,rowB,columnB);
        cout<<Matrixmultiply[x][y] <<"\t";

    }
    cout<<"\n";
}

int MultiplyTwoMatrices(int **MatrixA, int rowA, int ColumnA, int **MatrixB, int rowB,    int columnB)
{

int **temp = NULL;
int sum = 0;
double **Multiple = NULL;
int i=0, j=0;

temp = (int**)malloc(columnB*(sizeof(int*)));
for (int p=0; p<columnB; p++)
    {
        temp[p] = (int*)malloc(rowB*(sizeof(int))); 

    }

Multiple = (double**)malloc(rowA*(sizeof(double*)));
for (int p=0; p<rowA; p++)
    {
        Multiple[p] = (double*)malloc(columnB*(sizeof(double))); 

    }

for (int p =0; p<columnB; p++)
    {
        for (int q = 0; q<rowB; q++)
        {
            temp[p][q] = MatrixB[q][p];
        }

    }
for (i =0; i<rowA; i++)
    {
        for (j = 0; j<columnB; j++)
        {
            for (int r = 0; r<rowB; r++)
            {
            sum = sum + (MatrixA[i][r]  * temp[j][r]);
            }

            Multiple[i][j] = sum;
            return Multiple[i][j];
            //cout<<Multiple[i][j]<<"\t";
            sum=0;
        }
            //cout<<"\n";

    }


for (int p =0; p<columnB; p++)
    free (temp[p]);
    free(temp);
for (int p =0; p<rowA; p++)
    free (Multiple[p]);
    free(Multiple);
}

Answer 1

好的，首先，您的代码需要大量的清理工作。保持函数自包含是一种好习惯，因此只传递函数中的两个矩阵参数，并在函数本身中进行所有迭代。一个简单的参数是效率，否则你的代码必须进行n * m函数调用，而不是很好。所以：

int MultiplyTwoMatrices(int **MatrixA, int **MatrixB){}

更好的是，你使用的是C ++而不是C，所以要使用对象！将int **封装在矩阵类中，这将使您的生活更轻松，您只需要传递对象引用而不是指针！

我非常强烈建议您阅读我推荐的那本书，因为您将充分利用C ++。

无论如何，问题是你在for循环中返回。当你返回函数停止时，它不会遍历最后的for循环。

矩阵乘法（我不会返回所有值）

1 个答案: