Discussion :

[lilington]

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

Code :

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

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

./test v arg2 arg3 ...

Merci

[souviron34]

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

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 :

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

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 :

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

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.

[lilington]

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 :

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

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.

[diogene]

Ton code est équivalent à :

Code :

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

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;
}

[lilington]

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.

[Bousk]

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 :

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

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 :

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

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;
}

[lilington]

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

Code :

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

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