Kann ich statische Methoden für eine statische Klasseninstanz von einem Worker-Thread aufrufen?

Question

Ich verwende ein System.Threading.ThreadPool, um eine Warteschlange von Jobs von einem Dienst zu verwalten. Ich habe bereits umgesetzt wie diese Anmeldung ...

abstract class Global 
{ 
    public static LogFile LogFile = null; 
} 

public class LogFile : IDisposable 
{ 
    private StreamWriter sw; 
    public LogFile(string path){} 
    public void WriteEntry(string logText) 
    { 
     lock (sw) 
     { 
      sw.WriteLine(logText); 
     } 
    } 
} 

ich das Protokoll bei Dienststart erstellen möchten, und verwenden Sie es aus meiner Warteschlange Worker-Threads .. so etwas wie dieses ...

//On Service Start 
Global.LogFile = new LogFile("log.txt"); 

//Kick of worker thread 
ThreadPool.QueueUserWorkItem(objWrkrThread.CallbackMethod, iCount); 

//Worker thread logs an entry in CallbackMethod() 
Global.LogFile.WriteEntry("Hello World"); 

Ist das sicher? Wird das Aufrufen einer Methode für eine statische Instanz einer Klasse versehentlich meine Threads "synchronisieren" oder "blockieren"?

Michael

Answer 1

Nichts wird 'synchronisieren' oder 'blockieren', es sei denn, Sie schreiben Code in Ihre Methode. Es spielt keine Rolle, ob es sich um eine Instanzmethode oder eine statische Methode handelt.

Standardmäßig wird WriteEntry keine Aufrufe von Ihren Threads blockieren, aber es könnte sehr gut Datei beschädigen, wenn Sie nicht den Code für mehrere gleichzeitige Aufrufe schreiben.

Lesen Sie mehr zu diesem Thema hier:

Are static methods thread safe

Answer 2

Es ist nicht sicher mehrere Threads nennen Writeentry zur gleichen Zeit haben, es sei denn, es wurde entwickelt, um sicher zu sein.

Answer 3

Was Sie versuchen, für eine Singleton-Klasse klingt wie der perfekte Kandidat zu tun. Ich weiß, dass es einen schlechten Wickel bekommt, aber manchmal ist es die Einfachheit wert.

Sie können eine Protokollklasse wie diese erstellen und Sie sollten threadsicher sein.

public sealed class Log 
{ 
    static Log instance=null; 
    static readonly object lockObject = new object(); 
    static string path = "log.txt"; 

    Log() 
    { 
    } 

    public static Log Instance 
    { 
     get 
     { 
      lock (lockObject) 
      { 
       if (instance==null) 
       { 
        instance = new Log(); 
       } 
       return instance; 
      } 
     } 
    } 

    public void WriteLine(string message) 
    { 
     lock(lockObject) 
     { 
      using(StreamWriter sw = new StreamWriter(File.Open(path, FileMode.Append))) 
      { 
      sw.WriteLine(message); 
      } 
     } 
    } 
} 

Dann in Ihrem Code, rufen Sie es einfach so:

Log executionLog = Log.Instance; 
executionLog.WriteLine("this is a log message."); 

Sie können die Datei auch in ähnlichen Thread sichere Methoden verwalten Öffnen des über Kopf der Öffnung, um loszuwerden, und Schließen der Datei jeden Schreibvorgang.