Problem med iteratorer och loopar

for (std::vector<Sprite*>::iterator outerIter = sprites.begin(); outerIter != sprites.end(); outerIter++)
{
Sprite* spriteOne = *outerIter;
for (std::vector<Sprite*>::reverse_iterator innerIter = sprites.rbegin(); innerIter != sprites.rend(); ++innerIter)
{
Sprite* spriteTwo = *innerIter;
if (spriteOne->getRect() != spriteTwo->getRect() && spriteOne->getRect().overlaps(spriteTwo->getRect()))
{
spriteOne->collisionReaction(spriteTwo);
spriteTwo->collisionReaction(spriteOne);
spriteOne->collisionDestroy(sprites);
spriteTwo->collisionDestroy(sprites);
}
if (!spriteOne)
break;
}
}

Det som händer är att man hoppar in i yttre loopen, sen inre loopen och kör igenom inre loopen en gång. Sedan så ska den hoppa upp och köra inre loopen igen, men där ger den felet "Expression: vector iterator not decrementable".

Här är funktionen man kallar på (collisionReaction gör inget annat än att sätta lite olika lokala värden i objecten):

void Enemy::collisionDestroy(std::vector<Sprite*>& sprites)
{
for (std::vector<Sprite*>::iterator iter = sprites.begin(); iter != sprites.end(); iter++)
{
Enemy* tmp = dynamic_cast<Enemy*>(*iter);
if (this == tmp && collisionType == 3 || collisionType == 1)
{
sprites.erase(iter);
break;
}
}
}

Hjälp

EDIT: Ledsen över ingen indentering, vet inte hur man gör.

Vad är syftet? Du har en vektor av sprites, som du vill kolla mot varandra? Om de överlappar ska båda "reagera" och förstöras?

Om du lägger till eller tar bort element så blir existerande iteratorer felaktiga om datan flyttats till ett nytt minnesområde. Använd indexeringsoperatorn [] eller spara elementen som ska tas bort i en separat vector och ta bort dem efter att du klar med itereringslooparna.

http://www.flashback.org/t2302444

Har ett behov av att lösa det med iteratorer. Fler tips?

Använd returvärdet från erase().

Källa: http://stackoverflow.com/questions/3...ntable/3779252

Att använda iteratorn som erhålls från erase() är dock inte helt trivialt i detta fall med tre olika samtidiga itereringar över containern. Fundera på att göra en kopia på vektorn. Se också upp med minnesläckor om "sprites" innehåller pekare till dynamiskt allokerade objekt: erase() deallokerar ej sådant minne automatiskt.