Discussion :

[poukill]

Bonjour à tous,

Bien qu'ayant lu la FAQ là dessus, je ne trouve pas tout à fait mon bonheur...

voici ma classe Choc:

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
class Choc {
public:
	Choc (int num_choc); // Constructeur général
	Choc (Choc & choc);
	~Choc();
 
	void setNumeroChoc (int);
	void ajouterCFP(CFP * cfp);
	void ajouterCFP(std::string nom_composant);
	void ajouterCFPBiVisee(std::string nom_composant);
 
private:
	int m_numero_choc;
	std::map<std::string, CFP *> m_tableau_cfp;
	std::vector<std::string> m_cameras_disponibles;

Lorsque dans mon programme principal je lance

Code :

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

delete m_choc;

Ca plante sur mon destructeur : invalid parameters !

Code :

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

1
2
3
4
5
6
7
8
Choc::~Choc()
{
	std::map<std::string, CFP*>::iterator iter;
	for (iter = m_tableau_cfp.begin(); iter != m_tableau_cfp.end(); iter++)
	{
		m_tableau_cfp.erase(iter->first); // ICI
	}
}

Ma destruction n'est pas propre?
Merci de votre aide !

EDIT: je me demande même si je supprime mes pointeurs vers CFP ?

[Laurent Gomila]

Pourquoi ne pas utiliser le erase qui prend un itérateur ? Là ta map va devoir retrouver le noeud à supprimer, alors que toi tu le connais déjà.

Ensuite, après la suppression, l'itérateur est invalide, donc iter++ ne fonctionnera pas.

Tu pourrais aussi utiliser des pointeurs intelligents (voir FAQ).

Et enfin, ce que tu fais ne sert strictement à rien
Le destructeur de std::map s'occupe de supprimer les éléments stockés. Note pour le jour où tu voudras le faire hors d'un destructeur : il faut utiliser la fonction membre clear()
Par contre tu devrais faire un delete sur les éléménts, à moins que ce ne soit pas ta map qui gère leur durée de vie.

[poukill]

Pourquoi t'as toujours les réponses à toutes les questions, même celles que je me suis pas encore posées???

Oui, tu as raison partout. Je vais repartir sur un std::auto_ptr ou un boost::shared_ptr()...

Merci!

[Laurent Gomila]

Je vais repartir sur un std::auto_ptr

Attention : pas de std::auto_ptr dans un conteneur. Explications dans la FAQ.
boost::shared_ptr est très bien par contre. Ou encore mieux : directement un boost::map_ptr (ou un truc dans le genre) au lieu d'un std::map.

[poukill]

Merci pour le tuyau de std::auto_ptr, j'avais oublié !

Tu veux parler de Boost Property Map?

EDIT: non il s'agit en effet de : boost::ptr_map
Trouvé ICI

Merci Laurent !

[poukill]

Juste une petite question: ce ptr_map est une map de shared_ptr?
Là où je veux en venir, c'est comment le ptr_map décide s'il doit supprimer un élément?
Comment est mis à jour le compteur de référence?

Code :

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

1
2
3
boost::ptr_map<std::string, Classe> Map;
[...]
Classe * cl = Map["toto"];

Ce code va t-il incrémenter le compteur de référence d'un shared_ptr ?

Je me pose plein de questions, j'avoue...

[Laurent Gomila]

En fait, c'est le conteneur qui gère la vie des éléménts. En l'occurence quand tu en supprimes un, il va être détruit avec delete.

Ensuite l'interface est faite pour que tu manipules tes éléments directement, pas des pointeurs. Donc en fait tu ne t'occupes à aucun moment des références ou de la validité des pointeurs, c'est comme si tu avais un std::map<std::string, Classe>.

Enfin je ne l'ai jamais utilisé, donc tu devrais plutôt lire la doc pour avoir des explications plus précises