从向量C ++中删除多个元素

Question

这是一些代码，用于检查2个单元在相互攻击后是否被杀死，我通过了向量中的位置，但是当我删除一个单元时，向量的大小已更改，因此第二个单元超出范围。 如何同时删除两者？

if ((MyHealth <= 0) && (EnemyHealth <= 0))
{
    PlayerUnits.erase(PlayerUnits.begin() + MyUnit, PlayerUnits.begin() + EnemyUnit);
}
else if (MyHealth <= 0)
{
    PlayerUnits.erase(PlayerUnits.begin() + MyUnit);
}
else if (EnemyHealth <= 0)
{
    PlayerUnits.erase(PlayerUnits.begin() + EnemyUnit);
}

Answer 1

与其自己编写删除逻辑，不如使用algorithm std::remove_if更好地管理它。 根据您是否具有支持C ++ 11的编译器，谓词可以是lambda或命名函数。

Answer 2

第一点：在您的第一个程序段中，调用erase(x, y)所做的事情与您预期的不同-它删除了从索引x开始直到索引y之前的整个元素范围 。 例如，如果我们有向量[a，b，c，d，e，f，g]，则delete（2,5）将擦除索引2,3,4，因此我们以[a，b， F，G]。 我猜您只想删除两个元素，而不是整个范围。

第二点：正如DieterLücking指出的那样，首先只需删除较高索引元素，如下所示：

if (MyUnit > EnemyUnit) {
    PlayerUnits.erase(PlayerUnits.begin() + MyUnit);
    PlayerUnits.erase(PlayerUnits.begin() + EnemyUnit);
} else {
    PlayerUnits.erase(PlayerUnits.begin() + EnemyUnit);
    PlayerUnits.erase(PlayerUnits.begin() + MyUnit);
}

Answer 3

我认为处理此问题的更好方法是在“单元”类中添加“ isDead”条件：

void Unit::Update()
{
    //other stuff
    if(this->m_health <= 0) this->m_isDead = true;
}

然后在主循环中：

void Game::Update()
{
   size_t size = PlayerUnits.size();

   //iterate backwards, so there is no skipping
   for(int i = size-1; i>= 0; i--)
   {
       PlayerUnits[i]->Update();
       if(PlayerUnits[i]->isDead()) PlayerUnits.erase(PlayerUnits.begin() + i);
   }
}

至少这是我个人的方式。