刪除循環中的矢量元素

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

然後你得到的好處是不必做太多的結束元素轉移。