async void 导致程序崩溃

来源：https://note.guoqianfan.com/2022/04/23/dont-use-async-void/

前言

之前都是在文档里看到：除了winform的事件可以使用async void，其他情况下绝对不能使用async void，而是要用async Task。

对于这个规范，虽然不是很明白内里原因，但是一直遵守着。

直到这天看到了这篇博客：在 ASP.NET Core 中誤用 async void 竟引發了 502（Bad Gateway），说async void里出现异常时会导致程序崩溃。研究测试了一番，终于明白原因。

摘录重点如下：

根據使用者提供的另一個線索「網站的某個功能壞了」，我們繼續往下追查，從程式碼當中我看到了一個近期新加的方法，它使用了 async void，沒錯，它使用了 async void，而且很不幸地它會發生 Exception，更慘的是這個 Exception 沒有被處理。

對 C# 非同步程式設計有了解的朋友，看到這邊應該大致上可以知道是發什麼問題了，async void 是建議應該避免使用的宣告方式，其中一個原因就是當 async void 方法發生 Exception 時無法從呼叫端捕獲，即使加了 try...catch... 也沒用，async void 方法就有點像是我們自己起了另一個 Thread 去執行程式一樣，執行的過程中如果發生 Exception 沒有去處理，Exception 就會一路被往上拋，最終在 AppDomain 層級被捕獲，然後我們的應用程式就掛了。

async-void-方法的异常无法被捕获

async void方法抛出的异常无法被捕获，异常会被一直往上面抛，最终在AppDomain层级被捕获，然后程序就挂了。

示例代码如下：

[HttpGet]
public async void Get()
{
    try
    {
        ThrowExceptionAsync();
    }
    catch (Exception ex)
    {
        //这里不能捕获到异常，程序崩溃！
        _logger.LogInformation(ex, "ex...");
    }
}

async void ThrowExceptionAsync()
{
    throw new Exception("async void ex!");
}

注意

前面所说的是 async void方法抛出的无法预知到的异常。在async void方法内部，我们仍然能够使用try catch，逻辑是正常逻辑。具体见下面的示例：

[HttpGet]
public async void Get()
{
    //在async void方法内部，我们仍然能够使用try catch，逻辑是正常逻辑。
    //此处try catch是有效的。异常被捕获处理了，async void方法执行无异常，不会导致程序崩溃。
    try
    {
        await Task.Run(() =>
        {
            throw new Exception("ex!");
        });
    }
    catch (Exception ex)
    {
        _logger.LogInformation(ex,"ex...");
    }
}

测试

崩溃

出现异常时能导致崩溃的代码有2种，如下：

[HttpGet]
public async void Get()
{
    //异常会导致程序崩溃
    throw new Exception("ex!");
}

[HttpGet]
public async void Get()
{
    //异常会导致程序崩溃
    await Task.Run(() =>
    {
        throw new Exception("ex!");
    });
}

注意

下面的async void代码不会抛异常。

[HttpGet]
public async void Get()
{
    Task.Run(() =>
    {
        throw new Exception("ex!");
    });
}

代码里的async void没问题(不抛异常)，其实也符合逻辑。因为async void里面没有异常，自然就不会导致程序崩溃。

异常在Task.Run里面，因为没有使用await进行等待，那么异常就是被线程池线程捕获的，它们捕获到后，不会再往上面抛了，直接自己内部消化掉了。

因为async void在执行时没有异常，自然就不会导致程序崩溃。

但是由于我们不能保证所有代码都没有异常，所以不要使用async void！

不崩溃

只要不是async void，就算请求处理程序抛出了异常，也不会影响到主线程的。最多就是这次请求出错，返回500 Internal Server Error而已。

测试的几种代码如下：

[HttpGet]
public async Task Get()
{
    //500错误码
    throw new Exception("ex!");
}

[HttpGet]
public async Task Get()
{
    //500错误码
    await Task.Run(() =>
    {
        throw new Exception("ex!");
    });
}