Autofac聚合服务注入log4net.ILog的MVC代码重构问题

Got it, let's work through how to get this set up correctly. Since you're using Autofac's Aggregate Services with log4net's ILog and the LoggingModule, the key challenge here is making sure the log instance in your aggregate service is tied to the controller type that's using it (not the aggregate service's own type). Here are a couple of straightforward approaches:

1. 直接传递控制器的ILog实例给聚合服务 #

This is the simplest approach if your aggregate service methods only need logging in the context of a controller call.

Step 1: 更新聚合服务接口 #

在需要控制器特定日志的方法中添加ILog参数：

public interface IMyAggregateService
{
    // 为控制器特定日志添加ILog参数
    void PerformBusinessLogic(ILog controllerLog);
}

Step 2: 实现聚合服务 #

直接使用传入的ILog实例即可：

public class MyAggregateService : IMyAggregateService
{
    // 其他依赖项正常注入
    private readonly IOtherService _otherService;

    public MyAggregateService(IOtherService otherService)
    {
        _otherService = otherService;
    }

    public void PerformBusinessLogic(ILog controllerLog)
    {
        controllerLog.Info($"开始执行业务逻辑，来自控制器: {controllerLog.Logger.Name}");
        // 业务逻辑代码
        _otherService.DoSomething();
        controllerLog.Info("业务逻辑执行完成");
    }
}

Step 3: 在控制器中注入并使用 #

控制器会通过LoggingModule获取到自己的类型化ILog，然后传递给聚合服务：

public class HomeController : Controller
{
    private readonly IMyAggregateService _aggregateService;
    private readonly ILog _controllerLog;

    public HomeController(IMyAggregateService aggregateService, ILog log)
    {
        _aggregateService = aggregateService;
        _controllerLog = log; // 这个实例是针对HomeController类型的
    }

    public ActionResult Index()
    {
        _aggregateService.PerformBusinessLogic(_controllerLog);
        return View();
    }
}

2. 使用日志工厂解析控制器特定日志 #

如果你不想在每个方法都传递ILog，可以给聚合服务注入一个工厂，用来为指定类型创建日志实例。

Step 1: 更新聚合服务以使用日志工厂 #

public interface IMyAggregateService
{
    void PerformBusinessLogic(Type controllerType);
}

public class MyAggregateService : IMyAggregateService
{
    private readonly IOtherService _otherService;
    private readonly Func<Type, ILog> _logFactory;

    public MyAggregateService(IOtherService otherService, Func<Type, ILog> logFactory)
    {
        _otherService = otherService;
        _logFactory = logFactory;
    }

    public void PerformBusinessLogic(Type controllerType)
    {
        var controllerLog = _logFactory(controllerType);
        controllerLog.Info($"为{controllerType.Name}执行逻辑");
        // 业务逻辑代码
    }
}

Step 2: 在控制器中调用 #

把控制器的类型传递给聚合服务方法：

public ActionResult Index()
{
    _aggregateService.PerformBusinessLogic(this.GetType());
    return View();
}

3. 验证LoggingModule配置是否正确 #

请确保你的LoggingModule在Autofac中正确注册，并且它是基于目标组件类型解析ILog的。标准实现如下：

public class LoggingModule : Module
{
    protected override void AttachToComponentRegistration(IComponentRegistry componentRegistry, IComponentRegistration registration)
    {
        registration.Preparing += (sender, e) =>
        {
            var logParameter = new ResolvedParameter(
                (p, c) => p.ParameterType == typeof(ILog),
                (p, c) => LogManager.GetLogger(e.Component.Activator.LimitType));
            e.Parameters = e.Parameters.Union(new[] { logParameter });
        };
    }
}

然后在Autofac配置中注册它：

public static void ConfigureContainer(ContainerBuilder builder)
{
    // 注册日志模块
    builder.RegisterModule<LoggingModule>();
    
    // 注册聚合服务
    builder.RegisterAggregateService<IMyAggregateService>();
    
    // 注册MVC控制器（必须用AsSelf()让Autofac识别控制器类型）
    builder.RegisterControllers(typeof(MvcApplication).Assembly).AsSelf();
    
    // 注册其他服务
    builder.RegisterType<OtherService>().As<IOtherService>();
}

前两种方法都能让你在聚合服务中获取到控制器特定的ILog实例。第一种更直观直接，第二种适合有多个方法需要日志且不想重复传递ILog的场景。

内容的提问来源于stack exchange，提问作者Smoothjedi