一行触发并忘记：void vs. async void + await

Question

我有一个void 事件处理程序，它只包含一行调用awaitable 方法的行。

void Handler( object sender, EventArgs args ) => AwaitableMethod();

将返回类型更改为Task 不是一种选择，因此无论语法糖如何，它都是一劳永逸的。即便如此，Visual Studio 在 unawaited 调用下放置了一个绿色波浪线，建议我调用处理程序 async void 和 await。

async void Handler( object sender, EventArgs args ) => await AwaitableMethod();

我的理解是，在这种情况下添加 async void 和 await 只是无用的开销。 Visual Studio 是否知道一些我不知道的事情，还是只是不必要地烦人？

Answer 1

我的理解是，在这种情况下添加 async void 和 await 只是无用的开销。

没有。

如果Handler 返回了Task，那么这是正确的，eliding async/await 就可以了；没有async/await 的代码将直接返回Task，而不是用await“解包”它，然后用async“包”回Task。

但是，这里不是这样； Handler 返回void，因此没有async/await 的代码将忽略返回的Task，这在绝大多数情况下都是错误的（因此会出现编译器警告）。具体来说，忽略Task 将忽略来自Task 的任何异常。您的代码也无法知道被忽略的 Task 何时完成，但大概这是可以接受的，因为您的处理程序正在返回 void。

async void 方法会进行“注册”，以便框架知道有一个任务仍在进行中，因此框架知道何时可以安全关闭。唯一真正关心这一点的 .NET 提供的框架是 ASP.NET pre-Core；所有其他 .NET 提供的框架（包括所有 UI 框架）都会忽略该“注册”。

Answer 2

在这种特殊情况下，它主要取决于您的错误处理。

如果AwaitableMethod异步抛出，会在第二个版本的UI线程上抛出。

Answer 3

我建议在这种情况下避免使用事件处理程序，并尽可能使用类似的东西。

 public class Eventful
    {
        private readonly List<Func<Task>> _handlers = new List<Func<Task>>();

        public void AddHandler(Func<Task> handler) => _handlers.Add(handler);

        private async Task RunHandlers()
        {
            foreach (var handler in _handlers)
            {
                await handler();
            }
        }
    }

    public class Consumer
    {
        public Consumer()
        {
            var eventful = new Eventful();
            eventful.AddHandler(async () =>
            {
                await DoAsyncTask();
            });
        }

        private static async Task DoAsyncTask()
        {
            await Task.CompletedTask;
        }
    }