1

我正在尝试创建一个垂直滚动的射击游戏,当你按下空格键时会创建一个子弹,然后当子弹离开屏幕时,子弹会被摧毁。我通过矢量 delcared 跟踪子弹,因为vector<BULLET> bullets;当我尝试销毁屏幕外的任何子弹时,会出现大量错误,例如: c:\mingw\bin\..\lib\gcc\mingw32\4.7.0\include\c++\bits\stl_algobase.h|384| required from '_OI std::__copy_move_a(_II, _II, _OI) [with bool _IsMove = true; _II = BULLET*; _OI = BULLET*]'|

我的代码如下所示:

for( auto it = bullets.begin(); it != bullets.end(); ){
    if( it->is_dead()){
        it = bullets.erase(it);
    }else{
        it++;
    }
}

让我感到沮丧的部分是我有类似的循环,可以删除任何需要在包含指针的向量中删除的游戏对象:

for( auto it = activeInstances.begin(); 
it != activeInstances.end(); ){
    if( (*it)->is_dead()){
        it = activeInstances.erase(it);
    }else{
        it++;
    }
}

这个工作得很好。

编辑:我不确定它是否有区别,但仅供参考,我将稍后出现在将项目符号添加到向量的同一函数中添加部分:

if( key[SPACE] && reload == 0){
    reload = reloadTime;
    BULLET newBullet;
    newBullet.init( x, y);
    bullets.push_back( newBullet);
}
4

3 回答 3

1

您的代码片段不同:

if( it->is_dead()){

对比

if( (*it)->is_dead()){
于 2013-07-15T09:20:19.300 回答
1

我建议使用erase-remove idiom来删除项目std::vector (你可以考虑这个 StackOverflow Q&A进行详细讨论,也适用于其他容器):

// Erase elements matching "erasing_condition" from vector "v"
v.erase( std::remove_if(v.begin(), v.end(), erasing_condition), v.end() );

在您的特定情况下,您可能希望使用如下代码:

//
// Erase elements matching "BULLET.is_dead()" from vector "bullets".
// ("bullets" is a "vector<BULLET>")
//
bullets.erase
(     
    std::remove_if
    ( 
        bullets.begin(), 
        bullets.end(), 

        // Erasing condition
        [](const BULLET& bullet)
        { 
            return bullet.is_dead(); 
        }   
    ), 
    bullets.end() 
);

顺便说一句:作为风格指南,我更喜欢C++ 类名(通常为 C++ 中的预处理器宏保留......)BulletBULLETALL_UPPERCASE_STYLE

于 2013-07-15T11:27:06.723 回答
0

正如有人在其中一条评论中提到的那样,问题是我没有在我的 BULLET 类中使用移动构造函数(我是一个完全的菜鸟,在被提及之前我什至不知道那是什么)。我真的很感谢大家的帮助,我实际上学到了很多东西!

于 2013-07-15T18:48:02.257 回答