Question

我在尝试重载乘法运算符*以进行矩阵乘法时遇到了很多麻烦。我已经定义了一个矩阵类

#ifndef MMATRIX_H
#define MMATRIX_H
#include <vector>
#include <cmath>

// Class that represents a mathematical matrix
class MMatrix
{
public:
// constructors
MMatrix() : nRows(0), nCols(0) {}
MMatrix(int n, int m, double x = 0) : nRows(n), nCols(m), A(n * m, x)
{}

// set all matrix entries equal to a double
MMatrix &operator=(double x)
{
    for (int i = 0; i < nRows * nCols; i++) 
        A[i] = x;
return *this;
}

// access element, indexed by (row, column) [rvalue]
double operator()(int i, int j) const
{
    return A[j + i * nCols];
}

// access element, indexed by (row, column) [lvalue]
double &operator()(int i, int j)
{
    return A[j + i * nCols];
}


// size of matrix
int Rows() const { return nRows; }
int Cols() const { return nCols; }

// operator overload for matrix * vector. Definition (prototype) of member class
MVector operator*(const MMatrix& A);

private:
unsigned int nRows, nCols;
std::vector<double> A;
};
#endif

这是我尝试的操作员重载

inline MMatrix operator*(const MMatrix& A, const MMatrix& B)
{
MMatrix m(A), c(m.Rows(),m.Cols(),0.0);
for (int i=0; i<m.Rows(); i++)
{
    for (int j=0; j<m.Cols(); j++)
    {
        for (int k=0; k<m.Cols(); k++)
        {
            c(i,j)+=m(i,k)*B(k,j);
        }
    }
}
return c;

}

我确信元素的实际乘法没有任何问题。

我得到的错误来自我的主.cpp文件，我试图将两个矩阵相乘C = A * B;我收到了这个错误，

错误：不匹配＆＃39;运营商=＆＃39; （操作数类型是＆＃39; MMatrix＆＃39;和＆＃39; MVector＆＃39;）

Answer 1

有两种方法可以重载operator*：

MMatrix MMatrix::operator*(MMatrix); //or const& or whatever you like
MMatrix operator*(MMatrix, MMatrix);

这些都是有效的，但是语义略有不同。

为了使您的定义符合您的声明，请将定义更改为：

MMatrix MMatrix::operator*(const MMatrix & A)
{
    //The two matrices to multiple are (*this) and A
    MMatrix c(Rows(),A.Cols(),0.0);
    for (int i=0; i < Rows(); i++)
    {
        for (int j=0; j < A.Cols(); j++)
        {
            for (int k=0; k < Cols(); k++)
            {
                c(i,j) += (*this)(i,k)*A(k,j);
            }
        }
    }
    return c;
}

至于您所看到的错误，在您的班级中，您似乎声明操作符采用矩阵并返回向量。你可能想要返回一个矩阵。该错误告诉您，您无法将MVector分配给MMatrix。

Answer 2

我相信，您需要定义复制构造函数和复制赋值：

MMatrix(const MMatrix& other);
MMatrix& operator=(const MMatrix& other);

移动构造函数和赋值也不会让人心动：

MMatrix(MMatrix&& other);
MMatrix& operator=(MMatrix&& other);

同样适用于你的MVector。

C ++，重载*用于矩阵乘法

2 个答案: