probleme avec la recursivite

Version imprimable

J'ai un code qui permet de lister tout les repertoires et les fichiers de mon repertoire passe en parametre. Le probleme c'est que je voudrais qu'il fasse la meme chose lorsqu'il tombe sur repertoire et non sur un fichier. Donc je voudrais rappeler la fonction mais il seg fault.

Code:

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
int     do_ls(char *rep)
{
  DIR           *dir;
  struct dirent *direntry;
  struct stat   s;
 
  dir = opendir(rep);
  while ((direntry =readdir(dir)) != NULL)
    {
      stat(direntry->d_name, &s);
      if ((s.st_mode & S_IFMT) == S_IFDIR)
        {
          do_ls(direntry->d_name);
        }
      else
        {
          aff_type(s.st_mode);
          aff_perm(s.st_mode);
          my_putchar(' ');
          my_putnbr(s.st_nlink);
          my_putchar(' ');
          my_putstr(getpwuid(s.st_uid)->pw_name);
          my_putchar(' ');
          my_putstr(getgrgid(s.st_gid)->gr_name);
          my_putchar(' ');
          my_putnbr(s.st_size);
          my_putchar(' ');
          print_date(s.st_mtime);
          my_putchar(' ');
          my_putstr(direntry->d_name);
          my_putchar('\n');
        }
    }
 
}

06/11/2006, 16h30
Patriarch24

Sors ton debugger et c'est parti.

Citation:

dir = opendir(rep);

Vérifie le retour. (dir != NULL).
06/11/2006, 16h54
Pitou5464

merci pour ta reponse mais je crois pas que ce soit ici que ca peche. par contre si j'enleve cette ligne ca marche mais plus de recursivite...

Code:

do_ls(direntry->d_name);
06/11/2006, 16h59
Médinoc

Le premier répertoire trouvé ne serait-il pas "." ?
06/11/2006, 17h09
Pitou5464

oui enfin je le passe en argument, mais de toute facon dans nimporte quelle repertoire il y aura toujours le point non?
06/11/2006, 17h21
zooro

Citation:

Envoyé par Pitou5464

oui enfin je le passe en argument, mais de toute facon dans nimporte quelle repertoire il y aura toujours le point non?

Oui, mais si le "." est passé indéfiniment, tu finiras sans doute par manquer de mémoire...
Il faut tester le répertoire avant l'appel récursif, et ne pas le faire dans le cas de "." et de "..".

J'ai retouche le code mais cette fois il ne passe meme pas dans les repertoires.

Code:

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
int     do_ls(char *rep)
{
  DIR           *dir;
  struct dirent *direntry;
  struct stat   s;
 
  dir = opendir(rep);
  if (dir != NULL)
    {
      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);
          if ((s.st_mode & S_IFMT) == S_IFDIR)
            {
              do_ls(direntry->d_name);
            }
          stat(direntry->d_name, &s);
          aff_type(s.st_mode);
          aff_perm(s.st_mode);
          my_putchar(' ');
          my_putnbr(s.st_nlink);
          my_putchar(' ');
          my_putstr(getpwuid(s.st_uid)->pw_name);
          my_putchar(' ');
          my_putstr(getgrgid(s.st_gid)->gr_name);
          my_putchar(' ');
          my_putnbr(s.st_size);
          my_putchar(' ');
          print_date(s.st_mtime);
          my_putchar(' ');
          my_putstr(direntry->d_name);
          my_putchar('\n');
        }
    }
}

Code:

1 2 3 4 if (my_strcmp(direntry->d_name, ".") == 0) direntry=readdir(dir); if (my_strcmp(direntry->d_name, "..") == 0) direntry=readdir(dir);
Utilise continue à la place, ça t'évitera de risquer de passer par un pointeur NULL si un répertoire est vide...

Code:

if ((s.st_mode & S_IFMT) == S_IFDIR)

Où s est-il mis à jour dans la boucle?

06/11/2006, 17h54
Pitou5464
J'ai essaye de le mettre en dessous du while parce que c'est la que j'ai mis le readdir, mais ca peche encore
Code:

1 2 3 while ((direntry =readdir(dir)) != NULL) { stat(direntry->d_name, &s);
06/11/2006, 17h59
Pitou5464

ce que je ne comprend pas c'est qu'il devrait refaire la fonction parce qu'il y a des sous repertoires dans mon repertoire alors qu'il ne fait rien...:triste:
et j'ai beau mettre le stat partout pour mettre a jour s ca peche damn