Discussion :

[Pitou5464]

Salut,

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
int do_ls(struct dirent * direntry)
{
  char          **a;
  DIR           *dir;
  int           i;
  int           j;
 
  i = 0;
  dir = opendir(direntry);
  while ((direntry =readdir(dir)) != NULL)
    {
      if (my_strcmp(direntry->d_name, ".") == 0)
        {
          direntry=readdir(dir);
        }
      if (my_strcmp(direntry->d_name, "..") == 0)
        {
          direntry=readdir(dir);
        }
      *a = direntry->d_name;
      a++;
     }
  closedir(dir);
  return 0;
}

je voudrais faire un malloc de **a car jai un segfault pour linstant.

[Faiche]

Code :

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

1
2
 
a = malloc (nb * sizeof (char *));

où nb est le nombre de cases.

Et si tu fais a++, tu perds la tete de ton tableau donc c'est n'importe quoi.

Et si tu veux copier une string tu fais un strdup.

ps : ça ressemble a un ls façon epitech ou je ne m'y connais pas.

[Médinoc]

Euh... opendir() n'est pas censé prendre une dirent en paramètre, mais un char const *.

Ensuite, je crois qu'au readdir suivant, la chaîne contenue dans la dirent n'est plus valide.
Il peut être préférable de dupliquer la chaîne (malloc(strlen())+strcpy(), ou bien directement strdup() qui est certainement disponible si opendir() l'est).

Ensuite, il te faut connaître à l'avance le nombre d'entrées dans le répertoire si tu veux allouer un tableau de chaînes.
Aussi, soit tu fais deux lectures, soit tu utilises une autre structure comme une liste chaînée (quitte à la transformer en tableau à la fin), soit tu joues avec les realloc() quand tu n'as plus de place.

[Pitou5464]

explique moi pourquoi je peut pas faire a++. je comprend pas le seg fault ici

[Faiche]

explique moi pourquoi je peut pas faire a++. je comprend pas le seg fault ici

La je dirais que tu le fais expres : tu déclare un char **a que tu n'initialise pas, et ensuite tu le déreférence en faisant *a, et y a rien qui te choque ?

Je te propose de revoir tes bases avant de faire au radar des exos qu'on te demande.

[Pitou5464]

ok bien vu l'aveugle

[Stevie Wonder]

ps : ça ressemble a un ls façon epitech ou je ne m'y connais pas.

+1

Je te propose de revoir tes bases avant de faire au radar des exos qu'on te demande.

re +1, si t'as des exos a faire et que t'es a la bourre, c'est pas la bonne solution de venir la chercher ici. Etant a epitech, je sais qu'il vaut mieux faire son exo jusqu'au bout et se bouffer 0 si t'as pas completement reussi que de choper la reponse ici et manger des lacunes.