Discussion :

[Papy214]

Bonjour,

J'ai écris le code suivant:

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
 
sealed class FileSearch
{
 
    static void GetListFolder(System.Collections.Generic.List<string> files, string directory,
        string extension, bool recurse)
    {
        try
        {
            foreach (FileInfo fileInfo in new DirectoryInfo(directory).GetFiles())
            {
                if (fileInfo.Extension.Equals(extension))
                {
                    files.Add(fileInfo.FullName);
                }
            }
            if (recurse)
            {
                foreach (DirectoryInfo directoryInfo in new DirectoryInfo(directory).GetDirectories())
                {
                    GetListFolder(files, directoryInfo.FullName, extension, recurse);
                }
            }
        }
        catch (Exception)
        {
            // Silent
        }
    }
 
    public static System.Collections.Generic.List<string> GetFiles(string directory, string extension, bool recurse)
    {
        System.Collections.Generic.List<string> getFiles = new System.Collections.Generic.List<string>();
        GetListFolder(getFiles, directory, extension, recurse);
        return getFiles;
    }
 
}

ça fonctionne sans problème mais si je tente par exemple de chercher tous les fichiers .txt dans C:\ et ses sous-dossiers, c'est long ....

Comment je pourrais optimiser ça ?

Bien sur, une fois optimisé, je mettrai cette recherche dans un thread.

Papy !

[DarkHerumor]

Salut,

Il me semble que tu peux passer des paramètres de recherche à la méthode Directory.GetFiles() notamment une recherche sur le nom des fichiers.
Par contre je ne me souviens plus si il y a moyen de géré également la récursivité

Edit: En effet après avoir rapidement regardé il existe une méthode sur DirectoryInfo qui te retourne tout les fichiers avec un format de recherche et une option qui te permet de faire de la récursivité ou non => http://msdn.microsoft.com/en-us/library/ms143327.aspx

Edit 2: Ton soucis étant l'optimisation, je crains malheureusement que ça reste long... j'avais déjà dû faire une recherche sur tout un disque avec cette méthode (et en incluant les zip -_- ...). Ça peut être long voir même très long, mais je doute qu'il y ai un alternative.

[Papy214]

Oui, c'est ce que j'ai utilisé en premier mais ça pose un problème. Voir mon précédent post

C'est pour ça que j'ai écris ce code.

[sisqo60]

salut,
j'ai déjà utilisé ça et ça fonctionne très bien et en plus c'est assez rapide.

Code :

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

1
2
 
Directory.GetFiles("CheminDeMonDossier", "*.txt",SearchOption.AllDirectories);

Mais je crois qu'il se contente de chercher que dans les sous-répertoires mais pas dans les sous-sous-répertoires donc une récursive est certainement à prévoir...

[Papy214]

j'ai déjà utilisé ça et ça fonctionne très bien et en plus c'est assez rapide.

Oui merci mais à nouveau, ça ne fonctionne pas sur certains dossiers.
Pour l'exemple, essaye les 2 lignes suivantes:

Code :

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

1
2
3
 
            string[] list = System.IO.Directory.GetFiles("C:\\", "txt", System.IO.SearchOption.AllDirectories);
            listBox1.Items.AddRange(list);

[sisqo60]

c'est normal il te dis que "system volume information" n'est pas accessible parce que c'est un répertoire system et que tu 'as pas les droits d'acces!!!
maintanant toi essaye :

Code :

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

1
2
3
 
string[] list = System.IO.Directory.GetFiles("c:\\windows", "*.txt",SearchOption.AllDirectories);
     listBox1.Items.AddRange(list);