Discussion :

[sylvebarbe78]

Bonjour,

Afin de faire le décompte de fichiers présents dans un répertoire et ses sous répertoires, j'ai écrit une petite méthode qui parcours les dossiers en comptabilisant les fichiers qu'ils contiennent. Le problème est que cette méthode me renvoie un mauvais résultat.

En effet, après avoir parcouru toute l'arborescence des dossiers, il se passe quelque chose que je n'arrive pas à comprendre. En mode debug, je vois bien que la méthode arrive bien à l'instruction return avec le bon décompte de fichiers (sous-répertoires inclus), mais avant de quitter définitivement cette dernière pour me renvoyer ce bon décompte, elle refait un tour dans boucle foreach me renvoyant finalement le décompte de fichiers contenus uniquement dans le répertoire de départ !!?? :

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
 
 
        int filesCount = CountFilesRecursifDossier(@"c:\mondossier",0);
 
        public static int CountFilesRecursifDossier(string path, int result)
        {
            if (Directory.Exists(path))
            {
                result += Directory.GetFiles(path).Length;
 
                foreach (string directory in Directory.GetDirectories(path))
                    CountFilesRecursifDossier(directory, result);
            }
 
            return result;
        }

Quelqu'un pourrait-il m'expliquer pourquoi et me dire comment résoudre ce problème ?

Merci d'avance pour vos réponses.

[PixelJuice]

Bonjour ,

Le foreach que tu vois après le return est uniquement la fin de la toute première boucle.Enfin je crois.

Le problème c'est que result ne gardera toujours que la valeur de la première boucle car dans la suite de la récursivité , result ne sera modifié que dans la portée locale.Du coup ça sera le return de la première boucle qui aura le dernier mot.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
        public static int CountFilesRecursifDossier(string path, int result)
        {
            if (Directory.Exists(path))
            {
                result += Directory.GetFiles(path).Length;
 
                foreach (string directory in Directory.GetDirectories(path))
                   result = CountFilesRecursifDossier(directory, result);
            }
 
            return result;
        }

Ça marche car on mets a jour result , après c'est peut être un peu moins récursif mais ça devrait marcher.

EDIT : J'ai testé chez moi avec 3 niveaux de sous-dossiers et ça marche

[sylvebarbe78]

Super !

Effectivement ça fonctionne. CQFD !

Un grand merci à toi car je me voyais passer toute la nuit sur ce satané problème. Dans tous les cas je retiens la leçon, ça me servira pour la prochaine fois.