Discussion :

[italiasky]

Bonsoir,

J'aimerais utiliser une boucle for_each pour supprimer les éléments d'un conteneur avant de supprimer ce conteneur mais le delete dans le for_each, il n'aime pas trop... lol

Voici mon code :

Code :

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

1
2
3
4
5
6
7
 
vector<champ*> champs;
 
vector<champ*>::iterator it = champs.begin();
vector<champ*>::iterator it_end = champs.end();
 
for_each(it, it_end, delete);

Erreur de syntaxe, de parenthèse... enfin il doit manquer un truc, j'ai essayé avec delete it ou des trucs de ce genre, mais ca passe pas... c'est possible vous pensez ? ou je parcours avec un for tout simplement...

Merci
++

[Davidbrcz]

Il faut passer soit par un foncteur soit pas une fonction. Mais tu ne peux pas faire comme ca directement car delete n'est pas une fonction, or c'est ce qu'attend for_each. Avec les expressions lamba en C++0X, ca serra beaucoup plus léger à écrire

[Jean-Marc.Bourguet]

delete n'est pas callable...

Code :

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

1
2
3
4
5
6
7
8
9
10
 
void champDeleter(champ* ptr)
{
   delete ptr;
}
...
 
   foreach(it, it_end, champDeleter);
 
...

[italiasky]

D'accord donc pas trop d'intéret d'utiliser un for_each pour supprimer des objets, puisque qu'il faut passer par une "surcouche" pour appeler le delete...

autant faire un simple for :

Code :

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

1
2
3
 
for(it, it != it_end ; it++)
    delete it;

??

Merci
Bonne soirée

[loufoque]

Tu es conscient qu'un tel code ne sera pas exception-safe et peut avoir des fuites mémoires ?

[3DArchi]

Tu es conscient qu'un tel code ne sera pas exception-safe et peut avoir des fuites mémoires ?

++
Déjà tu devrais partir d'un vector<shared_ptr<champ> >. Ca se passera beaucoup mieux!

[Alp]

Mais sinon, il y a toujours checked_delete qui est fait pour ça

[Arzar]

Tu es conscient qu'un tel code ne sera pas exception-safe et peut avoir des fuites mémoires ?

Ben pourquoi ?

Le code proposée par italiasky et celui préconisé par la faq fait strictement la même chose non?

Code :

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

1
2
3
 
for(it = container.begin(), it != container.end() ; it++)
    delete (*it);

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
// FAQ
struct Delete 
{ 
   template <class T> void operator ()(T*& p) const 
   { 
      delete p;
      p = NULL;
   } 
}; 
 
...
   std::for_each(container.begin(), container.end(), Delete()); 
...

[italiasky]

Re,

ouais enfin moi j'avais oublié le (* ) ce qui est complètement différent du coup...

Mais pour le code de la FAQ, j'arrive pas bien voir un truc...

Etant donné que c'est un vector de pointeurs, l'it pointe sur chaque cellule du vector et le Delete() s'applique sur le it et non sur le (*it) non ?
Ca va pas du coup ?
Ou alors, au Delete, ce n'est pas l'it qui est transmis ?

Bref, si quelqu'un pourrait m'expliquer comment fonctionne ce code de la FAQ avec le foncteur...

En gros, je ne comprends pas très bien cette ligne je crois bien : le (T*& p) ?

Code :

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

template <class T> void operator ()(T*& p) const

Merci
++