Discussion :

[dancingmad]

Bonjour,

dans le cadre d'un exercice, je doit réaliser un programme my_ls qui imite la fonction ls. Il est dit dans le sujet que si aucun argument n'est fourni au programme, il utilisera le répertoire courant, c'est-à-dire que:

Code :

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

./my_ls

est équivalent à

Code :

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

./my_ls ./

Jusque là, pas de prob, voici à quoi ressemble le début du main:

Code :

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

1
2
3
4
5
if (argc==1) {
  execl(argv[0],argv[0],".",NULL);
  perror("execl");
  return EXIT_FAILURE;
}

Le problème, c'est que je dois maintenant rajouter des options (par exemple -a ou -l), et que dans ce cas,

Code :

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

./my_ls -l

doit être équivalent à

Code :

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

./my_ls -l ./

J'ai donc modifié le main:

Code :

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

1
2
3
recupOptions(argc,argv);
if (argv[optind]==NULL)  // optind est modifié par la fonction getopt()
  // on doit relancer le programme avec les mêmes options, mais en rajoutant "./"

Et je ne sais pas du tout comment rajouter à la fin de argv un nouvel argument !
Je n'ai pas trouvé de fonction exec* qui semble correspondre à ce que je veux. J'ai pensé à remplacer le NULL de argv par ".", mais c'est le meilleur moyen d'avoir un segfault. J'ai pensé créer dynamiquement un tableau identique à argv mais avec en plus la chaine ".", mais cette solution ne me parait pas très jolie...

Merci d'avance pour votre aide

[ram-0000]

J'ai pas bien compris l'intérêt de ce bout de code :

Code :

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

1
2
3
4
5
6
 
if (argc==1) {
  execl(argv[0],argv[0],".",NULL);
  perror("execl");
  return EXIT_FAILURE;
}

Visiblement, si il n'y a pas d'argument, tu relances le programme en fournissant un argument ".". Admettons mais quel gâchis, cela coûte cher en ressources de lancer un programme.

On peut imaginer un truc comme cela :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
char *repertoire = NULL;
if(argc == 1)
{
   repertoire = ".";
}
else
{
   repertoire = argv[1];
}
 
/* listing du répertoire */
...

Autre chose, je ne pense pas que tu aies regardé en détail la fonction getopt(). C'est elle qui te permettra de gérer correctement tous tes paramètres (et en plus dans l'aide, il y a un exemple d'utilisation, c'est trop facile ).

[dancingmad]

C'est vrai que j'avais pas trop réfléchi au fait que lancer un programme soit couteux... C'est effectivement plus simple comme ça

Sinon pour la fonction getopt, j'ai pas de problème, tout marche correctement...

Merci beaucoup !