Programme équivalent à DIR [Débutant(e)]

**Emmanuel Delahaye** · 15/09/2007, 20h30

Envoyé par _SamSoft_

Je suis allé sur : http://www710.univ-lyon1.fr/~jciehl/...strncat.3.html

Pas besoin d'aller si loin... http://man.developpez.com suffit bien...

Mais je n'ai pas compris quelque chose, size c'est la taille de quoi ?

du tableau de destination. C'est la seule chose qui nous intéresse, non ? Surtout ne pas déborder du tableau dans lequel on écrit.

Et aussi : "prendre de mes mesures", lesquelles ?

Arréter les dégâts, réallouer... tout ce qui évite 2 problèmes majeurs :

la boucle infinie (dans une fonction récursive c'est la mort).
le débordement de tableau.

bref, garder le contrôle.

**_SamSoft_** · 15/09/2007, 20h40

Merci

Je continuerai sûrement ca demain (après mes devoirs de maths

) ou mercredi prochain.

**_SamSoft_** · 16/09/2007, 16h52

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <sys/types.h>
#include <dirent.h>
#include <errno.h>
#include <sys/stat.h>
#include "myHeader.h"
int lister(char *directory);
 
int main (void)
{
   int end;
 
   do
   {
      char directory[256];
      printf ("nom ou <enter> pour quitter) : ");
      fgets (directory, sizeof directory, stdin);
      fclean (directory, stdin);
      end = (directory[0] == '\0');
      if (!end)
      {
         lister(directory);
      }
   }
   while (!end);
   return 0;
}
int lister(char *directory)
{
   DIR *dir = NULL;
   struct dirent *file = NULL;
   struct stat infos;
   stat (directory, &infos);
   size_t sizeDirectory = sizeof directory;
 
   if((dir = opendir(directory)) == NULL)
   {
      return EXIT_FAILURE;
   }
   printf("Repertoire %s :\n", directory);
 
   while((file = readdir(dir)) != NULL)
   {
      if(strcmp(file->d_name, ".") && strcmp(file->d_name, ".."))
      {
         if(S_ISDIR (infos.st_mode))
         {
            stat (file->d_name, &infos);
            if (strlen (file->d_name) + strlen (directory) + 1 < sizeDirectory)
            {
                strcat (directory, file->d_name);
            }
            else
            {
             /* prendre des mesures... */
            }
         }
         else
         {
            printf("\n\t%s\n", file->d_name);
         }
      }
   }
 
   printf("\nFin repertoire %s\n", directory);
 
   closedir(dir);
 
   return EXIT_SUCCESS;
}

J'ai ajouté la variable sizeDirectory contenant la taille de la variable directory ceci à pour conséquence d'enlever le problème de sortie de tableau. Pour la boucle, je ne vois pas le problème.

**Skyounet** · 16/09/2007, 16h55

Euh ici sizeof directory fera 4, la taille d'un pointeur.
Es-tu sur que c'est ce que tu veux faire ?

Tu veux pas plutot la taille ?

**Jean-Marc.Bourguet** · 16/09/2007, 17h16

Envoyé par Skyrunner

Euh ici sizeof directory fera 4, la taille d'un pointeur.
Es-tu sur que c'est ce que tu veux faire ?

Il y en a deux, un qui fonctionne comme attendu, un qui fait ce que tu dis.

**Emmanuel Delahaye** · 16/09/2007, 18h00

Envoyé par _SamSoft_

J'ai ajouté la variable sizeDirectory contenant la taille de la variable directory<...>

Je n'ai pas parlé d'ajouter une variable mais d'ajouter un paramètre... A ton avis, à quoi sert le 2 ème paramètre de fgets() ?

D'autre part, je rappelle que sizeof retourne la taille d'un objet en bytes et qu'un pointeur n'est pas un tableau.

**Skyounet** · 16/09/2007, 18h02

Envoyé par Jean-Marc.Bourguet

Il y en a deux, un qui fonctionne comme attendu, un qui fait ce que tu dis.

Oui je parlais du second, dans la fonction lister.

J'ai regarder le code, ça va provoquer une erreur, il faut que tu passes la taille en paramètre.

**_SamSoft_** · 16/09/2007, 18h52

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <sys/types.h>
#include <dirent.h>
#include <errno.h>
#include <sys/stat.h>
#include "myHeader.h"
int lister(char *directory, size_t sizeDir);
 
int main (void)
{
   int end;
 
   do
   {
      char directory[256];
      printf ("nom ou <enter> pour quitter) : ");
      fgets (directory, sizeof directory, stdin);
      fclean (directory, stdin);
      end = (directory[0] == '\0');
      if (!end)
      {
         size_t sizeDirectory = sizeof directory;
         lister(directory, sizeDirectory);
      }
   }
   while (!end);
   return 0;
}
int lister(char *directory, size_t sizeDir)
{
   DIR *dir = NULL;
   struct dirent *file = NULL;
   struct stat infos;
   stat (directory, &infos);
 
   if((dir = opendir(directory)) == NULL)
   {
      return EXIT_FAILURE;
   }
   printf("Repertoire %s :\n", directory);
 
   while((file = readdir(dir)) != NULL)
   {
      if(strcmp(file->d_name, ".") && strcmp(file->d_name, ".."))
      {
         if(S_ISDIR (infos.st_mode))
         {
            stat (file->d_name, &infos);
            if (strlen (file->d_name) + strlen (directory) + 1 < sizeDir)
            {
                strcat (directory, file->d_name);
            }
            else
            {
             /* prendre des mesures... */
            }
         }
         else
         {
            printf("\n\t%s\n", file->d_name);
         }
      }
   }
 
   printf("\nFin repertoire %s\n", directory);
 
   closedir(dir);
 
   return EXIT_SUCCESS;
}

Comme ceci ? Si oui, alors il reste un problème :

nom ou <enter> pour quitter) : c:\Dev-Cpp
Repertoire c:\Dev-Cpp :
main.o
Makefile.win
mingw32
NEWS.txt
Packages
Packman.exe
Packman.map
Projet1.dev
Projet1.exe
Projet1.layout
Templates
uninstall.exe
Fin repertoire c:\Dev-CppbinclearScreen.hcopying.txtdevcpp.exedevcpp.mapExamples
HelpIconsIcons.rarincludeLangliblibexecmain.c
nom ou <enter> pour quitter) :

