在重写的Controller Initialize方法中调用异步API报错如何解决

你遇到的报错和override关键字无关，核心问题是ASP.NET MVC框架的Initialize方法本身是同步生命周期方法，即便你将其重写为async Task签名，框架调用该方法时也不会等待内部异步任务执行完成，请求线程会直接向下处理后续逻辑，等异步API调用返回时请求上下文已经被释放，就会抛出「异步模块或处理程序已完成，但仍有一个异步操作处于待处理状态」的异常。

方案1：重写基类异步生命周期方法（推荐） #

直接重写Controller原生支持的异步生命周期方法OnActionExecutionAsync，框架会主动等待该方法执行完成再处理后续逻辑：

public abstract class BaseController : Controller
{
    public override async Task OnActionExecutionAsync(ActionExecutingContext context, ActionExecutionDelegate next)
    {
        if (ViewBag.NavbarList == null)
        {
            NavbarList navbarListResponse = await Navbar();
            ViewBag.NavbarList = navbarListResponse.NavbarItemList;
        }
        // 保留该行调用后续请求管道逻辑
        await base.OnActionExecutionAsync(context, next);
    }
}

方案2：使用全局异步过滤器 #

如果不想用基类实现，也可以注册全局异步动作过滤器统一注入导航栏数据：

public class NavbarLoadFilter : IAsyncActionFilter
{
    public async Task OnActionExecutionAsync(ActionExecutingContext context, ActionExecutionDelegate next)
    {
        if (context.Controller is Controller controller && controller.ViewBag.NavbarList == null)
        {
            var navbarListResponse = await Navbar();
            controller.ViewBag.NavbarList = navbarListResponse.NavbarItemList;
        }
        await next();
    }
}

将该过滤器注册到全局过滤器集合后，所有控制器的动作执行前都会自动加载导航栏数据。

方案3：同步阻塞调用（仅临时救急，不推荐高并发场景使用） #

如果必须保留现有Initialize方法的逻辑结构，可以将异步调用改为同步阻塞，避免异步任务泄漏：

protected override void Initialize(RequestContext requestContext)
{
    base.Initialize(requestContext);
    if (ViewBag.NavbarList == null)
    {
        // 同步等待异步方法执行完成
        NavbarList navbarListResponse = Navbar().GetAwaiter().GetResult();
        ViewBag.NavbarList = navbarListResponse.NavbarItemList;
    }
}

注意该方案在高并发场景下存在死锁风险，仅适合临时调试使用。

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