重新定义运算符 C++

Question

我试图重新定义 + 运算符，它是对 2 个矩阵求和并返回 C++ 中的求和矩阵。但不知何故，它一直显示为 0。我错过了什么？这是我的代码。 我也写了这段代码。

#include <iostream>
using namespace std;
class matrix{
    public:
        int a[100][100], n;
        matrix(int b[100][100], int n){
            for(int i=0; i<n; i++){
                for(int l=0; l<n; l++){
                    a[i][l]=b[i][l];
                }
            }
            n=n;
        }
        matrix(){}
        void matrix_input(int n){
            for(int i=0; i<n; i++){
                for(int l=0; l<n; l++){
                    cin >> a[i][l];
                }
            }
        }
        void matrix_print(int n){
            for(int i=0; i<n; i++){
                for(int l=0; l<n; l++){
                    cout << a[i][l] << " ";
                }
                cout << endl;
            }
        }


} ;
matrix operator + (matrix x, matrix y){
            int s;
            int n=x.n;
            matrix sum;
            for(int i=0; i<n; i++){
                for(int l=0; l<n; l++){
                    sum.a[i][l]=y.a[i][l]+x.a[i][l];

                }
            }
            return sum; 
}

int main(){
    int n;
    cin >> n;
    matrix o;
    o.matrix_input(n);
    matrix c;
    c.matrix_input(n);
    matrix sum;
    sum=o+c;
    sum.matrix_print(n);
    return 0;
}

Answer 1

在您的矩阵构造函数中，n = n 将参数分配给自身。你的意思可能是this->n = n;。但是，您最好使用成员初始化器列表：

matrix(int b[100][100], int n) : n(n)
{
  for(int i=0; i<n; i++) {
    for(int l=0; l<n; l++) {
        a[i][l]=b[i][l];
    }
  }
}

而且你的operator + 没有设置sum.n。

更新

matrix_input() 也不会修改矩阵的 n 成员。换句话说，您的类不维护自己的不变量。在operator + 中查询x.n 则毫无意义。

您应该修改matrix，使其成员n 始终有效并反映矩阵中的内容。然后，您可以从matrix_print() 中删除参数n。

顺便说一句，你为什么称你的成员函数为matrix_something？它们已经在matrix 的范围内。只需 input 和 print 就可以了。

Answer 2

您的主要错误在n=n 行中，这是一个自赋值。

你可以猜到，当 n 是一个整数时，这几乎是一个空操作（并且应该是大部分时间）。你的意思是this->n = n;

（更好的方法是不要与命名冲突，但那个小修复会起作用。您还需要修复 operator+ 以分配 sum.n = n;

本地n 参数优先于类成员，因此您需要this->n 来限定类成员。

顺便说一句，您的矩阵非常大（10,000 个整数），因此复制起来很昂贵。因此，如果您的 operator+ 重载将其参数作为 const 引用，则效率会更高。

大概n 是一个逻辑大小，因为物理上你的矩阵总是100 * 100。你的默认构造函数至少应该将n 初始化为0，否则试图读取这个矩阵的后续行为将是未定义的。在读取可能超出矩阵范围的内容时，您应该进行断言检查。 （您的 operator+ 可以检查两者是否具有相同的维度）。

如果您想使用“动态”大小调整，因此您只使用您将实际用于矩阵的内存资源，那么您的实现方式会有所不同。我想这对你的水平来说是一个更高级的练习。

顺便说一句，它“一直打印 0”是随机的，因为您应该得到的行为实际上是未定义的，因为 x.n（以及另外的 sum.n）是您正在读取的未初始化变量。