proprete du code avec define

bonjour ,
J'ai ce petit code qui marche comme je lui demande seulement j'ai l'impression que c'est sale.

Code:

---

1 2 3 4 5 6 7 8 9 10 11

int main(int argc,char **argv) { if(argc>1) if (argv[1][0]=='v') #define VERBOSE   #ifdef VERBOSE fprintf(stdout,"Verbose \n"); #endif return EXIT_SUCCESS; }

---

c'est pour eviter de mettre les printfs sous commentaire en version production et enlever les commentaires quand je dois retester.
je me contente actuellement de lancer le programme comme suit

Citation:

---

./test v arg2 arg3 ...

---

Merci

Citation:

---

Envoyé par lilington

bonjour ,
J'ai ce petit code qui marche comme je lui demande seulement j'ai l'impression que c'est sale.

---

C'est d'autant plus sale que c'est faux :aie:

le define est effectué à la pré-compilation.. Il ne tient donc pas compte du if, qui lui est effectué à l'exécutuion...

2 solutions donc, une souple à l'exécution, une dépendante de la compilation :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

static int VERBOSE = 0 ;   int main(int argc,char **argv) { if(argc>1) if (argv[1][0]=='v') VERBOSE = 1 ;   if ( VERBOSE == 1 ) fprintf(stdout,"Verbose \n");   return EXIT_SUCCESS; }

---

Sinon

Code:

---

1 2 3 4 5 6 7 8 9

i nt main(int argc,char **argv) { #ifdef VERBOSE fprintf(stdout,"Verbose \n"); #endif   return EXIT_SUCCESS; }

---

mais alors il faut compiler avec le flag -DVERBOSE ou sans, et l'affichage ne dépendra pas de paramètre de lancement.

:D

oui pour sale, oui pour precompilation mais currieusement ca marche quand meme quand je lance le truc.
je sais pas pourquoi mais ca marche.
ptre un probleme avec gcc je suis sous fedora 16 / gcc version 4.6.2 20111027 (Red Hat 4.6.2-1) (GCC)

bref je retiens ta 2eme car la premiere me plait pas trop.
sinon j'ai pense a une 3eme methode pour eviter les 'if'

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

void verbose(const char *str,int truc) { if(truc==1) fprintf(stdout,"%s\n",str); } int main(int argc,char **argv) { int flag_v=0; if(argc>1 &&argv[1][0]=='v') flag_v=1; verbose("je suis verbeux",flag_v); .... verbose("hmmm je suis tres verbeux",flag_v); ... }

---

bon j'aurait du reflechir avant de poster. je mets resolut mais dis moi pourquoi mon premier exemple a marche? j'arrive pas a comprendre pourquoi.

Ton code est équivalent à :

Code:

---

1 2 3 4 5 6 7

int main(int argc,char **argv) { if(argc>1) if (argv[1][0]=='v') fprintf(stdout,"Verbose \n"); return EXIT_SUCCESS; }

---

Citation:

---

Envoyé par diogene

Ton code est équivalent à :

---

a la difference que je ne tape qu'une seule fois le if()
ca me fait economiser des lignes de code. avec 50 apparition fois if()printf ca devient long quoi...

je pense que le mien dois quand meme etre legerement moins rapide puisqu'il y a appelle de fonction supplementaire.

Non ton code est totalement faux et tombe en marche par miracle.

Les define et ifdef sont des oéprations preprocessor, avant même la compilation (je ne parle même pas encore d'exécution).

Le preprocessor remplace au fur et à mesure qu'il les rencontre les instructions qu'il sait gérer.

Ajoute une autre ligne de printf et lance ton programme sans le paramètre v, la ligne s'affichera..

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

int main(int argc,char **argv) { if(argc>1) if (argv[1][0]=='v') #define VERBOSE   #ifdef VERBOSE fprintf(stdout,"Verbose \n"); #endif #ifdef VERBOSE fprintf(stdout, "Oh ca ne marche plus !\n"); #endif return EXIT_SUCCESS; }

---

parce que ce code est équivalent à

Code:

---

1 2 3 4 5 6 7 8

int main(int argc,char **argv) { if(argc>1) if (argv[1][0]=='v') fprintf(stdout,"Verbose \n"); fprintf(stdout, "Oh ca ne marche plus !\n"); return EXIT_SUCCESS; }

---

oui, merci pour l'info je compris
en faite mon code marchait par hazar et apres compilation il est:

Code:

---

1 2 3 4 5 6 7

int main(int argc,char **argv) { if(argc>1) if (argv[1][0]=='v') fprintf(stdout,"Verbose \n"); return EXIT_SUCCESS; }

---

donc quand je lance avec V il fait le printf sinon il va directement return. Meci