Discussion :

[magicpm7]

Bonjour,

j'utilise une m�thode pour remplir une map d'objets index�s par leurs noms.

La d�claration de cette map est la suivante :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

std::map<CString,monTypeObjet> maMap;

Pour la remplir, je me sers d'une boucle, que voici :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
/*prototype*/
 
monTypeObjet *ptrObjet = new monTypeObjet();
 
/* Autres déclarations, blabla */
 
/*boucle*/
while(maCondition)
{
  ptrObjet -> remplirObjet(unParametreQuiChangeAChaquePassageDansLaBoucle);
  maMap[nomObjet]=*(ptrObjet);
}
 
/* ...*/

J'ai l'impression qu'il y a un probl�me d'allocation (j'en suis m�me sur, car j'obtiens des r�sultats faux) mais je ne vois pas trop comment le r�soudre...

[befalimpertinent]

L� je vois pas trop.
Donnes nous le code de remplirObjet si possible et de monTypeObjet

[magicpm7]

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
13
class monTypeObjet
{
private:
BSTR unAttribut;
int unAutreAttribut;
/*etc.*/
 
public:
monTypeObjet();
~monTypeObjet();
 
void remplirObjet(CString filePath);
};

et

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
void monTypeObjet::remplirObjet(CString filePath)
{
/*Ouverture d'un fichier, lecture dans ce fichier, et remplissage des attributs, du genre: */
 
unAutreAttribut = unTrucLuDansLeFichier;
/*etc.*/
}

Rien de sorcier, en somme...

[r0d]

Deux choses:

1/ Je d�conseille de m�langer MFC et STL (CString c'est la MFC, std::map c'est la STL). la classe std::string remplace avantageusement la CString. (Bon parfois on n'a pas vraiment le choix, j'en conviens.)

2/ La ligne

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

maMap[nomObjet]=*(ptrObjet);

utilise le constructeur par copie de monTypeObjet. Es-tu certain que ce constructeur par copie est correct?

[loufoque]

Les conteneurs standard fonctionnent avec des valeurs, qui en particulier doivent �tre convenablement copiables dans le sens usuel du terme.
Ton type n'a pas l'air de se comporter comme une valeur � cause du membre BSTR, qui n'est qu'un pointeur � priori.

Ensuite tu utilises des pointeurs et de l'allocation dynamique, ce qui semble compl�tement inutile ici.

[befalimpertinent]

Si tu utilises ceci :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

std::map<CString,monTypeObjet> maMap;

Pour la remplir, je me sers d'une boucle, que voici :

[CODE]/*prototype*/

monTypeObjet *ptrObjet = new monTypeObjet();

parceque tu penses ainsi r�utiliser le m�me objet pour �conomiser des instanciations dans ta boucle --> dis toi bien que c'est une tr�s mauvaise id�e.

Il vaut mieux au contraire limit� la port� des variable � la boucle.