Discussion :

[vladvad]

Hello

J'obtiens le message

le processus ne peut pas accéder au fichier car ce fichier es utilisé par un autre processus

lorsque j'essaie d'accéder à un fichier avec une commande du type File.Open(MonFichier) ...

Il semblerait que c'est lié au fait que mon fichier n'a pas été correctement fermé ...

Alors ma question est : comment fait-on pour fermer correctement un fichier ???

MERCI !

[olibara]

Code :

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

1
2
FileStream monstream=File.Open("Fichier",mode);
monstream.Close();

[vladvad]

Code :

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

1
2
FileStream monstream=File.Open("Fichier",mode);
monstream.Close();

Merci olibara mais ca ne marche pas !

[olibara]

Peux tu montrer to code ?

[StormimOn]

Le fichier que tu cherches à ouvrir, est-il ouvert ailleurs dans ton code ? A ce moment il faudrait le fermer avant (si possible dès qu'il n'est plus utile).

Ou bien est-il ouvert par une application externe ? Dans ce dernier cas il est alors possible qu'il soit ouvert en accès exclusif auquel cas tu ne pourras pas l'ouvrir.

[vladvad]

Peux tu montrer to code ?

Ca ressemble à peu près à celà :

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
 
              FileStream fs = new FileStream(fileName, FileMode.Create);
                BinaryWriter BW = new BinaryWriter(fs);
                BW.Write(DateHeureCreation);  
                BW.Write(Form1.VersionSoft); 
                BW.Write(this.N);
                BW.Write(this.Gauss.g1); 
                BW.Write(this.Gauss.g2);
            ....
                BW.Write(this.Monde.reservoirs);
                BW.Write(this.Monde.max_time);
                BW.Write(this.Monde.objec_perfo);
                BW.Write(this.Monde.objec_complex);
 
                BW.Close();
                fs.Close();

Cette séquence peut être appelée plusieurs fois consécutivement. La première fois, no problem, mais ensuite ça plante !


Merci

[olibara]

Salut

Je ne vois pas de probleme fondamental dans ton code
Mais comme le suggere Stormimonn il faudrait peut etre vérifier dans l'environnement général les acces possible a ce fichier !

[vladvad]

Le fichier que tu cherches à ouvrir, est-il ouvert ailleurs dans ton code ? A ce moment il faudrait le fermer avant (si possible dès qu'il n'est plus utile).

Ou bien est-il ouvert par une application externe ? Dans ce dernier cas il est alors possible qu'il soit ouvert en accès exclusif auquel cas tu ne pourras pas l'ouvrir.

Hello

Je ne pense pas appeler ce fichier ailleurs que dans ces lignes de code ...

Un autre phénomène étrange : ce pb n'apparait pas sur toutes les machines ...

[olibara]

Et quand le problème survient ?
C'est systematiquement au deuxieme appel ?
C'est aléatoire ?
Peux tu le reproduire en step by step a la main ?

[vladvad]

Et quand le problème survient ?
C'est systematiquement au deuxieme appel ?

C'est en effet quasi systématiquement au deuxieme appel

Peux tu le reproduire en step by step a la main ?

Oui et j'ai le message

le processus ne peut pas accéder au fichier car ce fichier es utilisé par un autre processus

au niveau de

Code :

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

FileStream fs = new FileStream(fileName, FileMode.Create);

[MetalGeek]

Salut,
une bonne pratique est d'utiliser la syntaxe avec le mot-clé 'using' pour les objets qui implémentent IDisposable (tels que les stream).
Ex. :

Code :

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

1
2
3
4
5
6
 
using (var fs = new FileStream(@"C:\toto\tata.dat"))
{
    //code...
}
//Quoi qu'il arrive, 'fs.Dispose()' est appelé et donc le flux est libéré.

[olibara]

Salut

Metalgeek a raison

Mais ici ce n'est pas un problème de dispose de l'objet mais de collision d'acces au fichier qui devrait avoir été fermé donc accessible

Mais j'imagine que le step by step tu le fait en mettant un breakpoint quelque part
A mon avis un autre process ou thread accede ce fichier

