Question

public class SimpleLogger
{
    static readonly string logFile = ConfigurationManager.AppSettings["LogFile"];

    static StreamWriter GetStream()
    {
        return File.Exists(logFile) ?
            File.AppendText(logFile) : File.CreateText(logFile);
    }

    public static void Write(string msg)
    {
        using (var sw = GetStream())
        {
            sw.Write(msg);
        }
    }
}

The above code fails in use as it doesn’t appear to be closing/disposing of the stream correctly. Subsequent writes give a ‘file in use’ IOException.

If the class is modified to use non-static methods, it appears to work correctly.

I don’t understand why there would be any behavioural difference?

Answer 1

The disposal is fine; GetStream delivers an open writer; Write closes/disposes it – sorted. if I had to guess, though, the issue is concurrent use – i.e. multiple threads (in particular in a web application) accessing the file at the same time. If that is the case, options:

• make Write (and any other access to the file) synchronized, so only one caller can possibly try to have the file open at once
• use a pre-canned logging framework that will already handle this scenario (common approaches here include synchronization, but also: buffering the data locally and then pushing the data down periodically – avoids opening the file over and over and over and over)

In particular; your only static state is the file path itself. There will therefore be no significant difference between using this as a static versus instance method.

As a side-note, File.AppendAllText may be useful here, but does not avoid the issue of concurrency.