Question

我正在尝试实现一个为堆分配内存的容器，但好像我的基础构造函数和我的参数构造函数不喜欢彼此。下面，我发布的代码没有任何注释。就目前而言，它崩溃了。

#include <iostream>
using namespace std;

class foo
{
public:
    foo() {size=1; vals = new double[1]; vals[0]=0;}
    ~foo() {delete[] vals;}

    foo(const foo& other)
    {
        size=other.getsize();
        delete[] vals;
        vals = new double[size];
        for(long unsigned i=0; i<size; i++)
            vals[i]=other[i];
    }

    foo& operator=(const foo& other)
    {
        size=other.getsize();
        delete[] vals;
        vals = new double[size];
        for(long unsigned i=0; i<size; i++)
            vals[i]=other[i];
        return *this;
    }

    foo(double* invals, long unsigned insize)
    {
        size=insize;
        delete[] vals;
        vals = new double[size];
        for(long unsigned i=0; i<size; i++)
            vals[i]=invals[i];
    }

    double operator[](long unsigned i) const {return vals[i];}

    long unsigned getsize() const {return size;}
private:
    double* vals;
    long unsigned size;
};


int main()
{
    double bar[3] = {5,2,8};
    foo B(bar, 3);

    cout<< B[0]<< " "<< B[1]<< " "<< B[2]<<endl;    //couts fine

    foo A;    //crashes here

    return 0;
}

但是，当我将main更改为：

int main()
{
    double bar[3] = {5,2,8};
    foo B(bar, 3);

    cout<< B[0]<< " "<< B[1]<< " "<< B[2]<<endl;    //couts fine

    foo A();    //works now

    return 0;
}

运行正常。但是我不能指定A = B，因为它认为foo是一个函数或其他东西。

Answer 1

我认为你有一些非常令人信服的理由不在这里使用std::vector<double> ......

但无论如何......在你的拷贝构造函数中，你不想delete[] vals。

foo(const foo& other)
{
    size=other.getsize();
    vals = new double[size];
    for(long unsigned i=0; i<size; i++)
        vals[i]=other[i];
}

调用复制构造函数时，您的对象尚未初始化，因此vals*甚至没有指向任何有效的对象。因此，删除它会调用undefined behavior（并且您的程序崩溃了。）您只需要在赋值运算符中delete[] vals。

此外，当您声明Foo变量A时，您不希望在变量名后面加上这些括号。只是说：

foo A;

当您将这些括号放在变量名后面时，您实际上是使用从C继承的语法编写函数声明，A成为函数指针类型。

在Copy Constructor和Assignment运算符中删除私有数组

1 个答案: