Problème d'allocation dans ce code ?

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:

---

std::map<CString,monTypeObjet> maMap;

---

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

Code:

---

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

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

Code:

---

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:

---

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

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:

---

maMap[nomObjet]=*(ptrObjet);

---

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

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.

Si tu utilises ceci :

Citation:

---

Envoyé par magicpm7

Code:

---

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.

Médinoc est de retour de vacances, et pense que ROD et Loufoque ont mis le doigt sur le problème :
monTypeObjet n'a pas de constructeur de copie ni d'opérateur d'affectation personalisés, donc la BSTR n'est pas gérée correctement (une bête copie de son pointeur alors qu'il n'y a pas de comptage de référence) --> Crash.

Donc: Définir un constructeur de copie et un opérateur d'affectation corrects.