删除循环中的矢量元素

Question

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

'operator ='功能在'Player'中不可用。

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

我该怎么办？

更新：您认为vector::erase with pointer member问题属于同一问题吗？我是否需要一个赋值操作符？为什么？

Answer 1

你不应该在for循环增量it：

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++在那里是不需要的，因为it在身体本身中增加了。

至于错误“‘运算符=’功能处于“播放”不可用”，它来自的erase()使用其内部使用operator=移动向量中的元素。为了使用erase()，类Player的对象必须是可分配的，这意味着您需要为Player类实现operator=。

Answer 2

忘掉循环并使用std或boost范围algorthims。
使用Boost.Range连接LAMBDA它应该是这样的：

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

Answer 3

您的具体问题是，你的Player类没有赋值运算符。您必须让“玩家”可以复制或移动，才能将其从矢量中移除。这是由于该矢量需要连续，因此需要重新排列元素以填充删除元素时创建的间隙。

另外：

使用std算法

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; 
}); 

见TIMW的答案，如果你没有对C++ 11的支持lambda表达式。

Answer 4

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

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

Answer 5

或者向后循环。

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

Answer 6

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

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

然后你得到的好处是不必做太多的结束元素转移。