Question

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

我正在使用一种Factory模式来构建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

您的方法中存在一些错误：

您的服务取决于违反Dependency Inversion Principle的具体LoggerFactory类型。
执行此额外初始化可能会导致构建对象图不可靠，而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的所有消费者。这也意味着为ILogger获得正确MyService成为Composition Root的责任，Oracle custom JSP tag help page:是获得此知识的正确位置。

但这确实意味着您可能需要从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>());

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

1 个答案: