Déclaration de variable dans une macro

Version imprimable

Bonjour,

Indépendamment de la propreté du code et de l'hérésie de procéder ainsi, soit le code suivant :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 #define DEFINITION_VARIABLE(_type_, _valeur_) { \ _type_ toto; \ toto = (_valeur_) * ((_valeur_) - 1); \ } #define COMPARE_AFFECTE(_type2_, _valeur2_) { \ if (toto > (_valeur2_)) \ *sortie = (_type_2)toto: \ else \ *sortie = (_type_2)0; \ return; \ } void short2int(short entree, int * sortie) { DEFINITION_VARIABLE(short, entree - 2); COMPARE_AFFECTE(int, 125); } void short2long(short entree, long * sortie) { DEFINITION_VARIABLE(short, entree + 3); COMPARE_AFFECTE(long , 3254); } void uchar2short(unsigned char entree, short * sortie) { DEFINITION_VARIABLE(unsigned char, entree); COMPARE_AFFECTE(short , 17); }
etc.

Dans chacune de mes fonctions, je vais avoir pour COMPARE_AFFECTE le message d'erreur "variable toto non déclaré".
J'avoue être surpris puisque je pensais naïvement que le pré-processeur faisait du copier / remplacer de macros avant la compilation.

J'ai raté quelque chose ?

Merci d'avance pour vos réponses.

25/04/2013, 16h31
gangsoleil
Bonjour,

Je pense que le mieux dans ce cas est d'aller voir quelle est le code apres passage du preprocesseur.

Code:

gcc -E mon_fichier.c > mon_fichier_preprocess.c

[Edit]
J'avais un peu de temps, j'ai fait le test. Tu as simplement un probleme de declaration de bloc : dans DEFINITION_VARIABLE, tu declares _toto_ dans un bloc ; donc a la fin du bloc, la variable toto n'existe plus.

[Edit2]
Tu as aussi un soucis dans ton code tu as _type2_ et _type_2 :
Code:

1 2 3 #define COMPARE_AFFECTE(_type2_, _valeur2_) { \ if (toto > (_valeur2_)) \ *sortie = (_type_2)toto: \
26/04/2013, 00h28
olivier_u

[Résolu] Déclaration de variable dans une macro
Citation:

Envoyé par gangsoleil

Bonjour,

Je pense que le mieux dans ce cas est d'aller voir quelle est le code apres passage du preprocesseur.

Code:

gcc -E mon_fichier.c > mon_fichier_preprocess.c

[Edit]
J'avais un peu de temps, j'ai fait le test. Tu as simplement un probleme de declaration de bloc : dans DEFINITION_VARIABLE, tu declares _toto_ dans un bloc ; donc a la fin du bloc, la variable toto n'existe plus.

Oui, j'ai fini par comprendre tout à l'heure...
On m'avait toujours appris de déclarer une macro avec { \ à la fin de la première ligne et } à la toute fin.
En fait, si on retire les {}, ça marche.
Citation:
Envoyé par gangsoleil

[Edit2]
Tu as aussi un soucis dans ton code tu as _type2_ et _type_2 :

Code:

1 2 3 #define COMPARE_AFFECTE(_type2_, _valeur2_) { \ if (toto > (_valeur2_)) \ *sortie = (_type_2)toto: \
En fait, j'ai réécris un code d'exemple, je n'avais pas accès au code de base. Donc pas de souci de ce côté.
Merci pour votre aide néanmoins.
26/04/2013, 06h29
Neckara
Bonjour,

Citation:

Envoyé par olivier_u

On m'avait toujours appris de déclarer une macro avec { \ à la fin de la première ligne et } à la toute fin.
En fait, si on retire les {}, ça marche.

Comme les macro ne sont au final que du texte qui sera copié/généré dans le code par le pré-processeur, il faut vraiment se méfier des priorités :
Code:

1 2 #define TOTO 5+4 TOTO*5
Donnera : 5+4*5 = 25 au lieu de (5+4)*5 = 45

C'est pour cela qu'on conseille de mettre des parenthèses :
Code:

1 2 #define TOTO (5+4) TOTO*5
Donnera : (5+4)*5 = 45

Le fait de mettre des accolades me semble un peu ridicule car à part provoquer des effets de bords, ta macro ne servira à rien.
Dénonces-les, qui t'as dit de mettre systématiquement des accolades ? :?
26/04/2013, 08h08
olivier_u

Citation:

Envoyé par Neckara

Bonjour,
Le fait de mettre des accolades me semble un peu ridicule car à part provoquer des effets de bords, ta macro ne servira à rien.
Dénonces-les, qui t'as dit de mettre systématiquement des accolades ? :?

