Question

为了说明我的问题，我最小化了我的来源：

#include <iostream>
class vec{
    public:
    float* X;
    int DIM;
    vec(int dimension)
    {
        DIM = dimension;
        X = new float[dimension];
        for (int i=0;i<DIM;i++)
            X[i] = (float) rand()/RAND_MAX;
    }
    ~vec(void){
        delete [] X;
    }

    vec operator-( vec const& rhs )
    {
        vec ans(DIM);
        for (int i=0;i<DIM;i++)
            ans.X[i] = X[i] - rhs.X[i];
        return ans;
    }
};

int main(){
    vec A(5),B(5),C(5);
    C= A-B;
    return 0;
}

当我执行这个程序时，我得到一个堆被销毁的错误。我很确定析构函数是我的问题。在C= A-B;行中，变量ans将被构造函数销毁，无法返回。是对的吗？如果我删除行delete [] X;一切正常。但它不会释放内存。

我完成了我的作业并咨询了一个最着名的搜索引擎，但没有找到任何答案。我该如何解决？

Answer 1

C = A-B调用默认的复制赋值运算符（因为您尚未定义）。因此，两个不同的对象将指向相同的动态分配数组。

您需要阅读Rule of Three（简而言之：如果您定义任何析构函数，复制构造函数或复制赋值运算符，您可能需要全部定义它们。）

但最好是你应该完全避免使用原始数组;使用管理自己内存的容器类型。

动态数组和运算符重载导致内存崩溃

1 个答案: