向量迭代器不是增量.erase（）

Question

我正在尝试删除此向量中与玩家发生冲突的任何元素。 但是，当我尝试从向量中删除元素时，程序崩溃，并且出现了错误； “向量迭代器不增量”。

for (std::vector<Coin>::iterator i=CoinSet.begin(); i!=CoinSet.end(); i++) 
{
    if (i->PlayerClear(player.collider()) == true)
    {
        score++;
        cout<<score<<endl;
        CoinSet.erase(i);
    }
}

直到“ CoinSet.erase（i）”为止，这段代码都运行良好，我尝试在各个点使用“ CoinSet.clear（）”，但无济于事。 在这方面的任何帮助将是巨大的，谢谢！

Answer 1

已经讨论到死了。 您不得对无效的迭代器进行操作。 您想要这样的东西：

for (auto it = CoinSet.begin(); it != CoinSet.end(); /* no increment here! */ )
{
    if (/* ... */)
    {
        // ...
        CoinSet.erase(it++);
    }
    else
    {
        ++it;
    }
}

Answer 2

我不喜欢将++语句放入参数中。 因此，delete（）返回一个指向下一个元素的迭代器，因此可以将擦除行替换为：

it = CoinSet.erase(it); // iterator is replaced with valid one