0

我是 C++ 新手，所以我正在尝试借助 Wikipedia 中的示例来学习它。我玩了一些类，我有一个分段错误错误。

这是我的代码：

class SomeClass {};

class AnotherClass {
    SomeClass* sc
    public:
        AnotherClass(SomeClass* SC):( sc = SC; ){}

        //***********************************************************************
        ~AnotherClass(){ delete sc; } //here I'm getting rid of internal pointer
        //***********************************************************************
};

int main( int argc, char* argv[] ) {
    SomeClass* SC = new SomeClass();
    AnotherClass* AC = new AnotherClass(SC);

    delete AC;

    // *****************************************************
    delete SC; //i think that this line might cause an error
    //******************************************************

    return 0;
}

我想，我应该delete让每个指针都释放堆内存？！你能指出我的错误吗？

编辑：

这是我的真实代码：

#include <iostream>
#include <string>

using namespace std;

class Pizza {
    string dough;
    public:
        Pizza(string d):dough(d) {}
            void setDough( string value ) { dough = value; }
            string getDough() { return dough; }
};

class PizzaBuilder {
    Pizza* pizza;

    public:
        PizzaBuilder( Pizza* p ) { pizza = p; }
        ~PizzaBuilder() { delete pizza; cout << "PizzaBuilder Destructor." << endl;}

        PizzaBuilder* addExtra(string extra) { 
            string special = pizza->getDough() + " and extra " + extra;
            pizza->setDough(special);
            return this;
        }
        Pizza* getPizza() { return pizza; }
    };

int main(int argc, char* argv[]) {

    Pizza* p = new Pizza("My Special DOVE!");
    PizzaBuilder* pb = new PizzaBuilder(p);

    pb->addExtra("Mushrooms")->addExtra("Anchovies")->addExtra("Zefir")->addExtra("Chilli");

    cout << p->getDough() << endl;

    delete pb;
    delete p;

    return 0;
}
4

3 回答 3

9

delete不会像您想象的那样删除指针。它删除指针指向的对象。因此，您将SC两次删除指向的对象：一次来自main指向的对象的析构函数，另一次来自 指向的对象的析构函数，AC删除时自动调用。

多次删除一个对象是未定义的行为。

于 2013-09-03T14:32:46.470 回答
4

您需要决定谁拥有该对象。看起来你想AnotherClass拥有这个SomeClass对象，delete在析构函数中给你它，所以你不应该delete在main(). Double deleteing 会导致未定义的行为。

避免这种情况的最好方法是使用std::shared_ptr，它管理对内存中同一对象的多个引用：

#include <memory>

class SomeClass {};

class AnotherClass {
    std::shared_ptr<SomeClass> sc
    public:
        AnotherClass(std::shared_ptr<SomeClass> SC): sc(SC){}

        //***********************************************************************
        ~AnotherClass(){ /* nothing */}
        //***********************************************************************
};

int main( int argc, char* argv[] ) {
    std::shared_ptr<SomeClass> SC(new SomeClass());
    std::shared_ptr<AnotherClass> AC(new AnotherClass(SC));


    return 0;
}

请注意缺少对 的调用delete。

于 2013-09-03T14:35:58.227 回答
2

首先，尽量不要处理原始指针。尤其是在开始 C++ 时。尝试改用值。

也就是说，当您传递SC给构造函数时，该对象存储指针，因此它有一个指向您在main. 在析构函数中，它删除了该对象。然后main也尝试删除该对象。繁荣。

于 2013-09-03T14:34:09.907 回答