Discussion :

[minitchoup]

voila ce que j'espère faire :

Code :

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

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

[krieg]

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

[3DArchi]

Bonjour et bienvenu,

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 :

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

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

[Médinoc]

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

[minitchoup]

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

[Matthieu Brucher]

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.

[Médinoc]

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

[minitchoup]

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

[Matthieu Brucher]

Tu fais des list<PointeurIntelligent*> avec des :

Code :

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

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

[minitchoup]

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

c'est La solution.

[Davidbrcz]

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.

[Alp]

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.

[loufoque]

Sinon tu peux aussi utiliser boost::any.