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

**Emmanuel Delahaye** · 17/09/2007, 18h45

Envoyé par _SamSoft_

Donc un gros problème (mais où ?

)

En commençant par corriger ça.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `main':
main.c:55: warning: control reaches end of non-void function
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 1 seconds)
0 errors, 1 warnings

J'ai indiqué 3000 fois comment configurer le compilateur. Je ne comprends pas que ce genre de bug ne soit pas corrigé. A croire que tu ne lis ni les réponses ni les messages du compilateur...

Autre problème, comme tu t'obstines à mener plusieurs projets de front, arrive ce qui devait arriver. Tu mélanges les questions, les codes, les résultats... Bref, c'est le souk...

Je serais toi, j'arrêterais de me disperser et je traiterais un sujet à la fois, mais à fond et étape par étape.

**Skyounet** · 17/09/2007, 18h53

Tu t'es trompé de code _SamSoft_ tu as mis le code d'un autre projet.

**_SamSoft_** · 17/09/2007, 19h12

C'est quoi la blague (j'ai pas saisis) c'est bien le code du projet du listeur de fichiers/dossiers -_-

**Skyounet** · 17/09/2007, 19h13

Envoyé par _SamSoft_

C'est quoi la blague (j'ai pas saisis) c'est bien le code du projet du listeur de fichiers/dossiers -_-

Pas dans ce message non.
http://www.developpez.net/forums/sho...6&postcount=80

**_SamSoft_** · 17/09/2007, 19h20

Ah et m****e [aie] (toutes mes excuses, Emmanuel m'avait pourtant prévenu -_-)

Je corrige (et je reprecise que je reste sur ce projet) !

**_SamSoft_** · 17/09/2007, 19h21

Et moi je vous ai dit 3000 fois que je l'ai fais :

Code :

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

1
2
 
-Wall -Wextra -O2 -Wwrite-strings -Wstrict-prototypes

Je l'ai mis dans Outils/Options du compilo/
Appeler lors de la compilation.

Et je n'ai pas votre message -_-

EDIT: En reconstruisant et en recompilant j'ai vu :

size_t sizeDir = sizeof directory ne sert à rien donc j'ai supprimé :

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;
}

PS: Ok, je reste sur ce projet il est plus interessant

**_SamSoft_** · 18/09/2007, 13h08

Je suis très proche de la fin du logiciel, j'ai réussi à comprendre les erreurs (il en reste encore -_- mais je cherche)...

code :

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
74
75
76
77
 
#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 list(char *directory, size_t sizeDir);
 
int main(void)
{
  int end;
  size_t sizeDir;
  char directory[256];
  sizeDir = sizeof directory;
 
 
   do
   {
      printf ("NAME OR <ENTER> FOR EXIT) : ");
      fgets (directory, sizeof directory, stdin); /* we take the path */
      fclean (directory, stdin);
      end = (directory[0] == '\0');
      if (!end)
      {
         list(directory, sizeDir);
      }
   }
   while (!end); //while end[0] != '\0'
   return 0;
}
int list(char *directory, size_t sizeDir)
{
   DIR *dir = NULL;
  int err;
   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);
   err = stat(file->d_name, &infos);
      if(strcmp(file->d_name, ".") && strcmp(file->d_name, ".."))
      {
     if(err)
     {
          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);
          }
    }
        else
        {
         printf ("'%s' IS UNKNOW\n", directory);
         return 1;
        }
      }
   }
 
   printf("\nEND OF THE FOLDER: %s\n", directory);
 
   closedir(dir);
 
   return EXIT_SUCCESS;
}

J'ai toujours le problème : si je met c:\dev-cpp il met tout le temps [FOLDER] file->d_name;

Si vous avez une idée (de(s) ligne(s) ?)

Merci d'avance

**Thierry Chappuis** · 18/09/2007, 13h40

En cas de réussite, la fonction stat() renvoie la valeur 0 en cas de réussite. Dès lors, je ne comprends pas ça:

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
            if (err)
            {
                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);
                }
            }
            else
            {
                printf ("'%s' IS UNKNOW\n", directory);
                return 1;
            }

Thierry

**Thierry Chappuis** · 18/09/2007, 14h25

Il y a un problème avec le code de ta fonction list() au niveau de l'appel de stat(). En effet, à moins de tester le répertoire courant (i.e. directory pointe sur la chaine "."), stat(file->d_name, &infos) retourne -1, car le fichier en question ne peut être trouvé dans le répertoire courant (ou alors c'est un hasard, et il ne s'agit pas du bon fichier).

Pour que la fonction list() fonctionne correctement, je mémorise le répertoire courant avec la fonction getcwd(), puis le change en appelant chdir(directory) avant d'entrer dans la boucle while (naturellement, je reviens dans le répertoire initial une fois la travail effectué).

Les fonctions chdir() et getcwd() sont POSIX.1. Je crois qu'elles sont implantées par la bibliothèque fournie avec MinGW.

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
int list(char *directory, size_t sizeDir)
{
    DIR *dir = NULL;
    int err;
    struct dirent *file = NULL;
    struct stat infos;
    char cwd[32] = "";
 
    if ((dir = opendir(directory)) == NULL)
    {
        return EXIT_FAILURE;
    }
    printf("FOLDER %s:\n", directory);
 
    /* -tc- on memorise le repertoire courant de travail */
    if (getcwd(cwd, sizeof cwd) == NULL)
    {
        /* -tc- gestion des erreurs a ameliorer */
        return EXIT_FAILURE;
    }
    /* -tc- on change le repertoire de travail courant qui devient directory */
    if (chdir(directory) != 0)
    {
        /* -tc- idem: a ameliorer */
        return EXIT_FAILURE;
    }
 
    while ((file = readdir(dir)) != NULL)
    {
        /* -tc- ici, tu appelais deux fois stat() */
        err = stat(file->d_name, &infos);
        if (strcmp(file->d_name, ".") && strcmp(file->d_name, ".."))
        {
            if (err == 0)
            {
                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);
                }
 
            }
            else
            {
                printf ("'%s' IS UNKNOW\n", directory);
                chdir(cwd);
                return 1;
            }
        }
    }
 
    /* -tc- on retourne dans le repertoire de travail initial */
    chdir(cwd);
    printf("\nEND OF THE FOLDER: %s\n", directory);
 
    closedir(dir);
 
    (void) sizeDir;
 
    return EXIT_SUCCESS;
}

Juste une petite remarque au niveau de la gestion des erreurs dans la fonction list. Cette dernière retourne toujours EXIT_FAILURE lorsqu'une erreur intervient. Tu pourrais mettre au point un système un peu plus informatif... Par ailleurs, je suis un adepte de la programmation structurée dont un des principes fondamentaux est d'avoir un seul point de sortie par fonction. C'est une question de goût, je trouve que ça rend le code beaucoup plus facile à lire.

Thierry

**Thierry Chappuis** · 18/09/2007, 14h36

Envoyé par Emmanuel Delahaye

C'est pas utile...
... un coup de chdir(".."); et c'est OK.

Il y a sans doute quelque chose qui m'échappe. Pourquoi chdir("..") nous ramènerait dans le cwd initial. Celui-ci n'est pas nécessairement le répertoire parent, puisque directory est n'importe quel répertoire saisi par l'utilisateur. Je me trompe?

Thierry

**Emmanuel Delahaye** · 18/09/2007, 14h51

Envoyé par Thierry Chappuis

Il y a sans doute quelque chose qui m'échappe. Pourquoi chdir("..") nous ramènerait dans le cwd initial. Celui-ci n'est pas nécessairement le répertoire parent, puisque directory est n'importe quel répertoire saisi par l'utilisateur. Je me trompe?

Non, c'est moi. Je pensais que tu parlais du parcours de l'arborescence...

**Jean-Marc.Bourguet** · 18/09/2007, 14h52

Envoyé par Thierry Chappuis

Il y a sans doute quelque chose qui m'échappe. Pourquoi chdir("..") nous ramènerait dans le cwd initial. Celui-ci n'est pas nécessairement le répertoire parent, puisque directory est n'importe quel répertoire saisi par l'utilisateur. Je me trompe?

De plus, il faut faire attention aux liens symboliques (attention, experimenter avec un shell ne suffit pas: certains implementant cd .. de maniere a bien revenir quand on a suivit un lien symbolique) et ce meme pendant le parcours de l'arborescence.

**Thierry Chappuis** · 18/09/2007, 14h58

Envoyé par Jean-Marc.Bourguet

De plus, il faut faire attention aux liens symboliques (attention, experimenter avec un shell ne suffit pas: certains implementant cd .. de maniere a bien revenir quand on a suivit un lien symbolique) et ce meme pendant le parcours de l'arborescence.

Donc, même lors du parcours d'une arborescence, tu conseilles de sauvegarder le chemin du répertoire parent.

Thierry

**Jean-Marc.Bourguet** · 18/09/2007, 15h17

Envoyé par Thierry Chappuis

Donc, même lors du parcours d'une arborescence, tu conseilles de sauvegarder le chemin du répertoire parent.

Il faut le faire au moins quand on suit un lien symbolique (pour savoir si une entree est un lien symbolique, il faut utiliser lstat plutot que stat), ca me semble plus simple de le faire toujours.

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

Le problème c'est que je ne souhaite pas parcourir une arboressence. Je m'explique.

Avec ce que je veux faire :

L'utilisateur tape c:\monDoc et il voit :

[FOLDER] monDocument1
[FOLDER] monDocument2
[FILE] monFichier1.txt
[FOLDER] monDocument3

Et non pas :

[FOLDER] monDocument1
->monFichierX.x
->monFichierY.x
[FOLDER] monDocument2
->monFichierX.x
->monFichierY.txt
[FILE] monFichier1.txt
[FOLDER] monDocument3
->vide

PS: Je lis vos commentaires et je vais essayer de mon modifier mon code

**Thierry Chappuis** · 18/09/2007, 17h27

Envoyé par _SamSoft_

Le problème c'est que je ne souhaite pas parcourir une arboressence. Je m'explique.

Ca, je l'ai bien compris, mais les corrections que j'ai apporté à ta fonction list() sont nécessaires pour atteindre le but recherché. En effet, en l'état, lorsque tu lis une entrée de directory avec readdir() et que directory ne correspond pas au répertoire de travail courant, stat(file->d_name, &infos) renvoie toujours la valeur -1 indiquant qu'une erreur s'est produite.

Thierry

**_SamSoft_** · 18/09/2007, 18h02

Votre code ne marche pas: (c:\dev-cpp me sort rien) :

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
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
 
#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 list(char *directory, size_t sizeDir);
 
int main(void)
{
  int end;
  size_t sizeDir;
  char directory[256];
  sizeDir = sizeof directory;
 
 
   do
   {
      printf ("NAME OR <ENTER> FOR EXIT) : ");
      fgets (directory, sizeof directory, stdin); /* we take the path */
      fclean (directory, stdin);
      end = (directory[0] == '\0');
      if (!end)
      {
         list(directory, sizeDir);
      }
   }
   while (!end); //while end[0] != '\0'
   return 0;
}
int list(char *directory, size_t sizeDir)
{
    DIR *dir = NULL;
    int err;
    struct dirent *file = NULL;
    struct stat infos;
    char cwd[32] = "";
 
    if ((dir = opendir(directory)) == NULL)
    {
        return EXIT_FAILURE;
    }
    printf("FOLDER %s:\n", directory);
 
    /* -tc- on memorise le repertoire courant de travail */
    if (getcwd(cwd, sizeof cwd) == NULL)
    {
        /* -tc- gestion des erreurs a ameliorer */
        return EXIT_FAILURE;
    }
    /* -tc- on change le repertoire de travail courant qui devient directory */
    if (chdir(directory) != 0)
    {
        /* -tc- idem: a ameliorer */
        return EXIT_FAILURE;
    }
 
    while ((file = readdir(dir)) != NULL)
    {
        /* -tc- ici, tu appelais deux fois stat() */
        err = stat(file->d_name, &infos);
        if (strcmp(file->d_name, ".") && strcmp(file->d_name, ".."))
        {
            if (err == 0)
            {
                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);
                }
 
            }
            else
            {
                printf ("'%s' IS UNKNOW\n", directory);
                chdir(cwd);
                return 1;
            }
        }
    }
 
    /* -tc- on retourne dans le repertoire de travail initial */
    chdir(cwd);
    printf("\nEND OF THE FOLDER: %s\n", directory);
 
    closedir(dir);
 
    (void) sizeDir;
 
    return EXIT_SUCCESS;
}

**Thierry Chappuis** · 18/09/2007, 18h17

En ajoutant ce fichier d'en-tête:

Code :

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

#include <unistd.h>

ça fonctionne parfaitement chez moi. Je n'ai fais que quelques tests.

Thierry

**_SamSoft_** · 18/09/2007, 18h30

toujours rien -_-

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
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <sys/types.h>
#include <dirent.h>
#include <errno.h>
#include <sys/stat.h>
#include <unistd.h>
#include "myHeader.h"
int list(char *directory, size_t sizeDir);
 
int main(void)
{
  int end;
  size_t sizeDir;
  char directory[256];
  sizeDir = sizeof directory;
 
 
   do
   {
      printf ("NAME OR <ENTER> FOR EXIT) : ");
      fgets (directory, sizeof directory, stdin); /* we take the path */
      fclean (directory, stdin);
      end = (directory[0] == '\0');
      if (!end)
      {
         list(directory, sizeDir);
      }
   }
   while (!end); //while end[0] != '\0'
   return 0;
}
int list(char *directory, size_t sizeDir)
{
    DIR *dir = NULL;
    int err;
    struct dirent *file = NULL;
    struct stat infos;
    char cwd[32] = "";
 
    if ((dir = opendir(directory)) == NULL)
    {
        return EXIT_FAILURE;
    }
    printf("FOLDER %s:\n", directory);
 
    /* -tc- on memorise le repertoire courant de travail */
    if (getcwd(cwd, sizeof cwd) == NULL)
    {
        /* -tc- gestion des erreurs a ameliorer */
        return EXIT_FAILURE;
    }
    /* -tc- on change le repertoire de travail courant qui devient directory */
    if (chdir(directory) != 0)
    {
        /* -tc- idem: a ameliorer */
        return EXIT_FAILURE;
    }
 
    while ((file = readdir(dir)) != NULL)
    {
        /* -tc- ici, tu appelais deux fois stat() */
        err = stat(file->d_name, &infos);
        if (strcmp(file->d_name, ".") && strcmp(file->d_name, ".."))
        {
            if (err == 0)
            {
                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);
                }
 
            }
            else
            {
                printf ("'%s' IS UNKNOW\n", directory);
                chdir(cwd);
                return 1;
            }
        }
    }
 
    /* -tc- on retourne dans le repertoire de travail initial */
    chdir(cwd);
    printf("\nEND OF THE FOLDER: %s\n", directory);
 
    closedir(dir);
 
    (void) sizeDir;
 
    return EXIT_SUCCESS;
}