destruction d'un void *, je voudrais que le destructeur soit appelé si il existe

voila ce que j'espère faire :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

void *G_p = 0;   void enregistrer(void *p) { G_p = p; }   void detruire() { delete G_p; }   enregistrer(new A()); detruire(); //--> ~A() n'est pas appelé (normal)

---

comment puis-je faire pour que le destructeur soit appelé (si il existe)?

je pourrais écrire aussi :
enregistrer(new int);
enregistrer(new CString);
etc...

Salut,
tu peux t'en sortir avec des templates. si le type est bien defini delete appellera le destructeur.

Bonjour et bienvenu,

Citation:

---

Envoyé par minitchoup

comment puis-je faire pour que le destructeur soit appelé (si il existe)?

---

Tu n'as pas d'autre choix que typer correctement ton objet.

Pour rester dans ce que tu as fait :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

void *G_p = 0;   void enregistrer(void *p) { G_p = p; } template<typename T> void detruire() { delete reinterpret_cast<T*>(G_p); }   enregistrer(new A()); detruire<A>(); //--> ~A()

---

Cependant, ça ne marche pas pour un tableau !

Cependant, le bout de code que tu montre est très hasardeux : l'utilisation de variable globale n'est pas recommandée; dans ta fonction enregistrer, tu ne vérifie pas si ton pointeur n'est pas déjà alloué; l'utilisation de void* n'est pas recommandée (il vaut mieux rester dans du fortement typé); si tu oublies d'appeler detruire, ton pointeur n'est pas libéré.

Typiquement, pour remédier à ce problème, je restreindrais l'enregistrent à des classes héritant d'une classe parente, laquelle classe parente possèderait un destructeur virtuel...

merci pour vos réponses.

en fait
1/ je ne peux rien présupposer sur le type de pointeur qui est passé à 'enregistrer'.
2/ ici j'ai simplifié le pb en stockant le pointeur dans une variable globale, mais en réalité c'est dans une liste qu'il sera stocké (list<void *>)

en fait je crois que c'est sans solution... :(

Citation:

---

Envoyé par minitchoup

en fait je crois que c'est sans solution... :(

---

Plus ou moins. Si tu réécris un pointeur intelligent dérivant d'une classe parente (en gros un boost_shared_ptr<> mais avec l'héritage, tu peux t'affranchir du <>), ça devrait pouvoir fonctionner.

Hé oui: Tu ne peux rien faire de dynamique avec un simple void*.

peux t'on trouver une solution en mettant en œuvre RTTI et en stockant éventuellement une autre information en plus du pointeur (son type)?

Tu fais des list<PointeurIntelligent*> avec des :

Code:

---

1 2 3 4 5 6 7 8 9 10

class PointeurIntelligent { virtual ~PointeurIntelligent(){} };   template<class Type> class MonPointeurIntelligent : public PointeurIntelligent { /* Recopie de boost_shared_ptr si tu veux*/ };

---

A la fin, tu delete ton pointeur qui se chargera de faire un delete correct sur tes données.

Le RTTI ne fonctionne pas sur du void*.

merci à tous et plus particulièrement à Matthieu Brucher.

c'est La solution.

J'ajoute aussi que le résultat textuel du RTTI n'est pas garanti par la norme, il peut varier d'un compilateur à l'autre, ce qui oblige à comparer les objets avec des objets de référence dont on est sûr du type.

Citation:

---

Envoyé par Médinoc

Hé oui: Tu ne peux rien faire de dynamique avec un simple void*.

---

Plus généralement, sauf dans des cas très extrêmes, on ne fait pas réellement de C++ quand on utilise void*.

On résoud toujours les problèmes soit avec des templates, soit avec une bonne hiérarchie de classes.

Sinon tu peux aussi utiliser boost::any.