构建对象图时的简单注入器集属性

Question

使用Simple Injector 我遇到了一个与我的记录器相同的主题，我在注入记录器的服务对象的构造函数中设置记录器名称。当服务写入日志时，可以通过这个日志名轻松识别。

由于将为每个服务设置LogNames，因此每个对象图请求的记录器应该是唯一的。

我想在构建图表时自动执行此操作，我已经在ExpressionBuilt() 中四处寻找，但我正在苦苦挣扎，但还没有得到我想要的工作 - 这甚至可能（或者想要的好东西做）？

我的构造函数代码如下（LogName 属性设置代码在我的大多数服务中都很常见）。

谢谢，

克里斯

public interface ILogger
{
    void LogMessage(string message, LogLevel level,
        ILoggableCompany company = null);

    string LogName {get; set; }
}

public BusinessUnitService
{
    private readonly IUnitOfWork unitOfWork;
    private readonly ILogger logger;

    public BusinessUnitService(IUnitOfWork unitOfWork, 
        ILogger logger)
    {
        this.unitOfWork = unitOfWork;
        this.logger = logger;

        // it would be great if we could take away this 
        // line and set it automatically
        this.logger.LogName = this.GetType().ToString();
    }
}

Answer 1

这个设计看起来有点像 log4net 的Logger<T> 设计，其中T 将是创建记录器的类。虽然我无法查看您的设计，但我想知道：aren't you logging too much?

如果你真的需要这样做，至少从ILogger 接口中删除LogName 属性，因为它在那里没有业务。这意味着您必须从构造函数中删除设置此属性的代码，这绝对没问题，因为这到处都是代码重复。

您正在尝试做的是基于上下文的注入，这不是开箱即用的支持，但 Simple Injector wiki 包含一个 Context Bases Injection section，它解释了如何添加此支持。该文档页面甚至使用Logger<T> 作为示例:-)

使用 wiki 引用的the extension method，您可以执行以下操作：

public interface ILogger
{
    void LogMessage(string message, LogLevel level,
        ILoggableCompany company = null);

    // No LogName property here. Keep it clean.
}

public class LoggerImpl : ILogger
{
    public void LogMessage(string message, 
        LogLevel level, ILoggableCompany company)
    {
       // implementation
    }

    // Property only part of the implementation.
    public string LogName {get; set; }
}

// The parent contains information about the type in 
// which ILogger is injected.
container.RegisterWithContext<ILogger>(parent =>
{
    // Retrieve a new LoggerImpl instance from the container 
    // to allow this type to be auto-wired.
    var logger = container.GetInstance<LoggerImpl>();

    // ImplementationType is null when ILogger is
    // requested directly (using GetInstance<ILogger>())
    // since it will have no parent in that case.
    if (parent.ImplementationType != null)
    {
        // Set the LogName with the name of the class 
        // it is injected into.
        logger.LogName = parent.ImplementationType.Name;
    }

    return logger;
});

// Register the LoggerImpl as transient. This line is
// in fact redundant in Simple Injector, but it is important
// to not accidentally register this type with another
// lifestyle, since the previous registration depends on it
// to be transient.
container.Register<LoggerImpl>();

因为这是通过挂钩 BuiltExpression 事件并通过重新连接表达式来实现的，以这种方式解析实例的速度几乎与使用 Func<T> 工厂方法的注册一样快。