Discussion :

[peapol]

Bonjour,

dans un programme contenant une map, j'ai besoin de parcourir la map en question, afin de récupérer un élément. J'écris donc le code suivant:

Mais lors de l’exécution j'ai une erreur de segmentation. Je ne vois pas du tout comment faire.

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
 
Forme* Usine::trouverForme(const string& chaine)
{
        map<string, Forme*>::iterator it;
	it= mymap.find(chaine);
	if( it==mymap.end() )
	{
		cout << "non trouvé" << endl;
		return(NULL);
 
	}
	else
	{
		cout << "trouvé" << endl;
		return((*it).second);
	}
}

[Steph_ng8]

Bonsoir.

Tu es sûr que c'est dans cette fonction que survient l'erreur de segmentation ?
Qu'est-ce que mymap (une variable membre, une variable membre statique, une variable globale, etc.) ?
Usine::trouverForme est une fonction membre statique ou non ?
Usine::trouverForme peut-elle être appelée lors de la construction d'un objet statique ?

Je n'ai pas une grosse visibilité de ton projet, mais ça me fait penser à un problème (vicieux...) que j'ai rencontré récemment...
F.A.Q. C++: Qu'est-ce que le 'fiasco dans l'ordre d'initialisation des variables statiques' ?

Si ce n'est pas ça, il va falloir mettre un peu plus de code pour que l'on puisse y voir quelque chose...

[peapol]

mymap est ma donnée privé, de la forme map<string, Forme*>.

dans mon header voici le prototype de créer forme. (pas de statique)

Code :

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

Forme* creerForme(const string& chaine)

creerForme n'est pas appelé lors de la construction d'un objet statique. Il s'agit d'une foction qui ressort un objet de ma map correspondant à la clé qu'il a en argument.

L'erreur de segmentation survient lors du test du programme quand on appelle creerForme.

[Steph_ng8]

Dans ce cas, si tu pouvais au moins mettre le code de Usine::creerForme...

[peapol]

problème de recopiage de code, trouverFrome=creerForme (j'ai changé le nom pour que la fonction soit plus compréhensible.

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
 
Forme* Usine::creerForme(const string& chaine)
{
	map<string, Forme*>::iterator it;
	it= mymap.find(chaine);
	if( it==mymap.end() )
	{
		cout << "non trouvé" << endl;
		return(NULL);
 
	}
	else
	{
		cout << "trouvé" << endl;
		return((*it).second);
	}
}

[Steph_ng8]

Ok.

L'erreur de segmentation survient lors du test du programme quand on appelle creerForme.

De quel test parles-tu ?
Si c'est celui de la ligne 6, ça veut certainement dire que mymap n'est pas (complètement) construit au moment de l'appel de la fonction.

Je ne vois pas d'autre explication.
Mais encore une fois, avec si peu de code, c'est difficile de se rendre bien compte...

[ptyxs]

Il serait sans doute de meilleure qualité de programmation de passer ta mymap en référence comme argument à ta fonction. On y verrait déjà plus clair.

[peapol]

Dans mon main j'ai:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
bool b;
Forme* ptrF;
string cle;
 
cout << "quel forme voulez vous copier? " << endl;
cin >> cle;
ptrF=mon_usine.creerForme(cle);
cout << "quel clés pour le nouveaux? " << endl;
cin >> cle;
b=mon_usine.ajouterForme(cle,ptrF);

ma fonction creerForme (je l'ai modifié pour qu'elle soit plus courte)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
Forme* Usine::creerForme(const string& chaine)
{
	if( mymap.count(chaine)>0)
	{
		return(mymap[chaine]->dupliquer());
	}
	else
	{
		cout << "non trouvé" << endl;
		return(NULL);
	}

ma fonction dupliquer dépend de ma classe forme, je l'ai testé elle fonctionne bien sans faute de segmentation (elle rend une copie d'une forme sous forme de pointeur)
en données privé j'ai:

Code :

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

1
2
private:
	 map<string, Forme*> mymap;