Discussion :

[bleuerouge]

Bonjour,
J'ai développé une assembly qui logue des informations que lui envoie des applications.
Or dans certains cas, la dll provoque une exception afin d'indiquer que le fichier est utilisé par un autre processus, or il n'y a que ma dll qui l'utilise et elle se trouve dans le GAC ce qui pour moi voudrait dire qu'elle se trouve une seul fois en mémoire non ? J'ai bien rajouté un mutex mais rien à faire.

Y a t il un moyen afin de s'assurer que ma dll soit instancier un fois et utilisé par plusieurs instances d'application ?

Merci d'avance

[tomlev]

Or dans certains cas, la dll provoque une exception afin d'indiquer que le fichier est utilisé par un autre processus, or il n'y a que ma dll qui l'utilise

ça n'a rien à voir avec le fait qu'elle soit dans le GAC ou qu'elle soit une ou plusieurs fois en mémoire... C'est une question de process, pas de DLL. Même si la DLL est une seule fois en mémoire, si plusieurs process l'utilisent en même temps, tu peux rencontrer le problème.

En fait, le message qui dit que le fichier est utilisé par un "autre" processus est trompeur, car en fait ça peut très bien être le même process (et a priori c'est le cas). Tu as probablement déjà ouvert le fichier et oublié de le fermer, donc quand tu essaies de l'ouvrir à nouveau, ça pète...

[bleuerouge]

En fait, le message qui dit que le fichier est utilisé par un "autre" processus est trompeur, car en fait ça peut très bien être le même process (et a priori c'est le cas). Tu as probablement déjà ouvert le fichier et oublié de le fermer, donc quand tu essaies de l'ouvrir à nouveau, ça pète...

Voici le code qui pose problème :

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
 
 
try
{
   FluxFichier = new StreamWriter(Chemin, true);
 
   FluxFichier.WriteLine(monlog);
 
   FluxFichier.Flush();
}
catch (Exception ex)
{
Trace.TraceError("LogFile.Trace_Insert - Erreur " + ex.Message);
result = -1;
throw new Exception("LogFile - Impossible d'inscrire dans le fichier :" + Chemin, ex);
}
finally
{
FluxFichier.Close();
FluxFichier.Dispose();
}

Mon fichier est systématique clos et à la fin de l'écriture d'un log. J'ai rajouté un lock(this), ça ne fonctionne pas mieux.

[tomlev]

FluxFichier est une variable locale ou un champ de la classe ? Si c'est un champ ça peut effectivement poser des problèmes...

[bleuerouge]

FluxFichier est une variable locale ou un champ de la classe ? Si c'est un champ ça peut effectivement poser des problèmes...

Code :

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

1
2
 
public StreamWriter FluxFichier;

Un champ public de ma classe

[tomlev]

Code :

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

1
2
 
public StreamWriter FluxFichier;

Un champ public de ma classe

Très mauvaise idée, et même doublement...

D'une part, les champs publics, c'est mal ; c'est une violation de l'encapsulation, ça permet à n'importe quel code externe à ta classe de modifier sans aucun contrôle l'état interne de la classe.

D'autre part, cette variable n'a pas besoin d'être un champ (tu pourrais obtenir le même résultat avec une variable locale), et le fait que ce soit un champ va poser des problèmes si ta méthode est appelée en même temps à partir de différents threads, car les 2 threads vont partager la même variable et risquent de se "marcher sur les pieds". Je ne sais pas si c'est la cause de ton problème, mais c'est une explication plausible...

[bleuerouge]

Bonjour,
J'ai remplacer mon code par

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
 
 
 lock (this)
            {
                StreamWriter FluxFichier = null;
                try
                {
                    FluxFichier = new StreamWriter(Chemin, true);
 
                    FluxFichier.WriteLine(monlog);
 
                    FluxFichier.Flush();
                }
                catch (Exception ex)
                {
                    Trace.TraceError("LogFile.Trace_Insert - Erreur " + ex.Message);
                    result = -1;
                    throw new Exception("LogFile - Impossible d'inscrire dans le fichier :" + Chemin, ex);
                }
                finally
                {
                    FluxFichier.Close();
                    FluxFichier.Dispose();
                }
            }

Ca me marche toujours pas mieux...

[antoine.debyser]

Bonjour,

Et bien dans ce cas, il semble que tu ai plusieurs instance de ta classe.

[tomlev]

Tu dois sans doute ouvrir le même fichier à un autre endroit et oublier de le refermer...

[brnf1]

Bonjour,

Peut être que le problème ne vient ni de la dll ni des processus.
Il me semble que lors de la fermeture du stream, windows met parfois un petit temps avant de relâcher le jeton du fichier au niveau du system d'exploitation.
Du coup, si deux opérations se font de manière très rapprochées ça peut venir de la.