Question

我认为我下面的Vector类的析构函数应该有if语句来取消它使用的内存。如果arr有一个成员，则会有delete arr。如果arr有很多成员，我必须使用delete[] arr。

你能告诉我这是必要的吗？

我的代码：

class Vector {
    double * arr;
    short dim;

public:
    Vector(short d = 0): dim(d) {
        arr = NULL;
        if (dim < 0) {
            dim = 0;
        } else {
            arr = new double[dim];
        }
    }

    ~Vector() {
        if (arr != NULL) {
            if (dim == 1) {
                delete arr;
            } else {
                delete[] arr;
            }

            arr = NULL;
            dim = 0;
        }
    }
};

Answer 1

不仅没有必要，也只是不正确。使用new []创建的任何内容只能使用delete []删除。否则会产生未定义的行为。

Answer 2

这不仅是必要的，而且这是非法的，因为您只使用new[]分配内存。如果您拨打new，则需要delete，如果您拨打new[]，则需要致电delete[]。混合它们是不确定的行为。你的矢量应该看起来像：

class Vector {
    double * arr;
    short dim;

public:
    Vector(short d = 0): dim(d) {
        if (dim > 0)
            arr = new double[dim]
        else
            arr = nullptr;
    }

    Vector(const Vector& copy) : dim(copy.dim) {
        if (dim > 0) {
            arr = new double[dim]
            // copy data here
        }
        else
            arr = nullptr;
    }

    ~Vector() {
        delete [] arr;
    }

    Vector & operator=(Vector rhs) {
        // swap the contents of the copy.  you can make a swap function to do this
        double * temp = arr;
        arr = rhs.arr;
        rhs.arr = temp;
        dim = rhs.dim;
    }

};

现在我们有正确的副本，删除将是nullptr上的非操作或正确释放从构造函数分配的内存。

Answer 3

您永远不会在任何地方拨打new double;，因此不需要。始终致电delete [] arr;。也不需要检查NULL。

Answer 4

不，这不是必要的。确保在使用new关键字创建已分配的内存时删除没有方括号的内存，避免混用它们。

C ++：我是否需要析构函数的“if”语句？

4 个答案: