我知道有与此类似的问题，但我没有设法在他们的帮助下找到我的代码的方法。我只想通过检查循环内该元素的属性来删除/删除向量的元素。我怎样才能做到这一点？我尝试了以下代码，但收到了模糊的错误消息：
'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 属于同一个问题吗？因此我需要一个赋值运算符吗？为什么？
【问题讨论】：
您不应在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 在for-body 本身中会增加。
至于报错“'operator =' function is available in 'Player'”，是因为使用erase()内部使用operator=来移动vector中的元素。为了使用erase()，类Player的对象必须是可赋值的，这意味着你需要为Player类实现operator=。
无论如何，您应该尽可能避免使用raw loop1，而应该更喜欢使用算法。在这种情况下，流行的Erase-Remove Idiom 可以简化您的工作。
allPlayers.erase(
std::remove_if(
allPlayers.begin(),
allPlayers.end(),
[](Player const & p) { return p.getpMoney() <= 0; }
),
allPlayers.end()
);
1.这是我看过的the best talks by Sean Parent 之一。
【讨论】：
it = allPlayers.erase(it); 请仔细看作业！否则，请随时发布更好的答案。
忘记循环并使用 std 或 boost 范围算法。
使用 Boost.Range en Lambda 它看起来像这样：
boost::remove_if( allPlayers, bind(&Player::getpMoney, _1)<=0 );
【讨论】：
USE JQUERY!!1! 一样有用。
if(condition) it = x.erase(it); else { file << *it; ++it; }。正如您所看到的，当元素不适合删除时是否要执行其他操作，您不能使用remove_if。即使你使用它，你也可能不得不再次遍历循环。
您的具体问题是您的 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 的回答。
【讨论】：
if(allPlayers.empty() == false) {
for(int i = allPlayers.size() - 1; i >= 0; i--) {
if(allPlayers.at(i).getpMoney() <= 0) {
allPlayers.erase( allPlayers.begin() + i );
}
}
}
这是我删除向量中元素的方法。 很容易理解，不需要任何技巧。
【讨论】：
或者向后循环。
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());
然后您就不必对末端元素进行如此多的移动了。
【讨论】：
std::vector::erase(iterator) 删除迭代器指向的单个元素。在您的示例中，它将尝试删除 std::remove_if 返回的迭代器所指向的元素——这是一个传递结束的迭代器，因此这几乎肯定是不正确的（并且会导致崩溃）。它应该是：allPlayers.erase(std::remove_if(...), allPlayers.end())，而是删除范围内的所有元素。
迟到的答案，但看到了低效的变体：
std::remove 或 std::remove_if 是要走的路。高效移除元素的代码：
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());
您可能需要显式编写这样的循环 e. G。如果您需要迭代器本身来确定是否要删除元素（条件参数需要接受对元素的引用，还记得吗？），例如。 G。由于与继任者/前任者的特定关系（如果这种关系是平等的，则有std::unique）。
【讨论】：