Si je met à la place de sizeDir, sizeof directory dans :

Code :

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

1
2
 
if (strlen (file->d_name) + strlen (directory) + 1 < sizeof directory)

Il me met :

nom ou <enter> pour quitter) : c:\Dev-Cpp
Repertoire c:\Dev-Cpp :
main.o
Makefile.win
mingw32
NEWS.txt
Packages
Packman.exe
Packman.map
Projet1.dev
Projet1.exe
Projet1.layout
Templates
uninstall.exe
Fin repertoire c:\Dev-Cpp
nom ou <enter> pour quitter) :

Ce qui est beaucoup mieux et pourtant vous n'aimez pas ?!

**Skyounet** · 16/09/2007, 19h03

Tu as un bug dans ton programme c'est tout.

C'est à cause de cette ligne

Code :

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

if (strlen (file->d_name) + strlen (directory) + 1 < sizeDir)

Et il faut bien laisser sizeDir, si ça marche mieux avec sizeof directory c'est que tu as un problème d'algo, point

Je vois le problème moi, et je comprends pourquoi ça fait pas ce que tu veux que ça fasse, mais j'aimerais savoir ce que tu veux faire avec cette condition.

**_SamSoft_** · 16/09/2007, 19h10

A vrai dire c'est Emmanuel qui m'a proposé cette ligne de code et j'ai oublié de dire que je n'ai presque rien compris.

Pouvez - vous m'expliquer comment elle se comporte ?

**Skyounet** · 16/09/2007, 19h32

Envoyé par _SamSoft_

A vrai dire c'est Emmanuel qui m'a proposé cette ligne de code et j'ai oublié de dire que je n'ai presque rien compris.

Pouvez - vous m'expliquer comment elle se comporte ?

Il concatène le nom du fichier au nom du repertoire, mais auparavant il vérifie si le buffer est assez grand.

C'est juste pour l'affichage pour afficher le chemin complet des fichiers présents dans un repertoire.

