c# - 静态工厂 - 好的做法吗？

Question

我有一个静态日志管理器类，它应该根据参数返回所需记录器的实例。

public static class LogManager {

    private static ILoggerFactory Factory { ... }

    public static ILogger GetLogger(string name) {
        return Factory.Create(name);
    }

}

由于必须设置 ILoggerFactory 的具体实现(初始化)，我想知道是否应该只保留实现类型的引用并在每次请求工厂时返回一个新实例 - 或者是否可以保留实现实例的静态引用。

版本 1

private static Type _factoryType;

public static void SetFactory<TFactory>()
    where TFactory : ILoggerFactory, new()
{
    _factoryType = typeof(TFactory);
}

private static Factory {
    get { return (ILoggerFactory)Activator.CreateInstance(_factoryType);
}

public static ILogger GetLogger(string name) {
    return Factory.Create(name);
}

版本 2

private static ILoggerFactory _loggerFactory;

public static void SetFactory<TFactory>()
    where TFactory : ILoggerFactory, new()
{
    _loggerFactory = (ILoggerFactory)Activator.CreateInstance<TFactory();
}

public static ILogger GetLogger(string name) {
    return _loggerFactory.Create(name);
}

Answer 1

可以说，主要区别似乎在于延迟加载:v1 将在调用 GetLogger 时创建记录器类型，而 v2 将在 SetFactory 期间创建记录器。其他方面没有太大区别。可以将记录器类型或工厂本身的引用保留为私有(private)静态字段。

但是，如果这是生产代码，我会完全摆脱这个日志管理器和工厂。相反，我会直接使用日志框架，例如 nlog 或 log4net。这简化了代码并使其更易于维护。在这种情况下构建抽象是过度设计的。