注射ILogger 每次都创建一个新的记录器?

问题描述 投票:4回答:1

the documentation中的日志记录示例中,有一个示例如何将记录器注入控制器:

public class TodoController : Controller
{
    private readonly ITodoRepository _todoRepository;
    private readonly ILogger _logger;

    public TodoController(ITodoRepository todoRepository,
        ILogger<TodoController> logger)
    {
        _todoRepository = todoRepository;
        _logger = logger;
    }
}

每次我将记录器注入此处时,DI框架是否会创建一个新的记录器?有没有更好的办法?

c# asp.net-core asp.net-core-2.0
1个回答
7
投票

通过查看源代码可以轻松回答这个问题。当你做services.AddLogging()时,默认行为是ILogger<T> is registered as a singleton

public static IServiceCollection AddLogging(this IServiceCollection services, Action<ILoggingBuilder> configure)
{
    // …

    services.TryAdd(ServiceDescriptor.Singleton<ILoggerFactory, LoggerFactory>());
    services.TryAdd(ServiceDescriptor.Singleton(typeof(ILogger<>), typeof(Logger<>)));

    // …
}

所以不,只要应用程序运行,ILogger<T>实例就会保留某种类型的T。因此,当将ILogger<TodoController>注入控制器时,每次都会将相同的记录器实例传递给它。

当然这仅适用于记录器,而不适用于您的控制器本身。默认情况下,控制器在DI之外激活,但实际上具有范围生命周期。因此,在每个请求中,都会有一个新的控制器实例;但那个人将从之前获得记录器实例。

要回答你的上一个问题,有更好的方法吗?否。除了这个行为已经很好(因为不需要新的记录器实例)之外,使用日志记录的正确方法确实是将ILogger<T>注入类型T,因此您可以获得正确分类的记录器实例。真的没有必要担心这里非常薄的记录器,因为在后台发生的事情太多,你可能永远都看不到;)

© www.soinside.com 2019 - 2024. All rights reserved.