Si tu n'en veux pas ben enlève le if.
En revanche si tu en veux faut modifier un peu (créer une autre variable et changer le printf), sinon tu va concatener à chaque fois à directory le nom du fichier (ça va rajouter à la suite).

Je crois que je suis pas bien clair, c'est normal j'ai pas l'esprit très clair

**_SamSoft_** · 16/09/2007, 19h43

Je croyais que c'était un des if majeurs en fait non. De plus je n'en ai pas besoin comme vous dites donc je le supprime (j'avais compris votre explication

)

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <sys/types.h>
#include <dirent.h>
#include <errno.h>
#include <sys/stat.h>
#include "myHeader.h"
int lister(char *directory, size_t sizeDir);
 
int main (void)
{
   int end;
 
   do
   {
      char directory[256];
      printf ("nom ou <enter> pour quitter) : ");
      fgets (directory, sizeof directory, stdin);
      fclean (directory, stdin);
      end = (directory[0] == '\0');
      if (!end)
      {
         size_t sizeDirectory = sizeof directory;
         lister(directory, sizeDirectory);
      }
   }
   while (!end);
   return 0;
}
int lister(char *directory, size_t sizeDir)
{
   DIR *dir = NULL;
   struct dirent *file = NULL;
   struct stat infos;
   stat (directory, &infos);
 
   if((dir = opendir(directory)) == NULL)
   {
      return EXIT_FAILURE;
   }
   printf("Repertoire %s :\n", directory);
 
   while((file = readdir(dir)) != NULL)
   {
      if(strcmp(file->d_name, ".") && strcmp(file->d_name, ".."))
      {
         if(S_ISDIR (infos.st_mode))
         {
            stat (file->d_name, &infos);
         }
         else
         {
            printf("\n\t%s\n", file->d_name);
         }
      }
   }
 
   printf("\nFin repertoire %s\n", directory);
 
   closedir(dir);
 
   return EXIT_SUCCESS;
}

**Emmanuel Delahaye** · 16/09/2007, 20h26

Envoyé par _SamSoft_

Je croyais que c'était un des if majeurs en fait non. De plus je n'en ai pas besoin comme vous dites donc je le supprime (j'avais compris votre explication

)

Je te rappelle que depuis le début, je te dis que la méthode de la concaténation n'est pas bonne (elle permet de descendre, mais pas de remonter, à moins de se rappeler le chemin supérieur, ce qui est assez complexe), et qu'il faut utiliser chdir(). Mais tu n'écoutes rien de ce qu'on te dis et tu fonces dans le tas sans réfléchir et surtout sans avoir exposé le but de ta manip.

En tout cas, si tu cherches à explorer une arborescence de fichiers, je te conseille de de poser et de traiter le problème pas à pas en n'écrivant pas une seule ligne de code que tu ne maitrises complètement.

Encore une fois, et c'est pas faute de te l'avoir signalé, tu vas beaucoup trop vite.

**Emmanuel Delahaye** · 16/09/2007, 20h29

Envoyé par _SamSoft_

A vrai dire c'est Emmanuel qui m'a proposé cette ligne de code et j'ai oublié de dire que je n'ai presque rien compris.

Pouvez - vous m'expliquer comment elle se comporte ?

Je ne vois pas ce qu'il y a de difficile à comprendre. Tu ne sais pas ce que fait strlen() ? Tu t'es renseigné ?

Elle permet de vérifier si il y a la place nécessaire à la concaténation et de réagir si ce n'est pas possible (plutôt que d'ignorer le problème silencieusement).

