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