Problème avec std::map

Bonjour,

j'ai un gros problème avec l'utilisation de std::map.
Je m'arrache les cheveux de la tête.
Lorsque j'appelle ::clear () cela plante , sans aucune initialisation, le conteneur étant totalement vide.
Même avec quelques éléments

Code:

---

1 2 3 4 5 6 7 8 9

struct Mastruct { std :: string strDetail; BOOL bBooleen ; };   std::map<std::string ,MaStruct> monConteneur; monConteneur.clear();/// plante

---

le code plante dans xtree de la STL

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

#if _HAS_ITERATOR_DEBUGGING void _Orphan_ptr(_Myt& _Cont, _Nodeptr _Ptr) const { // orphan iterators with specified node pointers _Lockit _Lock(_LOCK_DEBUG); const_iterator **_Pnext = (const_iterator **)&_Cont._Myfirstiter; while (*_Pnext != 0) if ((*_Pnext)->_Ptr == _Myhead || _Ptr != 0 && (*_Pnext)->_Ptr != _Ptr) _Pnext = (const_iterator **)&(*_Pnext)->_Mynextiter; else { // orphan the iterator (*_Pnext)->_Mycont = 0; *_Pnext = (const_iterator *)(*_Pnext)->_Mynextiter; } } #endif /* _HAS_ITERATOR_DEBUGGING */

---

Pour des raisons obscures (*_Pnext) est différent de NULL ou 0x00000000
Quelqu'un a-t-il une idée ?

J'utilise VS2008,le projet est compilé avec l'option multithread

Bonjour,

Comme le type MaStruct contient un membre std::string, il faut absolument créer un constructeur par défaut, un constructeur de copie, et un opérateur de copie. C'est strictement nécessaire pour pouvoir utiliser MaStruct dans un des conteneurs de la librairie standard.

Ces soucis viennent peut-être de là.

Didier

Bonjour,
j'ai exécuter ton code pas soucis.
Il manque pas quelque chose?

Par contre tu utilises le type BOOL (win32?), tu peux directement utiliser bool.

Sinon en ce qui concerne le constructeur par défaut, l'affectation et le constructeur par recopie etc...

Tu en as un par défaut sans l'écrire.

Il y a des chances que le problème soit ailleurs, et ait corrompu la mémoire de ta map...

Ok merci à tous pour les réponses , je vais voir concernant le constructeur par défaut
:ccool:

Citation:

---

Envoyé par ronan99999

j'ai exécuter ton code pas soucis.
Il manque pas quelque chose?

---

non j'ai donné le code tel que je l'utilise ; j'ai des plantages simplement en faisant clear() ou bien même avec MonConteneurMap(iterateur.begin(),iterateur.end());

Telle que présentée, Mastruct a tout ce qu'il faut pour être mise dans une map. Pas besoin de lui ajouter quoi que ce soit (et il y a toujours le risque de se planter en le faisant...).

Salut,

Citation:

---

Envoyé par dtrosset

Bonjour,

Comme le type MaStruct contient un membre std::string, il faut absolument créer un constructeur par défaut, un constructeur de copie, et un opérateur de copie. C'est strictement nécessaire pour pouvoir utiliser MaStruct dans un des conteneurs de la librairie standard.

Ces soucis viennent peut-être de là.

Didier

---

Absolument pas...

En C++, les classes et les structures sont strictement équivalentes, à la seule exception près de la visibilité par défaut de leur membres (private pour class et public pour struct).

Le compilateur introduit un constructeur par défaut, un constructeur par copie, un opérateur d'affectation et un destructeur automatiquement si le développeur n'en fournit pas d'équivalent.

Tu ne dois fournir une version propre de ces fonctions que lorsque tu manipule toi même des pointeurs à l'intérieur de ta classe ou de ta structure.

La classe std::string manipule, effectivement, un ou des pointeurs en interne, mais, comme les quatre fonctions sont correctement définies (en plus de beaucoup d'autres, d'ailleurs), tu n'a strictement aucun soucis à te faire pour une structure ou une classe qui manipule un objet de type std::string

