Discussion :

[neferis]

Bonjour,

je suis tombe sur un bout de code dont je ne comprends pas l'interet :

Code :

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

1
2
 
#define MYFREE(x) do{if((x)) free((x));}while(0)

je m'interroge sur l'utilisation du do while.
apres avoir fait des recherches, j'ai retrouve a plusieurs reprises cet utilisation du "do while" dans la definition d'une macro, mais je n'en comprends pas le sens

pourquoi ne pas simplement ecrire :

Code :

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

1
2
 
#define MYFREE(x) if(x){free(x);}

le resultat serait le meme avec une instruction en moins non?

[Médinoc]

C'est pour avoir une macro que tu peux utiliser "plus" comme une fonction que le simple if (d'où le do-while(0) sans le point-virgule à la fin).

Malheureusement, Visual est trop stupide pour reconnaître ça et donne un warning "condition is constant" quand il tombe dessus...

[Ekinoks]

Elle à l'aire interessente cette macro ! =)
Mais j'ai pas bien compris sont utilitée... :^/

A quoi sert le do-while(0) sans point-virgule ?

[Skyounet]

Elle à l'aire interessente cette macro ! =)
Mais j'ai pas bien compris sont utilitée... :^/

A quoi sert le do-while(0) sans point-virgule ?

Imaginons cette macro

Code :

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

#define ERROR(message) { fprintf(stderr, "%s", message); exit(EXIT_FAILURE); }

Si je l'utilise comme ceci

Code :

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

1
2
if(malloc(...) == NULL)
    ERROR("erreur malloc");

Le code final serait

Code :

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

1
2
3
4
5
if(malloc(...) == NULL)
{
    fprintf(stderr, "%s", message);
    exit(EXIT_FAILURE);
};

Et le ; à la fin de l'accolade peut poser problème c'est pour cela qu'on met le tout dans un do while sans ; à la fin (le ; étant celui à la fin de "l'appel" à la macro).

[Ekinoks]

Ha ok ! je comprend mieux, merci pour ces explications =)

Rusé comme astuce :^p

[Emmanuel Delahaye]

pourquoi ne pas simplement ecrire :

Code :

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

#define MYFREE(x) if(x){free(x);}

le resultat serait le meme avec une instruction en moins non?

Dans ce cas particulier,

Code :

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

#define MYFREE(x) free(x)

est équivalent à la macro initiale et sans risque, ce qui rend cette macro totalement inutile.

Par contre :

Code :

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

#define MYFREE(p) free(p), p = NULL

est utile.