De la difficulté de la suppression de vecteur de pointeurs

L'effacement d'un élément à partir du vecteur d'invalider l'itérateur, de sorte que vous ne pouvez pas continuer à l'itération par la suite. Dans ce cas, je ne serais pas effacer les éléments à l'intérieur de la boucle; j'avais clairement le vecteur par la suite:

for (auto it = gamestates_.begin(); it != gamestates_.end(); ++it){
    delete *it;
}
gamestates_.clear();

Bien que, si c'est dans le destructeur et le vecteur est sur le point d'être détruit, il n'y a pas de point de compensation.

Si vous avez besoin d'effacer à l'intérieur d'une boucle (peut-être parce que vous voulez effacer certains éléments), alors vous avez besoin d'un peu plus de soin de conserver l'itérateur valide:

for (auto it = gamestates_.begin(); it != gamestates_.end();){ //No ++ here
    if (should_erase(it)) {
        it = gamestates_.erase(it);
    } else {
        ++it;
    }
}

Une chose que je n'ai pas encore essayé est à l'aide de unique_ptr mais je suis sûr que ce doit être capable de les gérer sans les utiliser. S'il vous plaît corrigez-moi si je me trompe.

Si vous ne souhaitez gérer la dynamique des objets par la vapeur d'eau par cela, alors assurez-vous de suivre les La règle de Trois: vous aurez besoin de mettre en œuvre (ou supprimer) le constructeur de copie et de copie opérateur d'affectation pour éviter les "superficielle" de la copie, vous laissant avec deux vecteurs qui essayez de supprimer les mêmes objets. Vous aurez également besoin de prendre soin de supprimer les objets en tout autre lieu qui supprime ou remplace. Stocker des pointeurs intelligents (ou les objets eux-mêmes, si vous n'avez pas besoin de pointeurs pour polymorphisme) va prendre soin de toutes ces choses pour vous, alors je vous recommandons de toujours.

Je suis conscient que je devrais effacer le vecteur après la boucle, mais c'est ce que j'ai après avoir essayé chaque méthode normale de supprimer les pointeurs. Il ne semble pas aimer la commande supprimer.

La cause la plus probable est que vous n'êtes pas en suivant la Règle de Trois, et sont accidentellement en essayant de supprimer les objets mêmes deux fois après la copie du vecteur. Il est également possible que GameState est une classe de base et vous avez oublié de lui donner un destructeur virtuel, ou que les pointeurs ont été corrompu par un autre code.

OriginalL'auteur Mike Seymour

L'effacement d'un élément de vecteur invalide les itérateurs. Supprimer les objets pointeur par les éléments et puis clear() le contenu du vecteur.

OriginalL'auteur Marius Bancila

Préfèrent utiliser unique_ptr. Vous dites que vous devrait être en mesure de gérer sans l'aide de leur comme si l'obtention d'un pointeur intelligent pour faire le travail pour vous, c'est une sorte de terrible imposition.

Ils sont là pour rendre votre vie plus facile, vous n'avez pas à se sentir coupable de ne pas faire le dur travail à la main.

Et avec votre code existant, il suffit de ne pas appeler erase. Le vecteur va être détruit de toute façon, non? Il va prendre soin de tout ce que lui-même.

OriginalL'auteur Useless