Vecteur Des Itérateurs Incompatible

J'ai une classe avec un std::vector membre de données par exemple

class foo{
public:

const std::vector<int> getVec(){return myVec;} //other stuff omitted

private:
std::vector<int> myVec;

};

Maintenant à une partie de mon code principal, je suis en train de parcourir le vecteur comme ceci:

std::vector<int>::const_iterator i = myFoo.getVec().begin();
while( i != myFoo.getVec().end())
{
   //do stuff
   ++i;
}

Le moment je atteindre cette boucle, j'obtiens l'erreur susmentionnée.

  • Vous êtes de retour une copie du vecteur, probablement, vous voulez retourner un const std::vector<int>&
  • Avez-vous vraiment envie de retourner une copie de votre vecteur?
  • Veuillez ne pas retourner const valeurs des fonctions, car il inhibe C++11 sémantique de déplacement.
  • l'erreur est-ci dans le titre je présume.
  • Je n'ai jamais vu une erreur qui lit simplement, sur son propre, "Vecteur Itérateurs Incompatible"
  • C'est un VC++ runtime affirmation message.
  • Rien d'autre à tous est de sortie?
  • Non pas que je me souviens (autres que l'habituel affirmation stacktrace info), mais ensuite je ne vois pas cette affirmation très souvent moi-même. ;-]

InformationsquelleAutor IG83 | 2011-12-07
  1. 70

    La raison pour laquelle vous faites cela, c'est que les itérateurs sont de deux (ou plus) des copies différentes de myVec. Vous êtes de retour une copie du vecteur à chaque appel à myFoo.getVec(). Donc les itérateurs sont incompatible.

    Quelques solutions:

    Retour const référence à la std::vector<int> :

    const std::vector<int> & getVec(){return myVec;} //other stuff omitted

    Une autre solution, probablement préférable serait de faire une copie locale du vecteur et de l'utiliser pour obtenir votre itérateurs: 

    const std::vector<int> myCopy = myFoo.getVec();
std::vector<int>::const_iterator i = myCopy.begin();
while(i != myCopy.end())
{
  //do stuff
  ++i;
}

    Aussi +1 pour ne pas using namespace std;

    InformationsquelleAutor FailedDev
  2. 10

    Vous êtes de retour une copie du vecteur. Parce que vous êtes de retour en valeur votre appel à begin() et end() sont complètement différents vecteurs. Vous avez besoin de retourner un const & à elle.

    const std::vector<int> &getVec(){return myVec;}

    Je voudrais faire cela de façon un peu différente cependant. Je voudrais faire de la classe un peu comme un conteneur standard

    class Data
{
   public:
      typedef std::vector<int>::const_iterator const_iterator;

      const_iterator begin() const { return myVec.begin(); }
      const_iterator end() const { return myVec.end(); }
};

Data::const_iterator i=myFoo.begin();

while(i != myFoo.end())
{
//
}
    InformationsquelleAutor Adrian Cornish
  3. 4

    Une autre cause de l'MSVC STL debug assertion "vecteur itérateurs incompatible" exploitation invalidé itérateur.

    I. e. v.erase(i), puis de les comparer i != v.end() l'effacer invalide i et ne peut donc pas être utilisé dans une comparaison.

    InformationsquelleAutor nmr
  4. 2

    Le problème, c'est que vous avez toujours retourner une autre copie du vecteur. Référence:

    const std::vector<int>& getVec(){return myVec;} //other stuff omitted
    InformationsquelleAutor jdehaan
  5. 2

    bien, je ne pense pas que vecteur de copie pourrait être la seule cause, qui semble être trop obivious pour moi.

    dans mon cas, j'ai juste trouver que corrompu pile, tas, uninteneded changements pourraient aussi entraîner dans cette défaillance, et il sera en fait de cacher la raison sous-jacente. dans mon cas, j'ai changé pour l'utilisation de l'indexeur à parcourir et de trouver la cause racine.

    InformationsquelleAutor zinking
  6. 2

    Une autre raison pour laquelle cette assertion peut déclencher est si vous voulez allouer "foo" avec "malloc" au lieu de "nouveau", effectivement sauter le constructeur(s).

    Il est peu probable de se produire à un projet développé à partir de zéro en C++, mais lors de la conversion de la plaine-C code C++ (remplacement d'un tableau statique[] dans certains struct avec une stl-vecteur) vous pourriez ne pas réaliser que la dynamique des instances de ladite structure (et les membres à l'intérieur) ne vont pas avoir leur constructeur appelé - à moins que vous modifiez également les 'malloc' à 'nouveau'.

    InformationsquelleAutor kalmiya
  7. 1

    Vous faites une constante copie des états vecteur, de ne pas accéder au membre vecteur.

    Changer cela: 

    const std::vector<int> getVec(){return myVec;} //other stuff omitted

    à ceci:

    const std::vector<int> & getVec(){return myVec;} //other stuff omitted

    Pour aller un peu plus loin, l'itérateur vous obtenez à partir de cette déclaration:

    std::vector<int>::const_iterator i = myFoo.getVec().begin();

    est un itérateur à la copie temporaire de votre vecteur, qui disparaît après que l'exécution d'une instruction, d'invalider l'itérateur.

    • Légèrement hors-sujet, mais votre boucle for pour faire des trucs devrait être un membre de la méthode de la classe foo. En général, vous ne devriez pas avoir de choses à l'extérieur d'une classe d'exploitation sur les données internes d'une classe.
    • Et c'est un invalider itérateur pour un différentes conteneur. Deux problèmes.
    InformationsquelleAutor Rob K
  8. 1

    Changement

    const std::vector<int> getVec(){return myVec;}

    à

    const std::vector<int>& getVec(){return myVec;}
    InformationsquelleAutor Igor Oks
  9. 0

    Votre getVec() renvoie une copie de la membre de vecteur, de sorte que les deux getVec() vous appelle à faire pour récupérer les itérateurs obtenir des itérateurs pour des récipients différents. Qui est, vous ne pouvez pas atteindre getVec().fin() à partir d'une autre getVec().begin() itérateur sans invoquer un comportement indéfini.

    Vous pouvez résoudre ce problème de deux façons:

    1) Ont getVec retour const référence (qui est, const std::vector&) (de préférence) ou de...

    2) Remplacer les deux getVec() appelle à un et enregistrer le résultat dans un std::vector variable. Ensuite, utilisez cette variable pour les deux appels à begin() et end(). E. g:

    std::vector<int> v = myFoo.getVec();
std::vector<int>::const_iterator b = v.begin();
std::vector<int>::const_iterator e = v.end();
    InformationsquelleAutor Drew Hall
  10. -1

    Parce que vous êtes de retour en valeur votre appel à begin() et end() sont complètement différents vecteurs. Vous avez besoin de retourner un const & elle

    • Bonjour, et bienvenue à Débordement de Pile. Votre réponse, alors qu'il pourrait être correct, il est assez laconique et difficile à comprendre. Il serait utile si vous pouviez développer; par exemple, si vous incluez du code qui utilise le changement que vous recommandez?
    • Cette réponse a également été utilisé en réponse à une depuis-supprimé de markov généré non-sens de la question et choisi comme réponse à l'absurdité de la question.
    InformationsquelleAutor 亡羊补牢