Celui qui nous initié en C, en école.
Enfin, bon, c'était un peu bizarre aussi. Il considérait qu'à part 0 ou 1, aucune autre constante ne méritait d'être dans le code sans être déclarée en #define.
Quant au macro, ils nous a dit ça mais nous a dit aussi de ne jamais nous en servir :lol:. Le C discipliné, qu'il a appelé ça. Il avait même écrit un compilateur spécial et qu'on devait utiliser pour qu'on respecte ses règles.

De toute façon, c'est en industrie que j'ai vraiment appris le C et ses subtilités plus tard. L'inconvénient, c'est qu'on nous a toujours imposé l'ISO C89 ce qui fait que je suis un codeur d'il y a 20 ans :lol:
26/04/2013, 09h04
Metalman

Le préproc' n'est jamais assez abordé, comme d'habitude, et toujours car il n'y a pas assez de temps pour faire du C et du préproc'...

Le C ANSI/C89 a beau être "vieux", il n'en reste pas moins une norme tout à fait potable... excepté pour quelques fonctions (dommage...).
Enfin bref : écrire du C89 n'est pas négatif du tout, car c'est l'une des rares normes "réellement" disponible sur quasiment toutes les architectures.
26/04/2013, 10h18
gangsoleil

Citation:

Envoyé par olivier_u

Il considérait qu'à part 0 ou 1, aucune autre constante ne méritait d'être dans le code sans être déclarée en #define.

Et c'est plutot une bonne chose.

Lorsque tu ecris ton code, tu sais pourquoi tu mets telle valeur de constante ici, et telle autre la.
Mais lorsque tu reprends ton code 3 mois plus tard, tu te demandes vite pourquoi tu fais for (i=0 ; i < 42 ; i++), alors que le truc auquel tu accedes a une taille variable d'au moins 50.
Une constante nommee "NOMBRE_MAX_MACHINS" te dira que visiblement, dans ta variable, tu peux avoir au plus 42 machins (le reste etant des bidules)

D'ailleurs, le fait de rencontrer des constantes en chiffres dans du code s'appelle parfois des "nombres magiques".

Citation:

L'inconvénient, c'est qu'on nous a toujours imposé l'ISO C89 ce qui fait que je suis un codeur d'il y a 20 ans :lol:

C'est pas tres grave ca.

Le C99 apporte quelques bonnes choses, mais le fait que beaucoup de compilateurs ne l'aient pas implementes dans son integralite pendant tres longtemps etait aussi un frein.
Et puis si tu te retrouves a developper pour plusieurs cibles (ne serait-ce que plusieurs OS dont certains vieux), tu vas voir que l'avantage du C89 est qu'il compile partout.
26/04/2013, 12h35
Neckara
Citation:

Envoyé par gangsoleil

Et c'est plutot une bonne chose.

Lorsque tu ecris ton code, tu sais pourquoi tu mets telle valeur de constante ici, et telle autre la.
Mais lorsque tu reprends ton code 3 mois plus tard, tu te demandes vite pourquoi tu fais for (i=0 ; i < 42 ; i++), alors que le truc auquel tu accedes a une taille variable d'au moins 50.
Une constante nommee "NOMBRE_MAX_MACHINS" te dira que visiblement, dans ta variable, tu peux avoir au plus 42 machins (le reste etant des bidules)

D'ailleurs, le fait de rencontrer des constantes en chiffres dans du code s'appelle parfois des "nombres magiques".

Il faut aussi nuancer, pour certains cas utiliser des #define est une abhération :

Code:

AireTriangleRectangle = hauteur*longueur/2

Je pense que ce serait horrible d'utiliser un #define ici, sinon on se retrouvera assez vite avec des :
Code:

1 2 #define DEUX 2 #define SEIZE_VIRGULE_QUATRE 16.4
:aie:

Donc oui, il faut éviter les nombres magiques mais se les interdire en toute circonstance est une abhération.
26/04/2013, 13h55
kwariz
Citation:

Envoyé par Neckara

Bonjour,
...
Le fait de mettre des accolades me semble un peu ridicule car à part provoquer des effets de bords, ta macro ne servira à rien.
Dénonces-les, qui t'as dit de mettre systématiquement des accolades ? :?

Bonjour,

les accolades ont l'avantages de pouvoir créer des blocs de code plus ou moins indépendants de leur localisation, les variables crées locales à ce bloc etc ...
Une sorte de manière de créer des fonctions inlines avec pleins de désavantages en fait.
Pour être encore plus proche de la syntaxe des fonctions on peut imaginer :
Code:

1 2 3 #define mamacro(x,y) do {\ le corps de la "fonction" \ } while(0)
Le pseudo avantage d'entourer le code par un do {} while(0) et de pouvoir ensuite utiliser la macro comme une "fonction retournant void" «avec le point virgule à la fin» : mamacro(1,x); :)

Depuis la disponibilité des fonctions inline cela n'a plus grand intérêt en pratique quotidienne.