C# derived class type needed in base for logging using NLog

Bobby B picture Bobby B · Jan 3, 2012 · Viewed 7.1k times · Source

We're using NLog for logging in an C# MVC3 web application. All of our controllers extend a custom base "ApplicationController" that gives us access to a constantly needed methodes and s members.

I'd like all controllers to have access to the Logger via this base class, but want the detail of knowing what derived class the log statements originated in.

Our application controller looks like this:

public abstract class ApplicationController : Controller
{
    protected Logger _logger;
    protected virtual Logger Logger
    {
        get { return _logger ?? (_logger = LogManager.GetCurrentClassLogger()); }
    }

    protected ApplicationController()
    {
        Context = new Entities();
    }

If a derived controller doesn't override the Logger than all statements will show they originated from the Application controller. Currently, I've got essentially the same Logger statement in all derived controllers. For example:

public class PropertyController : ApplicationController
{
    private readonly DatatapeService _datatapeService;
    private readonly PropertyService _propertyService;
    protected override Logger Logger
    {
        get { return _logger ?? (_logger = LogManager.GetCurrentClassLogger()); }
    }

Obviously this is poor implementation practice.

  1. How can I dry this up? Specifically, what is my understanding of C# lacking to accomplish exactly this specific task?
  2. Is there a logging pattern that I should be following where I'm not exposing the logging class (NLog) directly?

TIA!

Answer

I am unfamiliar with NLog but in Log4Net the syntax LogManager.GetLogger(this.GetType()) will accomplish what you want. GetTypereturns the leaf type in your inheritance hierarchy, even if called in the base ApplicationController class, when the logger is first created (ie: on first access to the Logger property) it will instantiate it with type PropertyController