Question

我正在尝试设置我的函数并执行一些重载操作，以便我可以+， - ，==，*两个矩阵。我在第一次操作过载时遇到了问题：添加。

我的程序有效，直到我尝试添加2个矩阵。

感谢您的帮助。

include<iostream>
using namespace std;

class matrixType
{
private:
    int rows,cols;
    int** matrix;
public:

    matrixType( int r, int c)
    {
        rows=r;
        cols=c;
        matrix = new int*[rows];
        for(int i = 0; i < rows; ++i)
            matrix[i] = new int[cols];
    }

    ~matrixType()
    {
        for(int i = 0; i < rows; ++i) 
        {
            delete [] matrix[i];
        }
        delete [] matrix;
    }

    matrixType operator+( matrixType m2 )
    {
        if( rows==m2.rows && cols==m2.cols)
        {
            matrixType m3(rows, cols);
            for( int i=0; i<rows; i++)
            {
                for( int j=0; j<cols; j++)
                {
                    m3.matrix[i][j]=matrix[i][j]+m2.matrix[i][j];
                }
            }
            return m3;
        }
    }

    matrixType operator-(matrixType m2)
    {
        if( rows==m2.rows && cols==m2.cols)
        {
            matrixType m3(rows, cols);
            for( int i=0; i<rows; i++)
            {
                for( int j=0; j<cols; j++)
                {
                    m3.matrix[i][j]=matrix[i][j]-m2.matrix[i][j];
                }
            }
            return m3;
        }
    }

    friend istream& operator>> (istream& stream, matrixType m)
    {
        for ( int i=0; i<m.rows;i++)
        {
            for( int j=0; j<m.cols;j++)
            {
                cout<<"Matrix"<<"["<<i<<"]"<<"["<<j<<"]"<<"=";
                stream>>m.matrix[i][j];
                cout<<endl;
            }
        }
        return stream;
    }

    friend ostream& operator<<(ostream& out, matrixType m)
    {
        for ( int i=0; i<m.rows;i++)
        {
            for( int j=0; j<m.cols;j++)
            {
                cout<<"Matrix"<<"["<<i<<"]"<<"["<<j<<"]"<<"=";
                out<<m.matrix[i][j];
                cout<<endl;
            }
        }
        return out;
    }
};

Answer 1

完全不同的方法作为替代 - 基于模板：

template <size_t Rows, size_t Columns>
class Matrix
{
    int matrix[Rows][Columns];

public:
    void operator+=(Matrix<Rows, Columns> const& other)
    {
        for(size_t i = 0; i < Rows; ++i)
        {
            for(size_t j = 0; j < Columns; ++j)
            {
                matrix[i][j] += other.matrix[i][j];
            }
        }
    }

    Matrix<Rows, Columns>
    operator+(Matrix<Rows, Columns> const& other) const
    {
        Matrix<Rows, Columns> result(*this);
        result += other;
        return result;
    }

    template<size_t C>
    Matrix<Rows, C> operator*(Matrix<Columns, C> const& other) const
    {
        // just exemplary, actual implementation missing:
        return Matrix<Rows, C>();
    }

    // rest of operators coming here
};

它可能或可能不符合您的需求，但如果确实如此，您可以免费获得三个规则。此外，您可以自动阻止添加或乘以不合适尺寸的矩阵。

另一方面 - 好处总是带来一些成本...... - 你失去了灵活性。想象一下，你想将任意矩阵放入向量中 - 你需要一个基类然后必须使用（智能？）指针，添加或乘以任意矩阵需要强制转换，......

然而，最大的缺点是你需要在编译时知道你的矩阵大小 - 如果你不知道，我们就出去了。

顺便说一句，在原始实现中添加/乘法 - 如果矩阵大小不匹配，则不返回任何内容！你应该返回某种哨兵，e。 G。 <0x0矩阵 - 或可能甚至更好：抛出一些适当的异常。

Answer 2

这听起来像rule of three违规的情况。

您需要实现一个复制构造函数：

matrixType(const matrixType&)

复制赋值运算符：

matrixType operator=(const matrixType&)

对于C++11，最好还要实现移动构造函数和移动赋值运算符。

matrixType(matrixType&&)
matrixType& operator=(matrixType&& other)

重载运算符：Matrix Addition

2 个答案: