Discussion :

[Pitou5464]

J'essaye d'attribuer des options a mon prog mais le code que j'ai taper ne fonctionne pas. ca compile mais il n'effectue pas les operations demandees. le premier if fonctionne par contre, mais pour l'option -d et -l ca ne fonctionne pas.

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
int main(int argc, char **argv)
{
  int   i;
 
  i = 0;
  while (argv[i] != '\0')
    {
      if (argc == 1)
        {
          printf("Wrong number of arguments %d\n", argc - optind);
        }
      if (argv[i] == "-d")
        {
          my_putchar('.');
          my_putchar('\n');
        }
      if (argv[i] == "-l")
        {
          do_ls_l(argv[2]);
        }
      i++;
    }
}

[Trap D]

Il faut utiliser strcmp pour la comparaison des arguments avec "-d" ou "-l".

Code :

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

1
2
3
if (strcmp(argv[i], "-d")  == 0)
{
}

[Médinoc]

• argv[i] est un char*, pas un caractère: Il faut comparer à NULL et non '\0' (même si ça donne la même chose).
• Je ne sais plus s'il est garanti que le dernier argument est NULL. Par contre, il est garanti qu'il y a argc arguments valides: Tu dois donc limiter ta boucle avec argc (une belle boucle for, par exemple).
• Pourquoi mettre la comparaison de argc et 1 dans la boucle? Est-ce que argc change dans la boucle? Est-ce que 1 change dans la boucle?
• On ne compare pas deux chaînes avec == : On utilise la fonction strcmp() (déclarée dans <string.h>) qui retourne 0 si deux chaînes sont identiques.

PS: Il y a vraiment des gens qui trouvent que l'indentation GNU est lisible??

[Emmanuel Delahaye]

argv[i] est un char*, pas un caractère: Il faut comparer à NULL et non '\0' (même si ça donne la même chose).

Pas toujours. NULL peut être défini ainsi :

Code :

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

#define NULL ((void *)0)

Je ne sais plus s'il est garanti que le dernier argument est NULL.

C'est garanti par le langage.

PS: Il y a vraiment des gens qui trouvent que l'indentation GNU est lisible??

Pas moi, en tout cas...

[Pitou5464]

merci trap d pour ton aide, je vais utiliser strcmp

[gege2061]

J'essaye d'attribuer des options a mon prog mais le code que j'ai taper ne fonctionne pas. ca compile mais il n'effectue pas les operations demandees. le premier if fonctionne par contre, mais pour l'option -d et -l ca ne fonctionne pas.

Version simple : Analyser les options passées en ligne de commande
Version plus élaborée : Gestion des options de la ligne de commande

[Pitou5464]

je vais regarder ca gege