Discussion :

[salseropom]

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 :

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

1
2
3
4
5
6
 
#define DELETE(p) \
do{               \
  delete p;       \
  p = NULL;       \
} while 0

avec le code suivant

Code :

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

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 :

error C2059: syntax error : 'constant'

D'où provient mon erreur ?

Merci d'avance

[salseropom]

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

Code :

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

1
2
 
while(0)

et non

Code :

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

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) ?

[Davidbrcz]

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) ?

[Arzar]

Donc salseropom, tu préferes écrire :

Code :

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

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 :

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

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" ?

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 :

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

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 :

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

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

[salseropom]

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