Problème de warning : unused variable .

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

Citation:

---

unused varaible "..."

---

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

Code:

---

1 2	int Tmp = MaFonctionQuiRetourneUnInt() ;

---

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

Code:

---

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:

---

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 ...

Bonjour,

Citation:

---

Envoyé par Clad3

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:

---

(void)MaFonctionQuiRetourneUnInt() ;

---

;)

Citation:

---

Envoyé par gege2061

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:

---

(void)MaFonctionQuiRetourneUnInt() ;

---

;)

---

La syntaxe suivante suffit:

Code:

---

MaFonctionQuiRetourneUnInt();

---

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

Code:

---

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

---

Tu n'écris pas:

Code:

---

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:

---

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

---

Thierry

Citation:

---

Envoyé par Clad3

Sous gcc en mode -Wall il me reporte deux fois

Code:

---

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:

---

1 2 3 4

int Tmp = MaFonctionQuiRetourneUnInt() ; if (Tmp ...) {

---

soit inutile.

Code:

---

MaFonctionQuiRetourneUnInt() ;

---

voire

Code:

---

(void) MaFonctionQuiRetourneUnInt() ;

---

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

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:

---

1 2	Mafonction();

---

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

Citation:

---

Envoyé par mujigka

Tu n'écris pas:

Code:

---

1 2 3

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

---

---

Code:

---

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é.

---

:langue:
Et je crois que lint le remarque aussi.

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

Code:

---

__attribute__((warn_unused_result))

---

Bien plus pratique :P

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

Citation:

---

Envoyé par mujigka

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.

Citation:

---

Envoyé par mujigka

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

Citation:

---

Envoyé par mujigka

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...

Citation:

---

Envoyé par Emmanuel Delahaye

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 :(