Question

I am writing a C# application.
I have (a kind of) logging class. and this logging class will be used by many threads.
How to make this class thread safe?
Should I make it as singleton?
what are the best practices there?
Is there a document that I can read about how to make it thread-safe?

Thanks

Answer 1

In C#, any object can be used to protect a "critical section", or in other words, code that must not be executed by two threads at the same time.

For example, the following will synchronize access to the SharedLogger.Write method, so only a single thread is logging a message at any given time.

    public class SharedLogger : ILogger
    {
       public static SharedLogger Instance = new SharedLogger();
       
       public void Write(string s)
       {
          lock (_lock)
          {
             _writer.Write(s);
          }
       }
    
       private SharedLogger() 
       { 
          _writer = new LogWriter();
       }
       
       private object _lock = new object();
       private LogWriter _writer;
    }