Enregistrement de flux XML en C# dit qu'il est utilisé par un autre processus

Question

J'ai une bibliothèque open-source this que j'ai du mal à corriger un problème ... Cette bibliothèque permet de créer facilement un fichier XML pour stocker les paramètres de l'application. Mais j'ai un problème en enregistrant les changements.

J'ai une autre application où j'utilise cette bibliothèque et chaque fois que la fenêtre de l'application est redimensionnée, j'appelle la méthode Save() de la bibliothèque pour sauvegarder la taille/position de la fenêtre dans le fichier XML.

La plupart du temps cela fonctionne très bien, tout est sauvegardé. De temps en temps cependant, je reçois une exception disant que le fichier est utilisé par un autre processus.

J'ai vraiment besoin de m'assurer que les modifications sont sauvegardées à chaque fois que la méthode Save() est appelée, j'ai besoin de gérer cette exception ou de l'empêcher de se produire.

Que suggérez-vous pour mieux gérer cette situation?

Le code de la méthode Save() est la suivante:

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

Il se peut que les événements de redimensionnement de la fenêtre se déclenchent si rapidement que la fonction de sauvegarde est appelée, qu'elle est appelée à nouveau avant la fin de la première exécution. Cela entraînerait l'erreur que vous décrivez (l'autre processus utilisant le fichier est ... VOUS!). Try surrounding your code with a lock, thusly:

lock(some_shared_object) 
{ 
    //Your code here 
} 

Answer 2

XmlWriter est IDisposable. Vous devriez l'inclure dans une clause using(). http://msdn.microsoft.com/en-us/library/system.xml.xmlwriter.aspx

Answer 3

Vous pouvez également essayer d'utiliser une instruction lock. Il se pourrait que les méthodes se chevauchent les unes les autres.

Answer 4

Je dois aller avec une combinaison des réponses déjà données ici.

Votre XmlWriter doit être dans un bloc using pour plusieurs raisons. Vous devez le disposer pour que vos ressources soient libérées le plus rapidement possible. En outre, que faire si vous lancez une exception tout en interagissant avec elle. Le fichier ne sera pas fermé correctement, au moins jusqu'à ce que le finaliseur entre en jeu et libère vos ressources.

Même avec l'instruction using, vous "pourriez" avoir une contention sur le fichier et devez placer le code Save dans une instruction lock. La méthode est non réentrante par nature car le fichier est une ressource partagée. Mettre un verrou autour peut être trop long si vous n'avez pas plusieurs threads, mais vous devez vous assurer que vous avez correctement contrôlé l'accès au fichier.

L'autre chose à prendre en compte est que vous pouvez déplacer l'opération de sauvegarde vers un thread d'arrière-plan pour écrire le fichier. Si vous obtenez un fichier de paramètres volumineux, vous risquez de provoquer des interactions d'interface utilisateur étranges car vous attendez que le fichier soit écrit chaque fois que l'utilisateur redimensionne et que cela se produise dans le thread d'interface utilisateur. Si vous l'avez fait, vous devrez certainement verrouiller l'accès à la ressource de fichier.