如何解决C ++:列表迭代器不可递增
| 尝试删除列表的最后一个元素时遇到此错误。我调试了代码,并能够找出导致它的原因和原因,这是我的代码: for(Drop_List_t::iterator i = Drop_System.begin(); i != Drop_System.end() && !Drop_System_Disable; /**/)
{
if(Player->BoundingBox.Intersect(&(*i)->BoundingBox))
{
i = Drop_System.erase(i);
}
++i; //List iterator crashes here if last entry was deleted
}
我不知道自己在做什么错...有什么建议吗?
解决方法
您的算法有缺陷,因为您不了解ѭ1返回的结果。
使用
erase
时,它将删除迭代器指向的元素,并将迭代器返回到下一个元素。
如果要遍历列表的所有元素,则意味着每当使用erase
时,都不应进一步对其进行递增。
这是您应该获得的普通代码:
if (Player->BoundingBox.Intersect(i->BoundingBox)) {
i = Drop_System.erase(i);
}
else {
++i;
}
这就巧妙地解决了您遇到的问题!因为当您对最后一个元素erase
时,erase
将返回与end
相同的迭代器,即指向最后一个倒数元素的迭代器。该迭代器绝不能递增(如果列表不为空,则可以递减)。
, 您需要在else
子句中放入++i
。 erase
函数将返回下一个有效的迭代器-,然后对其进行递增,确保不对每个元素进行迭代。仅在选择不删除的情况下才应增加它。
, 你要:
if(Player->BoundingBox.Intersect(&(*i)->BoundingBox))
{
i = Drop_System.erase(i);
}
else {
++i;
}
版权声明:本文内容由互联网用户自发贡献,该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 dio@foxmail.com 举报,一经查实,本站将立刻删除。