Speichern XML-Stream in C# sagt, dass es von einem anderen Prozess verwendet wird

Question

Ich habe this Open-Source-Bibliothek, die ich habe einige Probleme bei der Behebung eines Problems ... Diese Bibliothek ermöglicht auf einfache Weise eine XML-Datei zum Speichern von Anwendungseinstellungen erstellen. Aber ich habe ein Problem beim Speichern der Änderungen.

Ich habe eine andere Anwendung, wo ich diese Bibliothek verwende und jedes Mal, wenn das Anwendungsfenster die Größenänderung vorgenommen wird, rufe ich die Save() -Methode der Bibliothek auf, um die Fenstergröße/-position in der XML-Datei zu speichern.

Die meiste Zeit funktioniert es gut, alles ist gespeichert. Hin und wieder bekomme ich eine Ausnahme, dass die Datei von einem anderen Prozess verwendet wird.

Ich muss wirklich sicherstellen, dass Änderungen jedes Mal gespeichert werden, wenn die Save() -Methode aufgerufen wird, ich muss diese Ausnahme irgendwie behandeln oder verhindern, dass es passiert.

Was sind Ihre Vorschläge für den besten Umgang mit dieser Situation?

Der Code für die Save() Methode ist folgende:

public void Save() { 
    // Create a new XML file if there's no root element 
    if(xDocument.DocumentElement == null) { 
     xDocument = new XmlDocument(); 
     xDocument.LoadXml("<?xml version=\"1.0\" encoding=\"utf-8\" ?>\n" + 
      "<" + XmlRootElement + ">\n</" + XmlRootElement + ">"); 
    } 

    // OMITTED CODE WAS HERE (NOT IMPORTANT FOR THE PROBLEM) 

    // Create a new XML writer for the XML file 
    XmlWriter xWriter = XmlWriter.Create(XmlFilePath, new XmlWriterSettings() { 
     Indent = true, 
     IndentChars = "\t" 
    }); 

    // Sort the XML file using the XSL sylesheet and save it 
    xslTransform.Transform(xDocument, xWriter); 

    // Clear the buffer and close the XML writer stream 
    xWriter.Flush(); 
    xWriter.Close(); 
} 

Answer 1

Es könnte der Fall sein, dass die Fenster-Resize-abgeschlossen-Ereignisse so schnell feuern, dass die Speicherfunktion aufgerufen wird, die erneut aufgerufen wird, bevor es das erste Mal ausgeführt wird. Dies würde zu dem Fehler führen, den Sie beschreiben (der andere Prozess, der die Datei verwendet, ist ... SIE!). Try surrounding your code with a lock, thusly:

lock(some_shared_object) 
{ 
    //Your code here 
} 

Answer 2

XmlWriter ist IDisposable. Sie sollten es in eine using() -Klausel einfügen. http://msdn.microsoft.com/en-us/library/system.xml.xmlwriter.aspx

Answer 3

Auch Sie könnten versuchen, eine Lock-Anweisung zu verwenden. Es könnte sein, dass sich die Methoden gegenseitig überholen.

Answer 4

Ich habe mit einer Kombination der Antworten gehen schon hier gegeben.

Ihr XmlWriter sollte aus verschiedenen Gründen in einem Verwendungsblock sein. Sie sollten es so entsorgen, dass Ihre Ressourcen so schnell wie möglich freigegeben werden. Was passiert auch, wenn Sie beim Interagieren eine Ausnahme auslösen? Die Datei wird nicht ordnungsgemäß geschlossen, bis der Finalizer startet und Ihre Ressourcen freigibt.

Auch mit der Anweisung using, haben Sie möglicherweise Konflikte in der Datei und müssen den Code speichern in eine lock-Anweisung. Die Methode ist von Natur aus nicht reentrant, da die Datei eine gemeinsam genutzte Ressource ist. Wenn Sie mehrere Threads verwenden, kann es sein, dass die Sperre aufgehoben wird. Sie können jedoch sicherstellen, dass Sie den Zugriff auf die Datei ordnungsgemäß steuern.

Die andere Sache, die Sie beachten sollten, ist, dass Sie den Speichervorgang möglicherweise in einen Hintergrundthread verschieben möchten, um die Datei zu schreiben. Wenn Sie eine große Einstellungsdatei erhalten, können Sie seltsame UI-Interaktionen verursachen, da Sie auf die Datei warten müssen, um jedes Mal zu schreiben, wenn der Benutzer die Größe ändert und dies im UI-Thread geschieht. Wenn Sie dies tun, müssen Sie den Zugriff auf die Dateiressource definitiv sperren.