Discussion :

[antoine2641]

Bonjour,

Je cherche le moyen de supprimer les éléments d'une std::list et de libérer la mémoire associée.
J'ai testé plusieurs moyens mais la mémoire n'est jamais libérée.
La structure utilisée contient des membres de type WCHAR de taille fixe.
Je travaille sur Windows CE 5.0

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
17
 
MASTRUCTURE * test3 ;
std::list<MASTRUCTURE *> test2;
 
for(int i = 0 ; i!= 1000 ;++i)
{
	test3 = new MASTRUCTURE ();
	test2.push_back(test3);				
}
 
 
std::list<MASTRUCTURE *>::iterator itDeb1 = test2.begin();
while (itDeb1 != test2.end ())
{
    delete (*itDeb1);
    itDeb1 = test2.erase (itDeb1);
}

Merci d'avance,
Antoine

[Bousk]

Salut,

faut simplement la parcourir et faire un delete sur chaque élément

Code :

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

while (!list.empty()) { delete list.back(); list.pop_back(); }

Après bien sur faut que le detructeur libère correctement son objet.

[Neckara]

Bonjour,

Ton code originel devrait normalement fonctionner, il faudrait nous montrer ta structure ainsi que son destructeur.

Mais je te conseille de faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
std::list<MASTRUCTURE *>::iterator it = test2.begin();

for ( ; it != test2.end() ; ++it ) // ou std::for_each
// for(auto & ptr : test2) // en C++11
{
        delete (*it);
        //delete(ptr); //C++11
}

test2.clear();

Pour construire ta liste, tu peux aussi faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
std::list<MASTRUCTURE *> test2(1000, nullptr);

std::list<MASTRUCTURE *>::iterator it = test2.begin();

for ( ; it != test2.end() ; ++it ) // ou std::for_each
// for(auto & ptr : test2) // en C++11
{
        *it = new MASTRUCTURE();
        //ptr = new MASTRUCTURE(); //C++11
}

Je trouve que le code sera plus "clair".

A noter qu'en C++11 on utilisera sûrement une liste de std::shared_ptr ou des std::unique_ptr.

[antoine2641]

Tout d'abord, merci d'avoir répondu.

Salut,faut simplement la parcourir et faire un delete sur chaque élément.

C'est ce que je fais.

Après bien sur faut que le destructeur libère correctement son objet.

Ton code originel devrait normalement fonctionner, il faudrait nous montrer ta structure ainsi que son destructeur.

Un destructeur pour la structure ?
J'ai testé en utilisant des float (sans pointeurs) dans la liste et le résultat est équivalent : la mémoire n'est pas libérée.
Même en utilisant : remove, clear, pop_back, erase, delete..... et j'en passe.

Antoine

[Bousk]

La list est un contenair, elle n'est pas responsable de ce que t'y mets. L'éventuelle mémoire allouée pour les éléments est de ta responsabilité, à fortiori la mémoire allouée par chaque élément.