variable «*ret*» définie mais non utilisée

Bonjour,

Bizarre. variable «*ret*» définie mais non utilisée

Code:

---

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

int main(int argc, char *argv[]) { int ret = 0;   if (argv[1][0] == '/' || argv[1][0] == '-' || argc == 1) { usage(); return 0; }     if (system(NULL)) { my_clrscr(); /* cls */ ret = (int)system(argv[1]); }

---

Code:

---

1 2 3

sysvars.c:26:7: warning: variable «*ret*» définie mais non utilisée [-Wunused-but-set-variable] int ret = 0;

---

Elle l'est.

Merci


JPD

Bonjour,

C'est gentil de ne pas donné le code complet, du coup les numéros de lignes ne correspondent pas :ptdr:

Et non elle n'est pas utilisée : tu lui donnes une valeur mais tu n'en fais rien. Donc c'est comme ci tu ne servais pas. ;)

et ça?

#16 ret = (int)system(argv[1]);

Citation:

---

Envoyé par J4e8a16n

et ça?

#16 ret = (int)system(argv[1]);

---

Citation:

---

Envoyé par Bktero

tu lui donnes une valeur mais tu n'en fais rien

---

Je ne peux pas mieux te dire... Tu supprimes la variable 'ret' de ton code, ton code fait la même chose. Ta variable n'est pas utilisée.

Au passage, ce cast en int ne sert à rien puisque system() renvoie un int.

+1 Éternelle confusion entre définition (et ici avec initialisation) et utilisation ;)

Au passage, tu as un soucis avec if (argv[1][0] == '/' || argv[1][0] == '-' || argc == 1).
Si argc vaut 1, alors argv[1] vaut NULL, et argv[1][0] est une erreur de segmentation.

Pour t'en prémunir, il faut écrire if (argc == 1 || argv[1][0] == '/' || argv[1][0] == '-').
Ainsi, grace au mécanisme dir "shortcut evaluation", si argc vaut 1, alors l'expression est vraie, et les deux argv[1][0] == ... ne sont pas évaluées.

:o)

Je m'en suis prémuni, après ............. avoir eu une erreur de segmentation. Ha!

JPD