删除循环内向量的元素

我知道也有与此类似的问题，但是我没有设法借助他们的帮助在我的代码中找到出路。我只想通过在循环内检查此元素的属性来删除/删除向量的元素。我怎样才能做到这一点？我尝试了以下代码，但收到了模糊的错误消息：

“播放器=”功能在“播放器”中不可用。

 for (vector<Player>::iterator it = allPlayers.begin(); it != allPlayers.end(); it++)
 {
     if(it->getpMoney()<=0) 
         it = allPlayers.erase(it);
     else 
         ++it;
 }

我该怎么办？

更新：您是否认为带有指针成员的问题vector :: erase属于同一问题？因此，我需要一个赋值运算符吗？为什么？

您不应it在for循环中递增：

for (vector<Player>::iterator it=allPlayers.begin(); 
                              it!=allPlayers.end(); 
                              /*it++*/) <----------- I commented it.
{

   if(it->getpMoney()<=0) 
      it = allPlayers.erase(it);
  else 
      ++it;
 }

注意注释部分；it++那里不需要，因为itfor-body本身在增加。

对于错误“ 'operator ='函数在'Player'中不可用”，它来自erase()内部使用operator=来移动向量中元素的用法。为了使用erase()，类的对象Player必须是可分配的，这意味着您需要operator=为Player类实现。

无论如何，您应尽可能避免使用原始循环¹，而应改用算法。在这种情况下，流行的“删除擦除惯用语”可以简化您的操作。

allPlayers.erase(
    std::remove_if(
        allPlayers.begin(), 
        allPlayers.end(),
        [](Player const & p) { return p.getpMoney() <= 0; }
    ), 
    allPlayers.end()
); 

^{1.这是我看过的肖恩·帕特里纳（Sean Parent）最好的演讲之一。}

if(allPlayers.empty() == false) {
    for(int i = allPlayers.size() - 1; i >= 0; i--) {
        if(allPlayers.at(i).getpMoney() <= 0) {
            allPlayers.erase( allPlayers.begin() + i ); 
        }
    }
}

这是我删除向量中元素的方法。这很容易理解，不需要任何技巧。

忘记循环并使用std或boost范围算法。
使用Boost.Range en Lambda看起来像这样：

boost::remove_if( allPlayers, bind(&Player::getpMoney, _1)<=0 );

您的特定问题是您的Player班级没有赋值运算符。您必须使“ Player”可复制或移动，才能将其从矢量中删除。这是由于向量必须是连续的，因此需要对元素进行重新排序以填充删除元素时创建的间隙。

也：

使用标准算法

allPlayers.erase(std::remove_if(allPlayers.begin(), allPlayers.end(), [](const Player& player)
{
    return player.getpMoney() <= 0;
}), allPlayers.end());

甚至更简单（如果您有增强能力）：

boost::remove_erase_if(allPlayers, [](const Player& player)
{
    return player.getpMoney() <= 0;
});

如果您不支持C ++ 11 lambda，请参见TimW的答案。

或向后循环。

for (vector<Player>::iterator it = allPlayers.end() - 1; it != allPlayers.begin() - 1; it--)
    if(it->getpMoney()<=0) 
        it = allPlayers.erase(it);

C ++ 11引入了新的函数集合，这些函数将在此处使用。

allPlayers.erase(
    std::remove_if(allPlayers.begin(), allPlayers.end(),
        [](auto& x) {return x->getpMoney() <= 0;} ), 
    allPlayers.end()); 

然后，您将获得不必对末端元素进行太多调整的优势。

答案较晚，但看到无效的变体：

1. std::remove还是std::remove_if要走的路。
2. 如果由于任何原因这些功能不可用或由于其他任何原因无法使用，请执行这些操作以使您远离自己。

有效删除元素的代码：

auto pos = container.begin();
for(auto i = container.begin(); i != container.end(); ++i)
{
    if(isKeepElement(*i)) // whatever condition...
    {
        *pos++ = *i; // will move, if move assignment is available...
    }
}
// well, std::remove(_if) stops here...
container.erase(pos, container.end());

您可能需要明确地编写这样的循环，例如，如果您需要迭代器本身来确定是否要删除元素（条件参数需要接受对元素的引用，还记得吗？），例如由于与后继者/前任者的特定关系（不过，如果这种关系是平等的，则存在std::unique）。