Discussion :

[buzzkaido]

Bonjour,

J'ai créé une classe d'objet.

Ces objets appartiennent à un conteneur.

Celui-ci ne sait pas forcement à l'avance si un de ses objet est déjà créé. Par exemple, si on demande d'ouvrir un objet, le conteneur verifie si il existe :

- si il existe, il l'ouvre
- sinon il le crée puis il l'ouvre

Donc pour savoir si un objet est déjà créé, lors de la création du conteneur, j'initialise la liste de pointeurs vers ses objets à NULL, comme ça je peux tester :

Code :

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

1
2
3
4
5
6
7
 
ouvrirObjet(*pObjet)
{
    if (pObjet == NULL)
        pObjet = new cObjet();
    pObjet->ouvrir();
}

Sauf que lorsque l'on detruit un objet avec delete, le pointeur vers cet objet ne revient pas à NULL...

Donc ma question : est-il possible de mettre ce pointeur à NULL à l'intérieur du destructeur ?

Un truc du genre this = NULL; mais en plus fin, puisque ça c'est interdit....

Merci !

[Laurent Gomila]

Donc ma question : est-il possible de mettre ce pointeur à NULL à l'intérieur du destructeur ?

Non, mais rien ne t'empêche de le faire juste après le delete.

[buzzkaido]

Ah !

Evidemment, j'oubliais de preciser, le delete est automatique...

J'ai une procedure fermerObjet() qui peut etre appelée depuis l'objet lui-meme et qui à la fin fait execute :

delete this;

Donc lors de l'execution du destructeur, je ne connait pas le pointeur vers l'objet.

Du coup je pensais à une astuce du genre :

- à la construction, stocker un pointeur vers le pointeur de l'objet (!!!!)
- à la destruction, mettre la valeur pointée par ce pointeur à NULL

C'est un peu tordu, et ça n'a pas vraiement marché (je m'y suis peut-etre mal pris)

[Laurent Gomila]

Ca devient un peu bourrin là

Tu pourrais détailler un peu plus ce que tu cherches à faire ? Je suis sûr qu'on peut trouver une solution plus élégante.

[buzzkaido]

Ouais, je vais essayer d'etre plus clair :

J'ai un conteneur, qui contient 4 objets (par exemple)

du genre :

Code :

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

1
2
3
4
5
6
7
cConteneur
{
    cObjet    *pObjet1;
    cObjet    *pObjet2;
    cObjet    *pObjet3;
    cObjet    *pObjet4;
}

Le constructeur : (aucun objet n'existe)

Code :

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

1
2
3
4
5
6
7
8
 
cConteneur::cConteneur()
{
    pObjet1 = NULL;
    pObjet2 = NULL;
    pObjet3 = NULL;
    pObjet4 = NULL;
}

Le destructeur :

Code :

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

1
2
3
4
5
6
7
cConteneur::~cConteneur()
{
    if (pObjet1  != NULL) delete pObjet1;
    if (pObjet2  != NULL) delete pObjet2;
    if (pObjet3  != NULL) delete pObjet3;
    if (pObjet4  != NULL) delete pObjet4;
}

Ouvrir un objet :

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
16
cConteneur::ouvrirObjet(uneCondition)
{
    switch (condition)
    {
       case condition_1 :
           if (pObjet1 == NULL) pObjet1 = new cObjet();
           pObjet1->ouvrir();
           break;
       case condition_2 :
           if (pObjet2 == NULL) pObjet2 = new cObjet();
           pObjet2->ouvrir();
           break;
       etc
       ......
    }  
}

Et dans cObjet : (cette méthode est appelée sous certaines conditions : fin du traitement...)

Code :

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

1
2
3
4
5
cObjet::fermer()
{
    fermerDesTrucs();
    delete this;
}

Donc mon probleme, c'est que si le conteneur ouvre un objet et que celui-ci s'est fermé (+ détruit) tout seul, le conteneur n'est pas au courant qu'il doit le ré-créer avant de le ré-ouvrir.....

Et pour d'autres raisons, je voudrait éviter de stocker un pointeur vers le conteneur dans chaque objet (les objets peuvent changer de conteneur etc...)

Donc mon vrai probleme, c'est de tester l'existence d'un objet via son pointeur....

[Laurent Gomila]

Tu as une architecture pas facile à gérer, où la responsabilité des ressources est partagée entre le conteneur et la ressource elle-même. Un truc qui aiderait bien serait les pointeurs intelligents à base de comptage de référence, ça éviterait déjà d'avoir à gérer des dépendances partout. Avec boost::weak_ptr je crois même que tu peux être notifié lorsque le boost::shared_ptr lié est détruit, ce qui pourrait convenir dans ton cas.

Bien sûr la solution la plus simple serait qu'une seule classe ait la responsabilité entière des ressources, mais apparemment ce n'est pas possible dans ton cas.

PS : delete NULL est tout à fait valide, pas besoin de tester le pointeur.

[Startux]

pourquoi tu fais pas une fonction qui ferai à peu près ça :

Code :

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

1
2
3
4
5
6
 
CContainer::liberer(*Objet)
{
       Objet->fermer();
       Objet = NULL;
}

non ?


- Hé dis t'as vu Travail quelque part ?
- Non, moi je suis en vacances...

[Ulmo]

Code :

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

1
2
3
4
5
cObjet::fermer()
{
    fermerDesTrucs();
    delete this;
}

Ca me parait vraiment bizzare, un objet qui appelle son propre destructeur...
En particulier, l'objet existe encore juste après puisqu'on est dans une de ses méthodes.