Discussion :

[Clad3]

Bonjour, j'ai une petite question :
Sous gcc en mode -Wall il me reporte deux fois

unused varaible "..."

J'ai regardé mon code, il s'agit deux fois du même type de code :

Code :

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

1
2
 
int Tmp = MaFonctionQuiRetourneUnInt() ;

Le compilateur rale et ne rale plus si je fait simplement :

Code :

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

1
2
3
 
int Tmp = 0 ;
Tmp = MaFonctionQuiRetourneUnInt();

Si ca viens du fait que le compilateur me singale simplement que je n'utilise pas cette variable dans mon programme, est-il préférable d'utiliser une syntaxe de ce type :

Code :

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

1
2
 
MaFonctionQuiRetourneUnInt() ;

Personnelement je trouve ca un peu bête de cacher e type de retour, ca doit porter à confusion dans un programme de voir les 2 portions de code ci-dessus l'une à coté de l'autre par exemple, on se demande ce que retourne la fonction .

Petite explication : Ma fonction prend en paramètre un flag (true/false) qui lui indique si le code de retour est important ou pas. J'ai fait cela dans le simple but de factoriser mon code, ayant deux fonctions TRES sensiblement équivalente à 1-2 lignes près ...

[gege2061]

Bonjour,

est-il préférable d'utiliser une syntaxe de ce type :

Oui ça règle le problème, sauf sous certains compilateur (tel que celui livré avec PellesC) où tu obtiens un warning, dans ce cas tu peux faire :

Code :

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

(void)MaFonctionQuiRetourneUnInt() ;

[Thierry Chappuis]

Bonjour,


Oui ça règle le problème, sauf sous certains compilateur (tel que celui livré avec PellC) où tu obtiens un warning, dans ce cas tu peux faire :

Code :

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

(void)MaFonctionQuiRetourneUnInt() ;

La syntaxe suivante suffit:

Code :

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

MaFonctionQuiRetourneUnInt();

Lorsque tu fais un appel à printf() qui retourne également un int:

Code :

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

int printf (const char *format, ...);

Tu n'écris pas:

Code :

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

1
2
3
(void) printf ("Bonjour les amis!\n"); 
/* mais */
printf ("Bonjour les amis!\n");

Si tu ne désires pas exploiter la valeur de retour de ta fonction MaFonctionQuiRetourneUnInt(), pourquoi récupères-tu cette valeur? Si tu penses l'utiliser dans le futur, et que tu ne veux plus recevoir d'avertissement du compilateur:

Code :

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

1
2
int rc = MaFonctionQuiRetourneUnInt();
(void) rc;

Thierry

[Emmanuel Delahaye]

Sous gcc en mode -Wall il me reporte deux fois

Code :

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

int Tmp = MaFonctionQuiRetourneUnInt() ;

Le compilateur rale

Normal. Tu définis une variable, tu l'initialises et tu ne la lis jamais. C'est que ton code est soit incomplet,

Code :

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

1
2
3
4
 
   int Tmp = MaFonctionQuiRetourneUnInt() ;
   if (Tmp ...)
   {

soit inutile.

Code :

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

   MaFonctionQuiRetourneUnInt() ;

voire

Code :

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

   (void) MaFonctionQuiRetourneUnInt() ;

pour les analyseurs de codes très tatillons...

[Clad3]

Merci, j'ai toujours cru qu'il ne fallait pas laisser une fonction qui renvoie autre chose que void, le faire dans le vide, sachatn que ce n'est pas gênant , effectivement :

Code :

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

1
2
 
Mafonction();

Est tout ce qu'il y a de plus logiuqe à écrire dans mon cas .

[gege2061]

Tu n'écris pas:

Code :

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

1
2
3
(void) printf ("Bonjour les amis!\n"); 
/* mais */
printf ("Bonjour les amis!\n");

Code :

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

1
2
3
4
Construction de main.obj.
E:\PellesC\Projects\test\main.c(5): warning #2216: Return value from function is never used.
Construction de test.exe.
Terminé.

Et je crois que lint le remarque aussi.

gcc possède un attribut que tu ajoute au prototype de la fonction :

Code :

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

__attribute__((warn_unused_result))

Bien plus pratique

[Thierry Chappuis]

Autant pour moi! J'admets n'avoir jamais utilisé que gcc (et lcc à l'occasion). Comment fais-tu alors? Tu places (void) devant chaque fonction dont tu ne récupères pas la valeur de retour?

Thierry

[gege2061]

Autant pour moi! J'admets n'avoir jamais utilisé que gcc (et lcc à l'occasion).

Je crois que PellesC est un dérivé de lcc.

Comment fais-tu alors? Tu places (void) devant chaque fonction dont tu ne récupères pas la valeur de retour?

Oui, sinon tu peux ignorer les warning (du moment que tu sais ce que tu fait) ou diminuer le niveau de warning

[Emmanuel Delahaye]

Autant pour moi! J'admets n'avoir jamais utilisé que gcc (et lcc à l'occasion). Comment fais-tu alors? Tu places (void) devant chaque fonction dont tu ne récupères pas la valeur de retour?

Oui, ou tu désactives l'option de lint, parce que franchement, c'est de la perte de temps. Si non a besoin du retour on l'utilise, sinon, on l'ignore, c'est tout. Faut laisser un peu d'initiative au programmeur...

[gege2061]

Oui, ou tu désactives l'option de lint, parce que franchement, c'est de la perte de temps. Si non a besoin du retour on l'utilise, sinon, on l'ignore, c'est tout. Faut laisser un peu d'initiative au programmeur...

C'est pour ça que l'option de gcc est plus adaptée, par exemple avec la fonction g_list_append de la glib, il peut arriver d'oublier la valeur de retour et tu te retrouve avec une liste chaînée obsolète