strlen() calcule la longueur des chaines actuelle (dest) à ajouter (src) et compare le résultat à la taille (au 0 final prêt, qui n'est jamais compté par strlen()).

**Skyounet** · 16/09/2007, 20h33

Oups pardon j'ai dis une bêtise, en fait il concatène pas le nom du fichier.

Si c'est un repertoire on rajoute son nom à notre variable pour pouvoir l'explorer à son tour.

D'ailleurs je viens de regarder le code pour de vrai cette fois (en lisant tout bien) et en fait il est faux il me semble.
Pourquoi tu appelles stat lorsque tu as un repertoire. Tu es censé appelé stat pour chaque fichier du repertoire. Si c'est un repertoire tu rappelles lister avec le nouveau nom (concatenation).

**_SamSoft_** · 17/09/2007, 13h00

Emmanuel :

-Tu as dis que chdir est une méthode de goret or tu dis qu'une méthode de goret n'est pas à utiliser !

-Strlen: Je sais ce que c'est. Je m'étais mal exprimé, je voulais dire à quoi servait la condition. (Apperement elle m'est très util pour créer l'arboressence des autres dossiers/fichiers)

Skyrunner:

-Oui, merci c'est le truc que j'arrivais pas à comprendre donc la condition d'Emmanuel va finalement m'être très utile. J'en suis là pour le moment :

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <sys/types.h>
#include <dirent.h>
#include <errno.h>
#include <sys/stat.h>
#include "myHeader.h"
int lister(char *directory);
 
int main (void)
{
   int end;
 
   do
   {
      char directory[256];
      printf ("nom ou <enter> pour quitter) : ");
      fgets (directory, sizeof directory, stdin);
      fclean (directory, stdin);
      end = (directory[0] == '\0');
      if (!end)
      {
         size_t sizeDirectory = sizeof directory;
         lister(directory);
      }
   }
   while (!end);
   return 0;
}
int lister(char *directory)
{
   DIR *dir = NULL;
   struct dirent *file = NULL;
   struct stat infos;
   stat (directory, &infos);
 
   if((dir = opendir(directory)) == NULL)
   {
      return EXIT_FAILURE;
   }
   printf("Repertoire %s :\n", directory);
 
   while((file = readdir(dir)) != NULL)
   {
      if(strcmp(file->d_name, ".") && strcmp(file->d_name, ".."))
      {
         if(S_ISDIR (infos.st_mode))
         {
                    printf("\n\t%s\n", file->d_name);
         }
         else
         {
            printf("\n\t%s\n", file->d_name);
            stat (file->d_name, &infos);
         }
      }
   }
 
   printf("\nFin repertoire %s\n", directory);
 
   closedir(dir);
 
   return EXIT_SUCCESS;
}

Je suis en train de réfléchier à la façon de lister les fichiers des dossiers dans le dossier principal (très clair hein ?

)

EDIT: Je viens de voir que je peux m'arreter là. Vu que si l'on fait :

c:\dev-cpp on a :

bin
templates
fichier.o
...

Ensuite on peut faire c:\dev-cpp\bin et on aura ce qu'il contient

Donc nullement besoin de faire c:\dev-cpp :

bin

fichier.dll
fichier.exe

templates

hello.c
sdl.c

...

En revanche je vais travailler sur la fiabilité du programme car si on tape n'importe quoi, ca devient instable.

**Thierry Chappuis** · 17/09/2007, 13h48

La boucle while dans lister() ne doit pas plutôt être comme ci-dessous?

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
while ((file = readdir(dir)) != NULL)
    {
        if (strcmp(file->d_name, ".") && strcmp(file->d_name, ".."))
        {
            stat(file->d_name, &infos);
            if (S_ISDIR (infos.st_mode))
            {
                printf("\n\t%s - (directory)\n", file->d_name);
            }
            else
            {
                printf("\n\t%s\n", file->d_name);
            }
        }
    }

Pourquoi stat() n'est-elle appelée que si file n'est pas un répertoire? Dans ton code, S_ISDIR(infos.st_mode) est toujours vrai, car stat() est appelé une seule fois au début de lister().

Thierry

**Emmanuel Delahaye** · 17/09/2007, 14h28

Envoyé par Thierry Chappuis

Pourquoi stat() n'est-elle appelée que si file n'est pas un répertoire? Dans ton code, S_ISDIR(infos.st_mode) est toujours vrai, car stat() est appelé une seule fois au début de lister().

Je lui ais déjà dit 3 fois, mais l'animal est têtu comme une mule et ne lit pas les réponses. C'est désespérant.

**Thierry Chappuis** · 17/09/2007, 14h47

Envoyé par Emmanuel Delahaye

Je lui ais déjà dit 3 fois, mais l'animal est têtu comme une mule et ne lit pas les réponses. C'est désespérant.

OK! Au temps pour moi... J'admets ne pas avoir lu en intégralité les 70 premiers posts.

Thierry

**_SamSoft_** · 17/09/2007, 18h38

Emmanuel: J'ai pas du saisir de quoi vous parliez.

Ce qui donne (code final) :

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <sys/types.h>
#include <dirent.h>
#include <errno.h>
#include <sys/stat.h>
#include "myHeader.h"
int lister(char *directory);
 
int main(void)
{
   int end;
 
   do
   {
      char directory[256];
      printf ("NAME OR <ENTER> FOR EXIT) : ");
      fgets (directory, sizeof directory, stdin);
      fclean (directory, stdin);
      end = (directory[0] == '\0');
      if (!end)
      {
         lister(directory);
      }
   }
   while (!end);
   return 0;
}
int lister(char *directory)
{
   DIR *dir = NULL;
   struct dirent *file = NULL;
   struct stat infos;
 
   if((dir = opendir(directory)) == NULL)
   {
      return EXIT_FAILURE;
   }
   printf("FOLDER %s:\n", directory);
 
   while((file = readdir(dir)) != NULL)
   {
   stat (file->d_name, &infos);
      if(strcmp(file->d_name, ".") && strcmp(file->d_name, ".."))
      {
         if(S_ISDIR (infos.st_mode))
         {
      printf("\n[FOLDER]\t%s\n", file->d_name);
         }
         else
         {
            printf("\n[FILE]\t%s\n", file->d_name);
         }
      }
   }
 
   printf("\nEND OF THE FOLDER: %s\n", directory);
 
   closedir(dir);
 
   return EXIT_SUCCESS;
}

Cependant il reste un problème, si je met c:\dev-cpp il me sort :

NAME OR <ENTER> FOR EXIT) : c:\dev-cpp
FOLDER c:\dev-cpp:
[FOLDER] bin
[FOLDER] clearScreen.h
[FOLDER] copying.txt
[FOLDER] devcpp.exe
[FOLDER] devcpp.map
[FOLDER] Examples
[FOLDER] Help
[FOLDER] Icons
[FOLDER] Icons.rar
[FOLDER] include
[FOLDER] Lang
[FOLDER] lib
[FOLDER] libexec
[FILE] main.c
[FILE] main.o
[FILE] Makefile.win
[FILE] mingw32
[FILE] NEWS.txt
[FILE] Packages
[FILE] Packman.exe
[FILE] Packman.map
[FILE] Projet1.dev
[FILE] Projet1.exe
[FILE] Projet1.layout
[FILE] Templates
[FILE] uninstall.exe
END OF THE FOLDER: c:\dev-cpp
NAME OR <ENTER> FOR EXIT) :

