Discussion :

[Arnaud-h]

Comment peut'on supprimer proprement un objet qui est pointé par deux pointeurs?
En fait je renplit une Tlist avec des objet crée dynamiquement (avec des "new") et en quittant mon programme détruit tous les éléments de la liste, mais builder me dit que la mémoire n'est pas totalement libérée et m'indique l'endroit ou je remplit ma liste comme source de l'erreur, je ne comprends plus rien svp aidez moi!

[Luc Hermitte]

J'aurais bien une solution propre et robuste, mais elle impliquerait de se passer de la VCL au profit du C++
=> std::list<boost::shared_ptr<OBJ_TYPE> >
Autrement, l'objet pointé doit être détruit une et une seule fois.

[Arnaud-h]

Justement c'est bien ce que je fais mais builder me dit que je ne libére pas toute la mémoire! Mais bon maitenant que je suis sûr qu'il ne faut supprimer qu'une seul fois l'objet pointé je vais vérifier mon code!
Merci

[henderson]

Salut !

Peut-être que ceci pourrait convenir :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
//tant que la liste n'est pas vide
while(Liste->Count != 0)
    {
//prendre le premier élément de la liste
    void *P = Liste->First(); //j'ai mis void*...voir ce qui convient...
//le retirer de la liste
    Liste->Delete(Liste->IndexOf(P));
//Tester si l'élément est dupliqué
    if(Liste->IndexOf(P) == -1)
        {
//S'il ne l'est pas alors détruire
        delete P;
        }
    }

A plus !

[Freeze]

je fais comme ca :

Code :

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

1
2
3
4
 
for (int i=0;i<Liste->Count;i++)
   delete (void *)Liste->Items[i];
delete Liste;

même remarque pour le void * à toi de voir ce que tu as mis dedans ...

[DjmSoftware]

bonjour,
dans ton code

Code :

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

1
2
 
delete (void *)Liste->Items[i];

fonctionne bien si les pointeurs sont de type void
dans le cas contraire la libération mémoire ne sera pas complète
c'est le probl'me avec un Tlist car c'est a toi de savoir ce que contient ta liste
si par exemple c'est des TButtons qui sont stockés dans ta liste le code correct pour la destruction est le suivant

Code :

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

1
2
3
4
 
for (int n=0;n<Liste->Count;n++)
 delete (TButton*)Liste->Items[i]; 
delete Liste

cordialement

[Arnaud-h]

Merci bcp pour vôtre praticipation active! je vais essayer comme vous faite, moi je fais comme ca, mais je sais pas si c'est la bonne solution:

Code :

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

1
2
3
4
 
for(int i = (List->Count - 1); i >= 0; --i)
     List->Delete(i);
delete List;

Merci encore!

[Freeze]

for(int i = (List->Count - 1); i >= 0; --i)
List->Delete(i);
delete List;

et bien justement non !!! quand tu fais cela tu ne supprimes l'allocation en mémoire de l'objet que tu as rangé dans ta TList ...

[Arnaud-h]

Oui je viens de m'en rendre compte! En plus c'est bien spécifié dans l'aide de Builder ! Désolé! Mais maitenant j'ai compris et tout fonctionne c'est super! Merci pour toute vôtre, ce forum est génial bye

[henderson]

Salut !

L'exemple que j'ai donné est vrai, par exemple, pour une liste de bitmaps, en supposant
qu'un bitmap puisse être dessiné plusieurs fois et ce grace à cette liste, mais ce qui
serait une conception relativement maladroite. Dans ce cas :

Code :

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

1
2
3
Graphics::TBitmap *B = (Graphics::TBitmap *)List->First();
...
delete B;

Par contre, si les objets descendent de TComponent, il existe une solution très simple
qui consiste à affecter aux objets crées un propriétaire commun du type TComponent
En effet, la destruction d'un objet entraine celle des objets qu'il détient.

Code :

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

1
2
TComponent *LeProprietaire;
TList *List = new TList;

Code :

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

1
2
//créer un objet qui va servir de propriétaire
LeProprietaire = new TComponent(this); //this ou ...

Code :

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

1
2
3
4
5
//création des objets
TPanel *P = new TPanel(LeProprietaire);
P->Parent = this;//this ou là où il s'affiche
P->SetBounds(10,20, 60, 24);
List->Add(P);

Code :

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

1
2
3
//destruction des objets
delete LeProprietaire; // tous les objets détenus sont détruits avec lui !
List->Clear();

A plus !