Bonjour à tous encore merci pour les réponses.
Eh bien c;est pareil avec un simple
std::map<std::string,bool> :furax: je suis bloqué à cause de cela et je m'arrache les cheveux de la tête
Même avec Vs2010 c'est pareil...
bon je vais continuer à chercher mais merci tout de même

Salut,
Comme l'a dit Loïc, on dirait que ta mémoire est corrompue ailleurs. Dit autrement, ton plantage de std::map est du à un bug ailleurs. (Une référence sur un map détruit ? )

Salut la compagnie :mouarf:
bon j'ai résolu le problème.
En déclarant comme pointeur avec un new cela fonctionne.
Je ne comprends pas pourquoi comme membre d'une classe cela plante ( non déclarée comme pointeur )

Code:

---

1 2 3

std::map<std::string ,MaStruct> *monConteneur; monConteneur=new std::map<std::string ,MaStruct>;

---

Citation:

---

Envoyé par 3DArchi

Salut,
Comme l'a dit Loïc, on dirait que ta mémoire est corrompue ailleurs. Dit autrement, ton plantage de std::map est du à un bug ailleurs. (Une référence sur un map détruit ? )

---

ok merci pour la réponse.
Non aucune insertion , rien..

Citation:

---

Envoyé par Mat.M

bon j'ai résolu le problème.
En déclarant comme pointeur avec un new cela fonctionne.
Je ne comprends pas pourquoi comme membre d'une classe cela plante ( non déclarée comme pointeur )

---

Je doute fortement que le problème de fond soit réellement résolu. Il est probable que le code tombe tout simplement en marche.

Peux-tu fournir un exemple minimal et compilable qui reproduit le problème ?

Citation:

---

Envoyé par gl

Je doute fortement que le problème de fond soit réellement résolu. Il est probable que le code tombe tout simplement en marche.

Peux-tu fournir un exemple minimal et compilable qui reproduit le problème ?

---

salut GL cela fonctionne parfaitement de toute façon l'appli n'est pas critique :mouarf:.
Je ne fais aucune opération spéciale

Tu as un probleme "caché". L'ignorer ne fera que te rendre encore plus ignorant.

Ici ta solution n'est pas censé changer quoi que ce soit à ton problème. Pire : tu complexifie l'utilisation de la map et permet de plus facilement faire des erreurs avec. Tu n'as fais que mettre le probleme sous le tapis, mais ça dépasse et ça se voit.

Si cette solution marche, c'est un hasard. De loin ça ressemble beaucoup a de la corruption de mémoire.

Je serais toi, je vérifierai tout le code : je chercherai d'abord tout ce qui est alloué avec new ou malloc et je chercherai alors les delete et free qui DOIVENT correspondre. Ensuite je vérifierai toutes mes boucles par index voir si je ne fais pas de déplacement. Enfin je vérifierai toutes manipulation de pointeurs, en particulier les déplacements "manuels" de pointeurs.

Citation:

---

Envoyé par Klaim

Je serais toi, je vérifierai tout le code : je chercherai d'abord tout ce qui est alloué avec new ou malloc et je chercherai alors les delete et free qui DOIVENT correspondre. Ensuite je vérifierai toutes mes boucles par index voir si je ne fais pas de déplacement. Enfin je vérifierai toutes manipulation de pointeurs, en particulier les déplacements "manuels" de pointeurs.

---

Certains outils peuvent aussi aider à debugger ce genre de choses (valgring, intel parallel studio...)

J'ai eu ce type de problème à une époque, plantage aléatoire lors de constructions/destructions d'objets. C'était dû à un dépassement de tampon en écriture sur un tableau dynamique qui avait lieu un peu avant. Si ca peut t'aider...

Citation:

---

Envoyé par JolyLoic

Certains outils peuvent aussi aider à debugger ce genre de choses (valgring, intel parallel studio...)

---

Oui, en fait j'ai tendance a aimer chercher a la main parceque ça me permet de faire du refactoring au passager (relire, simplifier, factoriser etc). :mrgreen:

Mais bon sur mes projets perso j'ai pas de contrainte forte d etemps...