0

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

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 中编译和运行，但有一些数据损坏。

我正在使用适用于 Windows 8 的 Visual Studio 2012 Beta。

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

4

1 回答 1

4

鉴于当您按值将项目添加到向量时它不起作用，但当您泄漏指针时它会起作用，我有 95% 的信心相信您的复制构造函数AlienShip会执行浅拷贝，从而导致您的问题。

编辑：请注意，这m_alienShipList.push_back(AlienShip(position, speed, m_screeSize, m_alienShipTexture));会导致您的类的副本，如果复制构造函数不能正常工作，它将在以后导致问题。

事实上，如果AlienShip您粘贴的定义是正确的，那么实际上只有默认的复制构造函数可能会做错事（您有自己的析构函数这一事实进一步强化了这一事实）。

要么实现一个执行深度复制的复制构造函数，要么更优选地重写您的类以使用 RAII 为您管理内存，以便默认副本是正确的。

于 2012-07-09T16:37:52.410 回答