运行出错了，已经写了拷贝构造函数了，还是出错

Question

s1=s2,这样写，就出错了，释放age了，不知道咋回事，拷贝构造函数写成这样没问题啊

Answer 1

以下内容由CHATGPT及阿里嘎多学长共同生成、有用望采纳：

---

您想要解决的问题

您提问的目的是解决在C++程序中使用拷贝构造函数时出现的内存释放错误。您已经实现了拷贝构造函数，但是在执行对象赋值操作时，程序出现了断言失败的错误。

图片内容

1. 第一张图片显示了person类的声明和拷贝构造函数的部分实现，但是存在语法错误和逻辑问题。
2. 第二张图片展示了程序中创建两个person对象并进行赋值操作的代码片段，以及程序输出。
3. 第三张图片是一个运行时断言失败的错误消息，指出程序在heap debug中检测到了无效的堆指针。

问题出现原因

问题出现的原因在于拷贝构造函数的实现不正确，导致在对象赋值时，原始对象的age指针被错误地释放，从而造成双重释放（double free）的错误。

问题解决方案

为了解决这个问题，您需要正确实现拷贝构造函数，确保在赋值操作中不会释放原始对象的资源。以下是拷贝构造函数的正确实现方式：

class person {
    int* age;
public:
    person(int a) {
        age = new int(a);
    }
    // 拷贝构造函数
    person(const person& s) {
        age = new int(*s.age); // 使用new分配内存并复制值
    }
    // 赋值操作符也需要实现，以避免自我赋值和资源泄漏
    person& operator=(const person& s) {
        if (this != &s) {
            delete age; // 先释放当前对象的资源
            age = new int(*s.age); // 然后分配新内存并复制值
        }
        return *this;
    }
    ~person() {
        delete age; // 析构函数中释放资源
    }
};

提供代码

以下是完整的可以解决问题的代码示例：

#include <iostream>
using namespace std;

class person {
    int* age;
public:
    person(int a) : age(new int(a)) {} // 使用初始化列表
    person(const person& s) : age(new int(*s.age)) {} // 拷贝构造函数
    ~person() {
        delete age; // 析构函数
    }
    person& operator=(const person& s) {
        if (this != &s) {
            delete age;
            age = new int(*s.age);
        }
        return *this;
    }
    void testo(int a) {
        cout << a << endl;
    }
};

int main() {
    person s1(10);
    person s2(12);
    s1 = s2; // 使用赋值操作符
    s1.testo(*s1.age);
    system("pause");
    return 0;
}

代码运行方式

这段代码可以在任何支持C++的编译器上运行，如GCC或MSVC。您需要将代码复制到文本编辑器中，保存为.cpp文件，然后使用C++编译器编译并运行。

代码预期运行结果

12

推荐相关链接

• C++拷贝构造函数
• C++资源管理
• C++赋值操作符
• C++析构函数