调试断言失败 - 矢量迭代器不可解除引用

时间:2012-07-09 16:30:16

标签: c++ vector

运行此代码时出现此运行时错误:

void AlienShipManager::Update(float timeDelta, 
        BulletManager* bulletManager,
        ParticleManager* particleManager,
        GameStringSystem* stringBatch)
{
    unsigned int i = 0;
    while (i < m_alienShipList.size())
    {
        AlienResult result = m_alienShipList[i].Update(timeDelta, bulletManager);
        switch (result)
        {
            case AlienResult::Dead:
                break;
            default:
                break;
        }
            ++i 
    }
}

在行

AlienResult result = m_alienShipList[i].Update(timeDelta, bulletManager);

我有如何将AlienShip添加到矢量类:

m_alienShipList.push_back(AlienShip(position, speed, m_screeSize, m_alienShipTexture));
如果我有机会这样做,

错误也会出现:

AlienShip* newAlien = new AlienShip(position, speed, m_screeSize, m_alienShipTexture);
    m_alienShipList.push_back(*newAlien);
    delete newAlien;

但如果我将其更改为:

,则不会显示
AlienShip* newAlien = new AlienShip(position, speed, m_screeSize, m_alienShipTexture);
    m_alienShipList.push_back(*newAlien);
因而导致巨大的内存泄漏。

这就是我的AlienShip类的看法:

#pragma once

#include "Body.h"
#include "BulletManager.h"
#include "ParticleManager.h"

enum AliensShipState
{
    flying,
    dying,
    dead,
    escaped
};

enum AlienResult
{
    No,
    Hit,
    Dying,
    Dead,
    Escaped
};

class AlienShip : public Body
{
public:
    AlienShip(void);
    AlienShip(float2& position, float2& speed, float2* screenSize, ID3D11Texture2D* alienTexture);
    ~AlienShip(void);

    AlienResult Update(float timeDelta, BulletManager* bulletManager);
    void Draw(BasicSprites::SpriteBatch^ spriteBatch);

protected:
    float m_baseY;
    AliensShipState m_state;
    float2* m_screenSize;
};

AlienShip类继承自Body类,其中包含Sprite类,其中包含另一个向量。 但是由于Sprite类在其他地方运行良好,我认为它不是错误的来源。

我想知道为什么会这样,因为我找不到删除临时对象和破坏矢量迭代器之间的关系,如果它完全被破坏了。

此外,程序在Release中编译并运行,但有一些数据损坏。

我正在使用Visual Studio 2012 Beta for Windows 8。

如果您需要更多源代码,请写信。 不幸的是,发布所有代码非常困难,因为这是一个复杂的程序。

1 个答案:

答案 0 :(得分:4)

鉴于当你通过值将项添加到向量时它不起作用但是当你泄漏指针时它确实没有用,我有95%的信心,你的AlienShip的复制构造函数做了浅拷贝,导致你的问题。

编辑:请注意,m_alienShipList.push_back(AlienShip(position, speed, m_screeSize, m_alienShipTexture));会导致您的类的副本,如果复制构造函数无法正常工作,则以后会导致问题。

事实上,如果你粘贴的AlienShip定义是正确的,那么实际上只有默认的复制构造函数可能做错了(由于你有自己的析构函数,这进一步加强了)。 p>

实现执行深层复制的复制构造函数,或者更优选地重写类以使用RAII为您管理内存,以便默认副本正确。