1

我正在写一个应用程序。我使用 NLog 进行日志记录。在这个应用程序中,几乎每个对象都可以写入日志。我为此定义了受保护的成员:

protected Logger logger;

protected virtual Logger Logger
{
     get { return logger ?? (logger = LogManager.GetLogger(this.GetType().ToString())); }
}

在这种情况下,我需要为应用程序中的每个基类复制/粘贴此代码。或者我看到其他选项:定义应用程序特定的根对象,其中包含记录器并将其子类化。但从语义上讲,这听起来是错误的,因为对我来说这不是真正的“是”情况。

有没有更好的选择?

4

2 回答 2

7

有时我真的希望 C# 能够支持多重继承或混合......

你可以写一个扩展方法:

public static Logger Logger(this object obj) {
    return LogManager.GetLogger(obj.GetType());
}

缺点是它会有点慢,因为创建的实例没有被缓存(除了在 NLog 内部,这是一个实现细节),但你可以自己做:

public static Logger Logger(this object obj) {
    Logger logger;
    Type type = obj.GetType();
    // s_loggers is static Dictionary<Type, Logger>
    if (!s_loggers.TryGetValue(type, out logger)) { // not in cache
        logger = LogManager.GetLogger(type);
        s_loggers[type] = logger;  // cache it
    }
    return logger;
}

你可以这样称呼它:

this.Logger.Log(...)

明显的缺点是任何对象都可以写入任何其他对象的记录器。

关于内存泄漏的评论(现已删除):

第一个实现解决了这个问题。然而,它并不是比任何静态对象更多的泄漏。如果您无法访问这些对象,那将是泄漏。作为替代方案,您可以缓存WeakReference到记录器而不是记录器本身,但我认为没有意义,因为我相信 NLog 本身已经有一些缓存。否则,NLog 总是必须为每种类型创建一个新的记录器实例。

于 2013-03-13T08:42:56.450 回答
2

我建议您使用静态记录器,以便获得每个类的记录器。这停止了​​为每个实例创建记录器的开销(记录器是线程安全的):

class MyClass
{
    static readonly Logger logger = LogManager.GetCurrentClassLogger();
}

GetCurrentClassLogger 将使您不必显式命名记录器,但缺点是额外的开销,因为它必须在运行时从堆栈跟踪中为您计算出记录器的名称。

在大多数情况下,这可能没什么大不了的,但除此之外,这样做会更快一些:

class MyClass
{
    static readonly Logger logger = LogManager.GetLogger("MyClass");
}

我会坚持使用 GetCurrentClassLogger 直到 / 除非轻微的开销成为您的解决方案中的一个问题。

我得到的印象是您正在尝试减少打字,和/或您正在解决(良好的)自然厌恶复制和粘贴重复代码的问题。

但是,这种每个类都有一个带有静态初始化程序的记录器的模式被广泛接受,并且在大多数情况下效果最好。您可能最好坚持这一点,并且可能设置一个代码片段来节省您的打字时间。

如果您仍然对此不满意,依赖注入(通过构造函数或属性注入)甚至面向方面的日志管理可能是您需要调查的其他事情。

于 2013-03-13T17:54:24.093 回答