Si je met c:\dev-cpp\bin :

NAME OR <ENTER> FOR EXIT) : c:\dev-cpp\bin
FOLDER c:\dev-cpp\bin:
[FOLDER] addr2line.exe
[FOLDER] ar.exe
[FOLDER] as.exe
[FOLDER] c++.exe
[FOLDER] c++filt.exe
[FOLDER] cpp.exe
[FOLDER] dlltool.exe
[FOLDER] dllwrap.exe
[FOLDER] g++.exe
[FOLDER] gcc.exe
[FOLDER] gccbug
[FOLDER] gcov.exe
[FOLDER] gdb.exe
[FOLDER] gprof.exe
[FOLDER] ld.exe
[FOLDER] make.exe
[FOLDER] mingw32-c++.exe
[FOLDER] mingw32-g++.exe
[FOLDER] mingw32-gcc-3.4.2
[FOLDER] mingw32-gcc.exe
[FOLDER] mingw32-make.exe
[FOLDER] mingwm10.dll
[FOLDER] nm.exe
[FOLDER] objcopy.exe
[FOLDER] objdump.exe
[FOLDER] ranlib.exe
[FOLDER] readelf.exe
[FOLDER] rm.exe
[FOLDER] size.exe
[FOLDER] strings.exe
[FOLDER] strip.exe
[FOLDER] windres.exe
END OF THE FOLDER: c:\dev-cpp\bin
NAME OR <ENTER> FOR EXIT) :

Donc un gros problème (mais où ?

)

Programme équivalent à DIR [Débutant(e)]

C

Discussions similaires

Partager

Partager