Problème avec std::map

**Mat.M** · 17/08/2010, 19h25

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

**dtrosset** · 18/08/2010, 09h18

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

**ronan99999** · 18/08/2010, 11h11

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.

**JolyLoic** · 18/08/2010, 13h34

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

**Mat.M** · 18/08/2010, 16h56

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

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());

**JolyLoic** · 18/08/2010, 19h33

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...).

**koala01** · 21/08/2010, 12h41

Salut,

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

**Mat.M** · 23/08/2010, 17h20

Bonjour à tous encore merci pour les réponses.
Eh bien c;est pareil avec un simple
std::map<std::string,bool>

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

**3DArchi** · 24/08/2010, 08h24

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 ? )

**Mat.M** · 24/08/2010, 19h44

Salut la compagnie

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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..

gl · 24/08/2010, 22h37

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 ?

**Mat.M** · 25/08/2010, 17h21

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

.
Je ne fais aucune opération spéciale

**Klaim** · 01/09/2010, 14h55

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.

**JolyLoic** · 01/09/2010, 17h58

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...)

**vikki** · 01/09/2010, 18h05

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...