suppression d'un pointeur

Bonjour, après chaque delete je désire mettre mon pointeur à NULL. J'ai chopé sous http://cpp.developpez.com/faq/cpp/?p...elete_sequence (comment détruire les pointeurs d'un conteneur ?) une fonction qui marche très bien. Mais je trouve que c'est un peu "lourd" comme code. Alors, je me suis fait la macro suivante :

Code:

---

1 2 3 4 5 6

#define DELETE(p) \ do{ \   delete p; \   p = NULL; \ } while 0

---

avec le code suivant

Code:

---

1 2 3 4 5 6 7 8

vector<DataExpRodage2000 *> dataexp; for(size_t i = 0 ; i < dataexp.size() ; i++) { /* ici je crée mon dataexp[i] */ }   for(size_t i = 0 ; i < dataexp.size() ; i++) DELETE(dataexp[i]);

---

mais à cette dernière ligne j'ai une erreur à la compilation :

Citation:

---

error C2059: syntax error : 'constant'

---

D'où provient mon erreur ?

Merci d'avance

Oups, j'ai trouvé mon erreur : dans ma macro, c'est

Code:

---

1 2

while(0)

---

et non

Code:

---

1 2

while 0

---

le nul...

ceci dit, quel est donc la différence entre ma macro "toute simple" et le code donné dans la faq (voir mon post précédent) ?

Quel est l'intérêt de faire un do{}while(0) ? en passant par une macro ?

Pourquoi ne pas passer par une fonction template (comme dans la FAQ) ?

Donc salseropom, tu préferes écrire :

Code:

---

1 2 3

for(size_t i = 0 ; i < dataexp.size() ; i++) DELETE(dataexp[i]); // macro toute pourrie, impossible à débugger

---

au lieu de :

Code:

---

1 2 3

for(size_t i = 0 ; i < dataexp.size() ; i++) Delete(dataexp[i]); // foncteur type-safe, débuggable.

---

Où est cette supposée "lourdeur" ? :koi:

Edit : Si c'est le foncteur qui te pertubes, il me semble qu'il est aussi possible d'écrire Delete sous forme de fonction libre :

Code:

---

1 2 3 4 5 6

template <class T> void Delete(T*& p) { delete p; p = NULL; }

---

Edit 2 : En fait, le foncteur de la FAQ est un peu plus agréable à utiliser avec les for_each:

Code:

---

1 2 3 4

std::vector<int> v; //... for_each(v.begin(), v.end(), Delete()); // foncteur de la FAQ for_each(v.begin(), v.end(), Delete2<int>); // fonction libre

---

Je viens tout juste de me mettre au C++ (j'ai fait pas mal de C avant). J'ai donc mes "réflexes" macro du C...

Je n'ai pas pensé à l'aspect débuggable / non debuggable..

Je vais méditer là dessus.

Merci pour vos réponses