Discussion :

[J4e8a16n]

Bonjour,

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

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

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

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

[Bktero]

Bonjour,

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

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.

[J4e8a16n]

et ça?

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

[Bktero]

et ça?

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

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.

[tatareau]

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

[ternel]

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.