编辑：

Question

我是C ++的新手，所以我试图借助维基百科的例子来学习它。我玩了一些类，我有一个分段错误错误。

这是我的代码：

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;
}

Answer 1

正如您所想，

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

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

Answer 2

您需要决定谁拥有该对象。您似乎希望AnotherClass拥有SomeClass对象，在析构函数中为您提供delete，因此您不应delete在main()中delete }}。 Double 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

请注意缺少对{{1}}的调用。

Answer 3

首先，尽量不要处理原始指针。特别是在开始使用C ++时。请尝试使用值。

也就是说，当您将SC传递给构造函数时，该对象存储指针，因此它具有指向您在main中创建的同一对象的指针。在析构函数中，它删除该对象。然后main也尝试删除该对象。吊杆。

删除传递给构造函数的对象指针，从而导致错误

这是我的代码：

编辑：

这是我的真实代码：

3 个答案: