Discussion :

[iori11]

Bonjour,

Je suis complètement perdue face à un bug que je n'arrive pas du tout à cibler... Je ne sais pas quelle démarche adopter pour le résoudre et je viens crier au secours et écouter tous les conseils avisés que vous pourrez me donner.


Je travaille en fait sur un projet existant, qui fonctionne sans problème. J'y ai rajouté 2-3 méthodes deci delà et là ça bugge.


Voici les exceptions qui sont générées:

Violation d'accès lors de la lecture de l'emplacement 0x00000014

L'endroit exact où ça plante est lors de l'appel à cette méthode, il ne trouve a priori pas mon_map[cle1][cle2].

Code :

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

1
2
3
4
5
6
7
8
9
10
void MaClasse::Ajout(Cle1* cle1_, Cle2* cle2_, int indice_, int valeur_){
 
	if (mon_map[cle1_][cle2_].empty() == 1 ){
		for (int i = 0; i<=CONSTANTE; i++) {mon_map[cle1_][cle2_].push_back(0);}
	}
 
 
 
	mon_map[cle1_][cle2_][indice_] = valeur_;
}

Pour info, mon_map est déclarée comme suit:

Code :

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

map<Cle1*, map<Cle2*, vector<int> > > mon_map;

A quelle type d'erreur cela peut correspondre? Est-ce possible que dans mon code il y ait une toute autre erreur qui se répercute par une violation d'accès par la suite?

Mon message est un peu brouillon, excusez-moi... Je ne sais pas si c'est très simple de relire des bouts de code comme ça, mais si vous avez déjà rencontré les mêmes erreurs peut-être que votre expérience pourra m'aider à orienter mes recherches!!


Merci d'avance pour vos réponses.


EDIT: j'ai changé quelques noms d'objets.

[3DArchi]

Salut,
Il faut savoir que map[] créé un élément si celui-ci n'existe pas. Je préfère systématiquement utiliser map::find pour retrouver un élément et n'utiliser [] que pour l'ajout ou en ayant au préalable testé l'existence de l'élément. Donc, la première ligne à laquelle je me suis arrêté :

Code :

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

1
2
3
if (mon_map[cle1_][cle2_].empty() == 1 ){
	for (int i = 0; i<=CONSTANTE; i++) {mon_map[cle1_][cle_2].push_back(0);}
}

Perso, je change en :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
if(
      (mon_map.find(cle1_)!=mon_map.end())
   && (mon_map[cle1_].find(cle2_)!=mon_map[cle1_].end())
{
   if (mon_map[cle1_][cle2_].empty()){
	   for (int i = 0; i<=CONSTANTE; i++) {mon_map[cle1_][cle2_].push_back(0);}
   }
   mon_map[cle1_][cle2_][indice_] = valeur_;
}

[iori11]

Merci pour l'info sur l'opérateur [], j'essaierai de l'utiliser avec précaution.

Là, rien ne semble être créé par cet opérateur car rien que le test mon_map[cle1_][cle2_].empty() == 1 plante.

Il semblerait que mon_map[cle1_][cle2_] n'existe pas, mais je ne comprends vraiment pas pourquoi...

Avec si peu de code vous pouvez difficilement m'aider, mais je prends toutes remarques d'ordre général qui m'aideraient à avancer!

[3DArchi]

Là, rien ne semble être créé par cet opérateur car rien que le test mon_map[cle1_][cle2_].empty() == 1 plante.

Mon message n'était pas très clair, mais il est possible que cela plante car il n'existe pas d'entrée avec les clés que tu lui fournis et donc il créé des valeurs par défaut qui sont mal initialisées -> d'où l'erreur.

[iori11]

Hum hum... intéressant.
Il me reste donc à voir pourquoi il pourrait ne pas exister d'entrée avec les clés fournies, et là c'est un grand mystère...

[3DArchi]

Une autre remarque qui m'a échappé à la première lecture :

Code :

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

if(mon_map[cle1_][cle2_].empty() == 1)

Il vaut mieux utiliser true pour les comparaisons avec un type bool :

Code :

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

if(mon_map[cle1_][cle2_].empty() == true)

Certain préfère écrire d'abord la constante et ensuite le test :

Code :

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

if(true==mon_map[cle1_][cle2_].empty() )

Ca évite les étourderies de type :

Code :

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

if(mon_map[cle1_][cle2_].empty() = true)

ou tout simplement :

Code :

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

if(mon_map[cle1_][cle2_].empty())

[iori11]

Oui, j'ai vu les dégâts que pouvaient engendrer ces étourderies sur une autre discussion du forum! Merci pour ces remarques.