How NOT to write a logger
time to read 1 min | 173 words

I am going over some library code at the moment, and I am following my usual routine of starting at the basic infrastructure to find the fault lines. Take a look at this:

public void Log(string pSource, string pMessage, EventLogEntryType pEntryType) {
    try {
        if (!EventLog.SourceExists(pSource)) {
            EventLog.CreateEventSource(pSource, "Application");
        }

        EventLog.WriteEntry(pSource, pMessage, pEntryType);
    }
    catch (Exception _ex) {
        Log("", _ex.ToString(), EventLogEntryType.Error);
    }
}
This is… impressive.
0 comments

Related posts that you may find interesting:

27 Aug 2013 Building async Unit of Work with MVC 4
raven development
15 Feb 2013 Rhino ETL Union Operation
rhino etl
18 Jun 2013 TPL and the case of the !@#(*@! hung process
raven bugs