类设置器函数的问题。类设置器函数未正确分配值

Question

函数调用下面的语句没有被执行。我很茫然，为什么会这样？有人可以澄清一下。请考虑下面的代码：

#include<iostream>
#include<cmath>
using namespace std;
class Matrix
{
private:
int row,col;
double *values;
public:
    Matrix();
    Matrix(int r, int c, double* x);
void setdim(int m, int n){row=m;col=n;}
int getrowdim() const {return row;}
int getcoldim() const {return col;}
    void set_values(int i, double x);
    double get_value(int i) const;
friend Matrix operator+(const Matrix &A, const Matrix &B);
};
Matrix::Matrix()
{
    this->row = 0;
    this->col = 0;
    this->values = NULL;
}
Matrix::Matrix(int r, int c, double* x)
{
    this->row = r;
    this->col = c;
    this->values = new double[r*c];
    for (int i =0;i<r*c;i++)
    {
        cout<<"Enter value ["<<i<<"] ";
        cin>>this->values[i];
    }
}
void Matrix::set_values(int k, double x)
{
    this->values[k] = x;
}
Matrix operator+(const Matrix &A, const Matrix &B)
{
    int rowa = A.getrowdim();
    int cola = A.getcoldim();
    int rowb = B.getrowdim();
    int colb = B.getcoldim();
   
    if(rowa == rowb && cola == colb)
    {
        Matrix C;
        C.setdim(rowa, colb);
        for(int i =0; i< rowa*cola ; i++)
        {  
        cout<<"i = "<<i<<", A.get_value = "<<A.get_value(i)<<", B.get_value =  "<<B.get_value(i)<<endl;
        double m = A.get_value(i) + B.get_value(i);
        cout<<m<<endl;
        C.set_values(i, m );
        cout<<"Returned from C.set_values()"<<endl;

// THIS STATEMENT DOES NOT GET PRINTED. PLEASE TELL THE REASON // WHY. I SUSPECT THE ERROR IS HERE

        }
        return C;
    }
    else
    {
      cout<<"Invalid Operation";
      return A;
    }
}
double Matrix::get_value(int i) const
{
    return this->values[i];
}

int main()
{
    Matrix A(2,2,NULL);
    Matrix B(2,2,NULL);
    Matrix C;
    C = A+B;
    return 0;  
}

语句 - 从 C.set_values() 返回的语句根本不会被打印出来。

有人可以帮助澄清为什么会这样吗？非常感谢您的帮助！

Answer 1

这里：

    Matrix C;                             // (1)
    C.setdim(rowa, colb);                 // (2)
    for(int i =0; i< rowa*cola ; i++)
    {  
    cout<<"i = "<<i<<", A.get_value = "<<A.get_value(i)<<", B.get_value =  "<<B.get_value(i)<<endl;
    double m = A.get_value(i) + B.get_value(i);
    cout<<m<<endl;
    C.set_values(i, m );                  // (3)

您默认构造一个Matrix (1)。默认构造函数只是将成员values 设置为NULL。然后在（2）中调整成员的大小，但values 仍然是NULL。然后，在 (3) 中调用 set_values 尝试访问数组元素，但没有数组。

问题是您的Matrix 有一个原始拥有指针作为成员。当班级管理资源时，您必须遵循rule of 3/5。如果你不这样做，班级就坏了。管理资源并非易事，遵循 3/5 规则只是最低要求。

您应该争取的更简单的替代方案是 0 规则。对于不管理资源的类，编译器生成的特殊成员恰到好处。如果您将原始指针成员替换为std::vector<double>，那么您无需担心复制、赋值或销毁，因为编译器会为您生成所需的方法。