Unicité des dlls en mémoire.

Version imprimable

01/06/2012, 10h38
bleuerouge

Unicité des dlls en mémoire.

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
01/06/2012, 10h51
tomlev

Citation:

Envoyé par bleuerouge

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...

Citation:

Envoyé par tomlev

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:

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.

01/06/2012, 15h08
tomlev

FluxFichier est une variable locale ou un champ de la classe ? Si c'est un champ ça peut effectivement poser des problèmes...
01/06/2012, 17h25
bleuerouge
Citation:

Envoyé par tomlev

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

1 2 public StreamWriter FluxFichier;
Un champ public de ma classe
01/06/2012, 20h19
tomlev
Citation:
Envoyé par bleuerouge

Code:

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...

Bonjour,
J'ai remplacer mon code par

Code:

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...

04/06/2012, 14h27
antoine.debyser

Bonjour,

Et bien dans ce cas, il semble que tu ai plusieurs instance de ta classe.
04/06/2012, 14h58
tomlev

Tu dois sans doute ouvrir le même fichier à un autre endroit et oublier de le refermer...
05/06/2012, 16h48
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.