Discussion :

[mithrendil]

Bonjour, j'ai un petit projet pour lequel je dois vérifier la présence d'au moins un fichier portant l'extension ".map" dans un répertoire donné par l'utilisateur. Voici mon 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
private void button3_Click(object sender, EventArgs e)
        {
            string[] files;
            bool map_exists = false;
            folderBrowserDialog1.ShowDialog();
            files = Directory.GetFileSystemEntries(folderBrowserDialog1.SelectedPath);
            int filecount = files.GetUpperBound(0) + 1;
            for (int i=0; i<filecount; i++)
            {
                File temp = new File (files[i]);
                if (temp.Extension.ToString("map"))
                {
                    map_exists = true;
                }
            }
            if (!map_exists)
            {
                textBox3.Text = "Aucune map dans ce répertoire!";
            }
            else{
                textBox3.Text = folderBrowserDialog1.SelectedPath;
            }
        }

La compilation me renvoit l'erreur suivante:

Erreur 1 Impossible de déclarer une variable de type static 'System.IO.File' D:\Documents and Settings\Admin\Local Settings\Application Data\Temporary Projects\mbuilder\Form1.cs 64 17 mbuilder

Pourquoi ne puis je pas instancier un objet "File"? Merci de votre aide.

[ced600]

File est une classe static, comme te le dit ton erreur, et donc par définition tu ne peux pas l'instancier.

Pour créer un fichier tu peux utiliser la méthode Create ou CreateText qui te retourne un StreamWriter, buffer qui encapsule ton fichier nouvellement créé, et que tu dois utiliser pour écrire des données dans le fichier.

Si tu veux créer un fichier et obtenir des informations du fichier comme l'extension, tu doit créer un Objet FileInfo. Alors à partir de celui-ci tu pourra créer le fichier, et obtenir des informations sur son extension, ....

Et sinon, je te conseille d'utiliser des Foreach lorsque tu parcours entièrement un tableau, ou une collection, ...., c'est plus simple et tout aussi efficace.

[smyley]

moi je dirai :

Code :

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

1
2
3
4
5
6
7
8
9
 
public virtual bool MapExists(string directory)
{
 DirectoryInfo info = new DirectoryInfo(directory);
 if(!info.Exists)
   return false;
 else
  return info.GetFiles("*.map").Length > 0; //Length ou Count, je fait de tête ^_^
}

[mithrendil]

Merci pour vos réponses, j'ai résolu le problème en utilisant un objet fileinfo. Mais j'ai un peu peur que le tableu ainsi que la boucle chargent un peu trop le programme en mémoire.

[Maniak]

smyley n'est pas loin, mais il y a encore plus bateau :

Code :

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

1
2
string path = folderBrowserDialog1.SelectedPath;
bool exists = Directory.GetFiles( path, "*.map" ).Length > 0;

C'est quand même un poil plus simple/clair/court non ? :)

[mithrendil]

C'est vrai c'est bien plus léger comme ça. Merci!

[smyley]

sC'est quand même un poil plus simple/clair/court non ?

Hihi, je préfère toujours vérifier l'existence des choses que je touche, tic que j'ai prit en ayant marre d'avoir des crash car un élément n'existait pas