Discussion :

[contremaitre]

Salut

Alors j'ai compris qu'on ne pouvait pas le faire, mais j'aimerai trouver un autre moyen.
Je vous explique mon cas.

J'ai une classe CThread de base qui s'ocupe de la création et destruction de threads.
Une classe CExample qui en hérite.

Le destructeur de Cthread attend la fin du thread, qui est en train de tourner dans une fonction de la classe CExample. Il fait un join sur le thread.
J'aimerai pouvoir apeller dans ce destructeur une fonction OnStop(), virtuelle, qui est redéfinie dans CExample et qui permet de demander à mon thread de s'areter.

Comme en C++, l'appel de la fonction virtuelle dans le destructeur de ma classe CThread n'apelle pas celle de la classe CExample, je suis obligé d'ecrire avant chaque delete cexample : cexample.OnStop().

Y a t'il un autre moyen ?

merci

[bruno_pages]

Bonjour,

c'est au destructeur de CExample d'appeler OnStop(), pas à celui de CThread

[contremaitre]

Ok.
J'aurai du y penser, merci.
Le petit inconvenient c'est qu'on doit l'ajouter dans chaque classe qui dérive de CThread, alors que ça aurait été pratique que ça fasse partie de l'implémentation de CThread justement de s'occuper de ça.

[bruno_pages]

le problème c'est que comme vous l'aviez dit cela n'est pas possible car le destructeur de Cthread est appelé après le destructeur des classes filles

il reste une autre solution : ne pas directement appeler delete sur les sous classes mais une autre opération définie sur Cthread et qui fera l'appel d'OnStop puis le delete.

[koala01]

Salut,

il reste une autre solution : ne pas directement appeler delete sur les sous classes mais une autre opération définie sur Cthread et qui fera l'appel d'OnStop puis le delete.

Si j'interprète bien ce que tu propose, cela donnerait quelque chose comme

Code :

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

1
2
3
4
5
6
7
8
9
10
11
class Cthread 
{
    public: 
        virtual void onStop() = 0;
        void destroy()
        {
            onStop();
            delete *this;
        }
        /*...*/
};

Mais il y a alors un problème, du fait de l'appel de delete sur this:

Si tu travailles avec un pointeur sur CThread, tu en arrivera à une situation dans laquelle ton pointeur semble toujours pointer vers une adresse valide, alors que la dite adresse a été invalidée:

Code :

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

1
2
3
4
5
6
7
void foo(CThead * ptr)
{
    /*...*/
    ptr->destroy(); // DANGER 
    /* y a d'autre choses à faire, on "oublie" que ptr a été détruit...*/
    ptr->machinChose(); //BOUM... ptr représente une adresse invalide
}

Si on travaille avec un objet, c'est encore pis: on cours tout droit vers une tentative de double destruction d'un objet:

Code :

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

1
2
3
4
5
6
7
8
9
 
void bar()
{
    /*CExemple hérite de CThread ;-) */
    CExemple mt(/*parametres éventuels */);
    /* Tout va très bien, madame la marquise ...*/
    mt.destroy(); // CRACK : première destruction de l'objet
    /* éventuellement une ou deux choses à faire après */
} // BOUM : destruction automatique de mt... qui a déjà été détruit :-P

[bruno_pages]

Bonjour,

il est bien évident qu'appeler machinchose après destroy est absurde, mais ni plus ni moins qu'après delete

bien-sur cela ne marche pas dans le cas d'une variable auto détruite, mais ni plus ni moins que

Code :

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

1
2
3
4
5
 
{
  CExemple e;
  delete &e
}

cette solution n'était qu'un pi aller (je ne sais pas si cela s'écrit ainsi ), encore une fois la bonne solution est d'appeler OnStop dans le destructeur des sous classes