Discussion :

[Invité]

Bonjour tout le monde
J'ai un petit ennui avec getopt... J'essaye de recuperer plusieurs valeurs pour une option mais apparement les options suivantes ne sont plus prises en compte...

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
 
  while ((c = getopt(argc, argv, "p:x:y:c:t:vn::" )) != EOF)
  {
      if (c == 'p')
    env.port  = atoi(optarg);
      else if (c == 'x')
    env.map_x = atoi(optarg);
      else if (c == 'y')
    env.map_y = atoi(optarg);
      else if (c == 'c')
    env.max_players = atoi(optarg);
      else if (c == 't')
    env.cronos = atoi(optarg);
      else if (c == 'v')
    env.verbose = 1;
      else if (c == 'n')
    {
      env.teams = xmalloc(3 * sizeof(*env.teams));
      env.teams[2] = "\0";
      env.teams[0] = strdup(optarg);
      optarg += (strlen(env.teams[0]) + 1);
      env.teams[1] = strdup(optarg);
    }
  }

et je lance cette commande :
./serveur -p 39115 -x 666 -y 69 -v-n LoL Ookay -t 4242 -c 11

Le resultat :

map_x : 666
map_y : 69
port : 39115
Team 1 : LoL
Team 2 : Ookay
verbose : 1
max_players : 0
cronos : 0

En fait le getopt ignore toutes les options passees apres...


Quelqu'un aurait il une petite idee ? Merci d'avance !

[DaZumba]

Le format de description donne a getopt() n'est pas correct pour l'option -n. Contrairement a l'intuition, un double : ne signifie pas 'deux options suivent', mais que l'option est de la forme -n<first> <second>, sans espace entre -n et <first>.
Ceci fonctionne (j'ai rajoute une troisieme option a -n pour montrer la bonne utilisation):

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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
 
#include <stdio.h>
#include <unistd.h>
#include <string.h>
 
int main(int argc, char *argv[])
{
  int c;
 
  while ((c = getopt(argc, argv, "p:x:y:c:t:vn:01" )) != EOF)
  {
    printf("%c: ", c);
    fflush(stdout);
 
    switch (c)
    {
      case 'p':
        printf("port = %d\n", atoi(optarg));
        break;
      case 'x':
        printf("map_x = %d\n", atoi(optarg));
        break;
      case 'y':
        printf("map_y = %d\n", atoi(optarg));
        break;
      case 'c':
        printf("max_players = %d\n", atoi(optarg));
        break;
      case 't':
        printf("cronos = %d\n", atoi(optarg));
        break;
      case 'v':
        printf("verbose switched on\n");
        break;
      case 'n':
        {
          char *p = optarg;
 
          printf("%s ", p);
          p += strlen(p) + 1;
          printf("%s ", p);
          p += strlen(p) + 1;
          printf("%s\n", p);
        }
        break;
    }
  }
 
  return 0;
}

Si j'appelle le programme ainsi:

Code :

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

1
2
 
toto -p 39115 -x 666 -y 69 -v -n team1 team2 team3 -t 4242 -c 11

j'obtiens bien:

Code :

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

1
2
3
4
5
6
7
8
 
p: port = 39115
x: map_x = 666
y: map_y = 69
v: verbose switched on
n: team1 team2 team3
t: cronos = 4242
c: max_players = 11

Derniere remarque: la fonction atoi() est desormais remplacee par strtol(). Aucun nouveau code ne doit utiliser atoi().

[Invité]

Merci pour ta reponse
Cependant...

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
25
while ((c = getopt(argc, argv, "p:x:y:c:t:vn:0")) != EOF)
  {
      if (c == 'p')
    env->port  = atoi(optarg);
      else if (c == 'x')
    env->map_x = atoi(optarg);
      else if (c == 'y')
    env->map_y = atoi(optarg);
      else if (c == 'c')
    env->max_players = atoi(optarg);
      else if (c == 't')
    env->cronos = atoi(optarg);
      else if (c == 'v')
    env->verbose = 1;
      else if (c == 'n')
    {
      char *p = optarg;
 
      env->teams = xmalloc(3 * sizeof(*env->teams), env);
      env->teams[0] = strdup(p);
      p += strlen(p) + 1;
      env->teams[1] = strdup(p);
      env->teams[2] = "\0";
    }
  }

Si je lance de cette maniere :
./serveur -p 39115 -n LoL Okket -x 22 -y 11 -v -t 100 -c 10

le programme reste bloque en boucle infinie...

[DaZumba]

Tu declares bien c comme etant un int? Si tu le declares comme char, c ne pourra jamais etre egal a EOF...

[Invité]

Oui c'est bien un int ... En fait tant que le nom des equipes est passe en tan que dernier parametre, tout va bien ! Des qu'il est passe en premier ou autre parametre, le programme se bloque

[DaZumba]

Je ne vois pas. L'erreur doit se produire ailleurs ou alors ton getopt() a un comportement inattendu, mais ce serait vraiment surprenant...