ASP.NET Core MVC中CS0120错误：如何在特性中注入IActivityLogging？

错误原因 #

C# 特性的构造函数参数只能是编译时常量（如bool、string、枚举等），无法传递类的实例成员（比如控制器中的_activityLogging）。这就是你遇到CS0120错误的根本原因——_activityLogging是控制器的非静态实例字段，不能作为特性构造参数传入。

正确解决方案 #

方案一：在OnException中从请求服务获取IActivityLogging #

这种方式保留特性的动态参数（isPartial），同时在异常处理时从ASP.NET Core的请求服务容器中获取IActivityLogging实例，是最灵活的方案。

修改ClientWebErrorAttribute：

public class ClientWebErrorAttribute : ExceptionFilterAttribute
{
    private readonly bool _isPartial;

    // 仅保留编译时常量参数
    public ClientWebErrorAttribute(bool isPartial)
    {
        _isPartial = isPartial;
    }

    public override async Task OnExceptionAsync(ExceptionContext context)
    {
        // 从当前请求的服务容器中获取IActivityLogging实例
        var activityLogging = context.HttpContext.RequestServices.GetRequiredService<IActivityLogging>();
        
        // 执行日志记录逻辑
        await activityLogging.LogError(context.HttpContext.Request, context.Exception, ActivityLoggingUtil.TYPE_UNHANDLED_ERROR.Value);

        // 原有错误视图返回逻辑
        string errorViewName = "Error"; // 根据你的实际视图名调整
        var viewData = new ViewDataDictionary(new EmptyModelMetadataProvider(), context.ModelState);
        viewData.Add("ErrorMessage", context.Exception.Message);

        context.Result = _isPartial 
            ? new PartialViewResult { ViewName = errorViewName, ViewData = viewData } 
            : new ViewResult { ViewName = errorViewName, ViewData = viewData };

        context.ExceptionHandled = true;
    }
}

控制器中使用：

// 仅传递isPartial常量参数
[ClientWebError(isPartial: false)]
public async Task<IActionResult> Menu()
{
    // 移除原有try-catch，异常将由特性自动捕获处理
    // ...你的业务逻辑代码
}

方案二：实现IServiceFilter进行依赖注入 #

如果isPartial参数不需要动态调整（全局固定），可以让特性实现IServiceFilter，通过构造函数直接注入IActivityLogging，更符合依赖注入的规范。

修改ClientWebErrorAttribute：

// 实现IServiceFilter，让框架自动注入依赖
public class ClientWebErrorAttribute : ExceptionFilterAttribute, IServiceFilter
{
    private readonly bool _isPartial;
    private readonly IActivityLogging _activityLogging;

    // 构造函数注入服务和固定参数
    public ClientWebErrorAttribute(bool isPartial, IActivityLogging activityLogging)
    {
        _isPartial = isPartial;
        _activityLogging = activityLogging;
    }

    public override async Task OnExceptionAsync(ExceptionContext context)
    {
        // 直接使用注入的服务
        await _activityLogging.LogError(context.HttpContext.Request, context.Exception, ActivityLoggingUtil.TYPE_UNHANDLED_ERROR.Value);

        // 错误视图返回逻辑（同方案一）
        string errorViewName = "Error";
        var viewData = new ViewDataDictionary(new EmptyModelMetadataProvider(), context.ModelState);
        viewData.Add("ErrorMessage", context.Exception.Message);

        context.Result = _isPartial 
            ? new PartialViewResult { ViewName = errorViewName, ViewData = viewData } 
            : new ViewResult { ViewName = errorViewName, ViewData = viewData };

        context.ExceptionHandled = true;
    }
}

在Program.cs中注册特性服务：

// 注册为作用域服务（根据实际需求选择Scoped/Transient/Singleton）
builder.Services.AddScoped(sp => new ClientWebErrorAttribute(false, sp.GetRequiredService<IActivityLogging>()));

控制器中使用：

// 不需要传递参数，因为服务注册时已配置
[ServiceFilter(typeof(ClientWebErrorAttribute))]
public async Task<IActionResult> Menu()
{
    // ...业务逻辑代码
}

总结 #

• 不能直接将控制器中的_activityLogging实例传入特性构造函数，这是C#特性的语法限制。
• 推荐使用方案一处理需要动态参数的场景，灵活且无需额外服务注册。
• 若参数固定，方案二更符合依赖注入设计原则。

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