Que se passe-t-il si tu reouvre le fichier imédiatement apres fermeture

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
FileStream fs = new FileStream(fileName, FileMode.Create);
                BinaryWriter BW = new BinaryWriter(fs);
                BW.Write(DateHeureCreation);  
                BW.Write(Form1.VersionSoft); 
                BW.Write(this.N);
                BW.Write(this.Gauss.g1); 
                BW.Write(this.Gauss.g2);
            ....
                BW.Write(this.Monde.reservoirs);
                BW.Write(this.Monde.max_time);
                BW.Write(this.Monde.objec_perfo);
                BW.Write(this.Monde.objec_complex);
 
                BW.Close();
                fs.Close();
 
                fs = new FileStream(fileName, FileMode.Create);
                BW = new BinaryWriter(fs);
                BW.Write(DateHeureCreation);  
                etc...

[vladvad]

Salut

Metalgeek a raison

Mais ici ce n'est pas un problème de dispose de l'objet mais de collision d'acces au fichier qui devrait avoir été fermé donc accessible

Mais j'imagine que le step by step tu le fait en mettant un breakpoint quelque part
A mon avis un autre process ou thread accede ce fichier

Que se passe-t-il si tu reouvre le fichier imédiatement apres fermeture

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
FileStream fs = new FileStream(fileName, FileMode.Create);
                BinaryWriter BW = new BinaryWriter(fs);
                BW.Write(DateHeureCreation);  
                BW.Write(Form1.VersionSoft); 
                BW.Write(this.N);
                BW.Write(this.Gauss.g1); 
                BW.Write(this.Gauss.g2);
            ....
                BW.Write(this.Monde.reservoirs);
                BW.Write(this.Monde.max_time);
                BW.Write(this.Monde.objec_perfo);
                BW.Write(this.Monde.objec_complex);
 
                BW.Close();
                fs.Close();
 
                fs = new FileStream(fileName, FileMode.Create);
                BW = new BinaryWriter(fs);
                BW.Write(DateHeureCreation);  
                etc...

Hello olibra,

Si je reouvre le fichier imédiatement apres fermeture (en rajoutant ton code), ça ne plante pas !

Par contre, ça replentera comme d'hab au second passage ...

[olibara]

Donc ca signifie bien que tu dois chercher ailleurs ce qui peut se passer !

Ou que dans les .... que tu a représenté il y a une condition de sortie qui evite la fermeture

[vladvad]

Bon ça y est, je crois avoir trouvé le HIC ...

J'utilise à un endroit des instructions try / catch et il s'y produit un plantage qui conduit à passer au code dans catch{ ...} sans exécuter la fin du code dans try{ ...}. Or c'est dans la fin de ce code que se trouve un fs.Close() qui n'est donc jamais exécuté et donc un fichier n'est jamais fermé ...

(j'espère que c'est assez clair comme explication ... )

[olibara]

C'est tout a fait clair

Et c'est bien ce qu'on t'avait suggéré !

Ou que dans les .... que tu a représenté il y a une condition de sortie qui evite la fermeture

[StormimOn]

J'utilise à un endroit des instructions try / catch et il s'y produit un plantage qui conduit à passer au code dans catch{ ...} sans exécuter la fin du code dans try{ ...}. Or c'est dans la fin de ce code que se trouve un fs.Close() qui n'est donc jamais exécuté et donc un fichier n'est jamais fermé ...

Pour la libération des ressources (appel de Dispose) il faut utiliser soit un bloc try/catch/finally soit un bloc using. Sinon ce genre de problème arrive.

[olibara]

Pour la libération des ressources (appel de Dispose) il faut utiliser soit un bloc try/catch/finally soit un bloc using. Sinon ce genre de problème arrive.

+1
Mais aucun garde fou ni protection ne dispense de regarder attentivement la logique de ce que l'on fait.
Un bon air-bag ne permet pas de ne plus regarder la route !

[vladvad]

Moi qui n'utilisais jamais de finally { } , je vais revoir ma façon de faire ...

Merci à tous pour votre aide

Je classe donc ce sujet dans "Résolu" ...