C ++复制赋值运算符类

我认为问题出在我重载运算符的类中=我不知道如何将正确的对象数据复制到左侧的对象数据中，我不确定是否正确删除...

您没有（正确删除）。

运算符=应该更改的值*this。而是删除w.m1和w.m2（其中w是您的参数）。

更正的代码：

Mat &operator=(const Mat &w){
    int *new_ptr1 = NULL;
    new_ptr1 = new int [w.rows*w.cols];
    int *new_ptr2 = NULL;
    new_ptr2 = new int [w.rows*w.cols];
    for(unsigned i = 0; i < w.rows*w.cols; i++){
        new_ptr1[i] = w.say_m1(i);
        new_ptr2[i] = w.say_m2(i);
    }
    delete[] m1; // <<< HERE
    delete[] m2; // <<< HERE
    m1 = new_ptr1;
    m2 = new_ptr2;
    rows = w.rows;
    cols = w.cols;
    return *this;
}

也就是说，您的代码还有很多其他问题：

假设您编写了这段代码来学习如何在一个类中操作动态数组：

1. 您正在使用直接管理两个不同分配资源的类。您应该考虑在m1和m2上使用智能指针（出于安全考虑）。

2. 实施以下三个规则：构造函数（您已经有了这个），复制构造函数（您没有这个）和赋值运算符（您已经有了这个）。

3. 向您的班级添加析构函数。

**假设这是您将要使用的代码，而不是为学习数组而编写的代码：**

1. []将std::vectors用于m1和m2。这将使您不必执行三和析构函数的规则。

2. 您using namespace std;在全球范围内使用。不要这样做，因为它会带来很多问题。

编辑：

更好的代码：

#include <iostream>
#include <vector>

class Mat{
std::vector<int> life;
    std::vector<int> neighborhood;

public:

    // this is unnecessary
    // void write(unsigned r_max, unsigned c_max);  {

    Mat() : life(1), meighborhood(1) // fill each with 1 int with default value (0)
    {
    }

    Mat(unsigned r, unsigned c)
      : life(r), meighborhood(c) // fill each with r(and c) ints with default value (0)
    {
    }

    // ~Mat() became unnecessary: destructors of std::vector will deallocate fine

    Mat &operator=(const Mat &w) {
        // create replacements before doing changing any value
        // this way, if you get an exception while creating the data
        // the value in the obhect does not change
        std::vector<int> new_life(w.life);
        std::vector<int> new_neighborhood(w.neighborhood);

        life.swap(new_life);
        neighborhood.swap(new_neighborhood);
        return *this;
    }

    // use std::vector<int>::at which throws an exception if the index is invalid
    // if you are not interested in the validation of the index
    // return life[i] and neighborhood[i]
    int say_m1(unsigned i) const{ return life.at(i); }
    int say_m2(unsigned i) const{ return neighborhood.at(i); }
};