Discussion :

[moussmouss]

Bonjour,

j'utilise getopt pour récupérer le soptions positionnées en argument de mon prog.

je boucle sans fin sur le while. Pour info, je suis sur une plateforme Unix AIX.

Voici le code concerné :

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
while((c = getopt(argc,argv,"FfMm?"))!=EOF) 
{ 
  switch(c) 
  { 
    case 'f': 
    case 'F': 
    case 'm': 
    case 'M': 
      printf("option %c\n",c); 
      break; 
    case '?': 
      printf("option %c\nargument %s\n",c,optarg); 
      break; 
  } 
}

Que je mette EOF comme test dans le while ou -1, le résultat est le même.

Merci par avance de votre aide...

[PRomu@ld]

getopt retourne EOF dès qu'il a fini, donc n'utilise pas -1.

Ensuite, est ce que tu passes les bons paramètres ?

Si ça peut d'aider :

http://man.developpez.com/man3/getopt.3.php

[fearyourself]

J'ai testé ton programme et ça marche chez moi... Par contre, en principe, c'est -1 qu'il faut tester...

Ensuite j'ai pensé à une discussion pas trop lointaine avec Emmanuel et je me suis dit:

"Tu fais un switch sur les caractères.... As-tu déclaré ta variable comme un caractère?"

Parce que si t'es un peu comme moi, ton code entier serait:

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
18
19
20
21
22
23
24
 
 
int main()
{
char c;
 
while((c = getopt(argc,argv,"FfMm?"))!=EOF)
 {
 switch(c)
  {
  case 'f':
  case 'F':
  case 'm':
  case 'M':
    printf("option %c\n",c);
    break;
  case '?':
    printf("option %c\nargument %s\n",c,optarg);
    break;
  }
 }
 
return 1;
}

Et alors je me suis dis: "Est-ce ton char est signé ou non par défaut?" Parce que si je mets:

Code :

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

unsigned char c;

Ca boucle aussi...

Il faudrait mettre:

Code :

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

signed char c;

Mais encore mieux:

Code :

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

1
2
3
4
5
6
7
8
 
int i;
char c;
 
while( (i = getopt(argc,argv,"FfMm?"))!=EOF) 
  {
  c = i;
  ...

"A tribute to Emmanuel"


Jc

[fearyourself]

Ahh la contradiction:

getopt retourne EOF dès qu'il a fini, donc n'utilise pas -1.

et

Par contre, en principe, c'est -1 qu'il faut tester...

Ca m'apprendra de lire le man anglais... C'est la première fois que je vois une différence flagrante entre les deux langues...

Je suppose que EOF == -1 donc ce n'est pas grave, mais est-ce vrai sur tous les systèmes (j'ai appris à être méfiant ces derniers jours...)?

[PRomu@ld]

Après une petite recherche, je suis tombé sur la même contradiction. Un man de linux en français donne EOF et le man de mon mac donne -1. Je suppose comme toi que EOF donne -1 mais j'ai un doute.

[Emmanuel Delahaye]

Code :

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

1
2
3
4
5
6
7
8
 
int i;
char c;
 
while( (i = getopt(argc,argv,"FfMm?"))!=EOF) 
  {
  c = i;
  ...

Pas la peine de faire si compliqué. getopt() retourne un int. c doit donc être de type int. point. De plus, c'est exactement le type attendu par switch()...

Code :

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

1
2
3
4
5
6
7
 
   int c;
 
   while ((c = getopt (argc,argv,"FfMm?")) != EOF) 
   {
      switch (c)
<...>

Toujours aller au plus simple. Le type 'naturel' pour un entier, c'est int. D'ailleurs c'est aussi le type d'une constante caractère : 'A' est de type int. (Tester un sizeof 'A' pour s'en convaincre)

[Pouic]

Sur ma Debian :

VALEUR RENVOYÉE
La fonction getopt() renvoie le caractère de l'option s'il en trouve une, « : » s'il
manque un paramètre pour une option, « ? » s'il y a un caractère inconnu, et EOF (-1) à
la fin de la liste des options.

8) Ca met tout le monde d'accord 8)

[moussmouss]

merci à vous tous.

chez moi ça marche aussi bien avec -1 que EOF, et l'erreur venait bien que par défaut mes 'char' sont unsigned.

j'ai redéclarer ma variable 'c' en 'signed char', et ça ne boucle plus.

Merci à vous tous pour vos répnses aussi rapide !!

[PRomu@ld]

Sur le site de GNU, ils mettent aussi -1 comme valeur de retour au cas où il n'y a plus d'arguments.

[Emmanuel Delahaye]

Je suppose que EOF == -1 donc ce n'est pas grave, mais est-ce vrai sur tous les systèmes (j'ai appris à être méfiant ces derniers jours...)?

EOF est défini par le langage C comme un 'int < 0'. C'est tout.

getopt() n'étant pas une fonction standard, on aura du mal à avoir une définition unique.

Mais supposer que EOF vaut -1 est une erreur qui peut mener à des erreurs graves. Si la doc de getopt() dit 'la valeur retournée est -1'. C'est -1. Point. C'est pas EOF.

Si elle dit 'c'est EOF', c'est EOF. Point. C'est pas -1. Il faut être cohérérent et ne pas faire de suppositions hasardeuses.

[Emmanuel Delahaye]

j'ai redéclarer ma variable 'c' en 'signed char', et ça ne boucle plus.

Mauvais type. Il faut mettre 'int'.

[Emmanuel Delahaye]

Sur ma Debian :

VALEUR RENVOYÉE
La fonction getopt() renvoie le caractère de l'option s'il en trouve une, « : » s'il
manque un paramètre pour une option, « ? » s'il y a un caractère inconnu, et EOF (-1) à
la fin de la liste des options.

8) Ca met tout le monde d'accord 8)

Non Ceci est peut être valable sur une implémentation donnée, mais peut être fausse ailleurs...

Encore une fois, getopt() n'est pas une fonction standard C. Il ne faut pas s'attendre à des définitions cohérentes d'une plateforme à l'autre (à moins qu'elle soit POSIX, dans ce cas seule cette norme fait autorité...).

[PRomu@ld]

Aparement, elle est bien POSIX (en tout cas à en croire le man de mon mac)

[Pouic]

Ouaip, elle est bien Posix.2 et elle doit rendre -1 :
http://man.developpez.com/man3/getopt.3.php

The getopt() function shall return -1, rather than EOF, so that <stdio.h> is not required.

Emmanuel>> Mon post était juste une blagounette, car mon man trouve le moyen de dire ``indifferemment'' EOF ou -1. Je sais bien que les manuels sont fortements dépendants de l'implantation (entre distribs, j'ai même déjà eu des surprises...)

[Emmanuel Delahaye]

The getopt() function shall return -1, rather than EOF, so that <stdio.h> is not required.

Voilà. Ca, au moins, c'est clair.