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

[Papy214]

ça marche très bien sur ton exemple, je suis d'accord, mais ça veut dire que si je veux faire une recherche sur tous les dossiers et sous-dossiers de C: , je ne peux pas. GetFiles ne résoud donc pas mon problème général

[sisqo60]

dans ce cas là tu peux utiliser Directory.getfiles mais seulement sur les répertoires non system.

Code :

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

1
2
3
4
5
6
7
 
DirectoryInfo directory = new directoryinfo(path);
if (diectory.Attributes != FileAttributes.System)
{
    //liste en ref de type List<String>
    liste.addRange(Directory.getfiles(path,string.Format("*.{0}",extension),SearchOption.TopDirectoryOnly));
}

en récursif bien sur...
sauf si tu veux balayer aussi les répertoires system...Dans ce cas tu essaye et tu met un try/catch silencieux pour les répertoires qui provoquent cette erreur

[DarkHerumor]

Je pense que la meilleure solution est celle de sisqo60 (je n'avais pas fait le rapprochement avec ton précédent post dsl...)

Par contre, comme je le disais déjà avant, si tu balaye tout le disque c:, attend toi a ce que ce soit long quoi qu'il arrive

[sisqo60]

merci DarkHerumor...

[sisqo60]

voila le 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
 
public static void ListFichiers(string sourceDir, ref List<string> files) 
{
    DirectoryInfo dir = new DirectoryInfo(sourceDir);
     if (dir.Exists) 
    {
 
        try
 
        {
            foreach (string d in Directory.GetDirectories(sourceDir))
                ListDir(d, ref files); 
        }
        catch (/*UnauthorizedAccessException*/)
        {
           //gérer ou non l'exception...
        } 
        foreach (string file in Directory.GetFiles(sourceDir)) 
            files.Add(file);
    }
}

A appeler comme ça :

Code :

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

1
2
3
 
List<string> files = newList<string>();
ListDir(@"LeChemin", ref files);

[Papy214]

Merci à tous ...

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