Comment gérer les accès simultanés aux fichiers avec un flux de fichiers/flux de données?

Question

Je suis en train d'écrire un fichier d'audit qui écrit le nom d'utilisateur, l'heure, et l'ancienne/modifié les valeurs de plusieurs variables dans l'application pour chaque utilisateur quand ils utilisent mon application. Il utilise un FileStream et StreamWriter pour accéder au fichier d'audit. Toutes les vérifications pour chaque utilisateur seront écrites dans le même fichier.

Le problème est que lorsque deux utilisateurs mettent à jour ce fichier d'audit en même temps, l '"ancienne valeur" de chaque variable est mélangée entre les utilisateurs. Pourquoi est-ce et comment pouvez-vous résoudre le problème de concurrence ici?

Une partie du code, raccourci par souci de concision ...

Dim fs As FileStream 
Dim w As StreamWriter 

Public Sub WriteAudit(ByVal filename As String, ByVal username As String, ByVal oldAddress As String, ByVal newAddress As String, ByVal oldCity As String, ByVal newCity As String) 
    Dim now As DateTime = DateTime.Now 
    Dim audit As String = "" 
    audit += now + "," + username + "," + oldAddress + "," + newAddress + "," + oldCity + "," + newCity 

    fs = New FileStream(filename, FileMode.Append) 
    w = New StreamWriter(fs) 
    w.WriteLine(audit) 
    w.Close() 
    fs.Close() 
End Sub 

Ce vit dans une classe AuditLogger, qui est référencé par une variable d'instance (réaffectés à chaque fois que la fonction est accessible).

Answer 1

deux façons:

Tout d'abord, utilisez une base de données partagée et non un fichier. Même une base de données d'accès simple peut gérer plusieurs utilisateurs beaucoup plus gracieusement qu'un fichier sur le disque. Deuxièmement, pouvez-vous utiliser un fichier distinct pour chaque utilisateur? peut-être le stocker dans leur dossier% APPDATA%? Ou peut-être sur un partage réseau quelque part?

Answer 2

Vous pouvez essayer ceci:

TextWriter tw = TextWriter.Synchronized(File.AppendText(filePath)); 

La méthode File.AppendText() retourne un objet StreamWriter, que la méthode TextWriter.Synchronized() enveloppe pour créer un TextWriter thread-safe qui peut être utilisé comme un StreamWriter.

Answer 3

Refactorisez l'application afin de ne pas avoir à créer une nouvelle instance de la classe AuditLogger à chaque fois. Utilisez le singleton pattern directement, ou un cadre dependency-injection pour utiliser la même instance dans toute l'application. De là, la mise en œuvre est beaucoup plus facile: entourez les opérations d'écriture avec lock instructions, ou utilisez le TextWriter.Synchronized comme cela a été mentionné dans la réponse de Robert.

Ce poste peut être pertinent:

• Dependency injection and logging interfaces