异步/等待任务操作中未捕获的异常

Question

我正在用。NET核心编写一个ASP.NET MVC站点。我试图封装一些常见的异常处理。在基类中，我有这个方法。

public abstract class BaseController<TController> : Controller where TController : Controller
{
    protected IActionResult ExceptionHandledOperation(Func<IActionResult> operation, Func<IActionResult> handleException)
    {
        try
        {
            return operation.Invoke();
        }
        catch (Exception exception)
        {
            Logger.LogError($"Operation {Request.Path} Exception", exception);

            return handleException.Invoke();
        }
    }
}

从继承自该基类的控制器中，我使用如下方法：

[Route("api/[controller]")]
public class MyController : BaseController<MyController>
{
    [HttpGet]
    public IActionResult Get()
    {
        return ExceptionHandledOperation(() => Ok(_someService.GetAsync().Result),
                                         NotFound);
    }
}

假设_someservice.getAsync（）方法如下：

public class SomeService
{
    public async Task<PreconfigurationData> GetAsync()
    {
        // http request removed for brevity

        if (!response.IsSuccessStatusCode)
            throw new Exception("SomeService Exception");
    }
}

这工作得很好，将捕获基类方法中的异常并返回NotFound结果。

但是，我希望避免从SomeService.GetAsync方法调用。result。我读到的任何地方都说不要那样做，因为它会死锁。

因此，我将基本控制器修改为：

public abstract class BaseController<TController> : Controller where TController : Controller
{
    protected async Task<IActionResult> ExceptionHandledOperationAsync(Func<IActionResult> operation, Func<IActionResult> handleException)
    {
        try
        {
            return await Task.Run(() => operation.Invoke());
        }
        catch (Exception exception)
        {
            Logger.LogError($"Operation {Request.Path} Exception", exception);

            return await Task.Run(() => handleException.Invoke());
        }
    }
}

[Route("api/[controller]")]
public class MyController : BaseController<MyController>
{
    [HttpGet]
    public async Task<IActionResult> Get()
    {
        return await ExceptionHandledOperationAsync(() => Ok(_someService.GetAsync()),
                                                    NotFound);
    }
}

但是，从SomeService.getAsync方法引发的异常从未被捕获，而且在处理异常时，我从未获得打算发送的NotFound响应。

我读到的每一个地方都说你只需要在尝试中等待任务，然后任何异常都会被发现，但我的从来没有。

已解决

Answer 1

您的代码中有一个逻辑错误。您调用return await task.run(()=>handleexception.invoke())；但是在函数委托中，您运行异步代码而不等待它（这里:await ExceptionHandledOperationAsync(()=>Ok(_someservice.getasync()),NotFound)。

因此，在try/catch块中，在异步调用完成之前，该方法将被执行并立即返回。

正如我的建议中所指出的，函数委托也需要是可等待的，请阅读:returnstask。

public abstract class BaseController<TController> : Controller where TController : Controller
{
    protected async Task<IActionResult> ExceptionHandledOperationAsync<T>(
        Func<Task<T>> operation,
        Func<object, IActionResult> successHandler,
        Func<IActionResult> exceptionHandler
    )
    {
        try
        {
            return successHandler.Invoke(await operation.Invoke());
        }
        catch (Exception exception)
        {
            //Logger.LogError($"Operation {Request.Path} Exception", exception);

            return exceptionHandler.Invoke();
        }
    }
}

[Route("api/[controller]")]
public class MyController : BaseController<MyController>
{
    [HttpGet]
    public async Task<IActionResult> Get()
    {
        return await ExceptionHandledOperationAsync(() => _someService.GetAsync(), Ok, NotFound);
    }
}

您还需要将successshandler（将结果传递给OK)移动到该方法中，如上图所示。但它真的是丑陋的代码。因此，SomeService应该自己处理服务故障，并在未找到任何值时返回null。

异常时返回notfound()似乎非常奇怪，因为它表明记录不存在，但可能由于网络连接或数据序列化而失败。