在 ASP.NET 中使用依赖注入和工厂模式传递服务

Question

我使用的是 ASP.NET Core，我知道框架已经提供了这样的 Logging 机制，但是用这个来说明我的问题。

我正在使用一种工厂模式来构建 Logger 类，因为我不知道日志记录的类型（因为它存储在 DB 中）。

ILogger 合同

Log(string msg)

那么LoggerFactory会根据DB传入的参数创建Logger后返回一个ILogger：

public class LoggerFactory
{
    public static Contracts.ILogger BuildLogger(LogType type)
    {
        return GetLogger(type);
    }

//other code is omitted, GetLogger will return an implementation of the related logger

现在，当我需要使用 Logger 时，我必须这样做：

  public class MyService
{
    private ILogger _logger
    public MyService()
    {
        _logger = LoggerFactory.BuildLogger("myType");
    }

但是，我打算保留我的类而不进行任何实例化，我需要在 MyService 中使用 Constructor DI，并且我需要在 Startup 上注入所有依赖项：

    services.AddTransient<Contracts.ILogger, LoggerFactory.BuildLogger("param") > ();

但这行不通，我们需要通过一个具体的实现。 如何使用 DI 来完成这项工作，有没有更好的方法来实现它？

Answer 1

你的方法有几个错误：

• 您的服务取决于具体的 LoggerFactory 类型，这是 Dependency Inversion Principle 违规。
• 执行此额外初始化可能会使构建对象图变得不可靠，而injection constructors should be simple。
• 它隐藏了ILogger 是您的消费者所依赖的真正服务这一事实。这使得系统更难测试、更难维护，并使对象图分析变得复杂。
• 使用工厂是一种气味，因为factories are hardly ever the right solution。

相反，您的服务应如下所示：

public class MyService
{
    private ILogger _logger;
    public MyService(ILogger logger)
    {
        _logger = logger;
    }
}

这极大地简化了依赖ILogger 的所有消费者。这也意味着为MyService 获取正确的ILogger 成为Composition Root 的责任，这是拥有这些知识的正确地方。

这确实意味着您可能需要从 ASP.NET Core 的内置 DI 容器转移到功能更丰富的 DI 库，因为内置容器无法进行上下文感知注册ILogger 同时让库自动连接其他构造函数依赖项。

使用 ASP.NET Core DI 容器，您只能使用委托手动连接您的服务。例如：

services.AddTransient<MyService>(c => new MyService(
    BuildLogger(typeof(MyService).Name),
    c.GetRequiredService<ISomeOtherDependency>(),
    c.GetRequiredService<IYetAnotherOne>());