为什么删除此对象会导致问题？

Question

实例化：

weapons.push_back(new Pistol());
weapons.push_back(new Rifle());
weapons.push_back(new Shotgun());

析构函数，当第一次删除发生时，代码中断。当我关闭程序时会发生这种情况。

Brain::~Brain()
{
    for (unsigned int i = 0; i < weapons.size(); i++)
    {
        delete weapons[i]; // this is where the code breaks
    }
}

我收到警告：

Unhandled exception at 0x0096371f in D3D10DEMO.exe: 0xC0000005: Access violation reading location   0x000002ce.

武器是这样的：

weapons(vector<Gun*>())

编辑 - 我已经从这个问题中删除了大部分代码，但我也削减了我的程序，以便在此处以更小的解决方案重现问题：

http://dl.dropbox.com/u/13519335/D3D10DEMO_0.25.MinRep.zip

Answer 1

你还没有为你的武器类定义虚拟析构函数。 http://www.parashift.com/c++-faq-lite/virtual-functions.html#faq-20.7

Answer 2

你的问题是定义

class Brain : public Entity
{
private:
    std::vector<Gun*> weapons;

以及Brain对象对Gun*的所有权。

Brain::~Brain()
{
    for (unsigned int i = 0; i < weapons.size(); i++)
    {
        delete weapons[i];
    }
}

如果一个 Brain 是副本构造的，删除将被多次调用，从不同的武器向量中删除相同的 Gun。当您在主函数中添加代理（代理是 Brain 的派生类）时，会创建一个临时大脑。

int main()
{
    Level* level;
std::vector<Agent> agents;

level = new Level(agents);

for (int i = 0; i < 1; i++)
    {
        //TODO - health is a pointless parameter here
        agents.push_back(Agent(100, *level, agents, level->Pickups(), D3DXCOLOR(1.0F, 0.4f, 0.4f, 1.0f)));
    }

delete level;

}

 如果你为 Brain 实现了一个复制构造函数来克隆 Gun* 向量，你应该没问题。或者你应该在你的向量中使用shared_ptr<Gun>，这样你就不必删除它们了。

总结你的问题归结为

class Foo{};

class Bar
{
public:
    Bar()
    {
        mFooVec.push_back( new Foo() );
        mFooVec.push_back( new Foo() );
    }

    ~Bar()
    {
        for( unsigned int i = 0;i < mFooVec.size(); ++i )
        {
            delete mFooVec[i];
        }
    }

    std::vector<Foo*> mFooVec;
};

int main()
{
    Bar x;
    Bar y = x;

    return 0;
}

这里 Bar x 和 y 在它们的 mFooVec

中都有相同的两个 Foo*