c# - 为什么调用 await 会过早地完成父任务？

Question

我正在尝试创建一个控件，该控件公开一个消费者可以订阅以执行加载操作的 DoLoading 事件。为方便起见，应从 UI 线程调用事件处理程序，允许消费者随意更新 UI，但他们也可以使用 async/await 来执行长时间运行的任务，而不会阻塞 UI 线程。

为此，我声明了以下委托(delegate):

public delegate Task AsyncEventHandler<TEventArgs>(object sender, TEventArgs e);

这允许消费者订阅事件:

public event AsyncEventHandler<bool> DoLoading;

想法是消费者将这样订阅事件(此行在 UI 线程中执行):

loader.DoLoading += async (s, e) =>
            {
                for (var i = 5; i > 0; i--)
                {
                    loader.Text = i.ToString(); // UI update
                    await Task.Delay(1000); // long-running task doesn't block UI
                }
            };

在适当的时间点，我为 UI 线程获取 TaskScheduler 并将其存储在 _uiScheduler 中。

loader 会在适当的时候使用以下行触发事件(这发生在随机线程中):

this.PerformLoadingActionAsync().ContinueWith(
            _ =>
            {
                // Other operations that must happen on UI thread
            },
            _uiScheduler);

请注意，此行不是从 UI 线程调用的，而是需要在加载完成时更新 UI，因此我使用 ContinueWith 在加载任务时在 UI 任务调度程序上执行代码完成。

我已经尝试了以下方法的几种变体，但都没有奏效，所以这就是我现在的位置:

private async Task<Task> PerformLoadingActionAsync()
{
    TaskFactory uiFactory = new TaskFactory(_uiScheduler);

    // Trigger event on the UI thread and await its execution
    Task evenHandlerTask = await uiFactory.StartNew(async () => await this.OnDoLoading(_mustLoadPreviousRunningState));

    // This can be ignored for now as it completes immediately
    Task commandTask = Task.Run(() => this.ExecuteCommand());

    return Task.WhenAll(evenHandlerTask, commandTask);
}

private async Task OnDoLoading(bool mustLoadPreviousRunningState)
{
    var handler = this.DoLoading;

    if (handler != null)
    {
        await handler(this, mustLoadPreviousRunningState);
    }
}

如您所见，我正在启动两项任务，并希望之前的 ContinueWith 能够完成其中一项所有。

commandTask 立即完成，因此暂时可以忽略。在我看来，eventHandlerTask 应该只完成一个事件处理程序完成的任务，假设我正在等待调用事件处理程序的方法，并且我正在等待事件处理程序本身。

但是，实际情况是，一旦我的事件处理程序中的行 await Task.Delay(1000) 被执行，任务就会完成。

这是为什么？我怎样才能得到我期望的行为？

Answer 1

您正确地意识到 StartNew()返回 Task<Task>在这种情况下，你关心内部 Task (虽然我不确定你为什么在开始 Task 之前等待外部 commandTask)。

然后你返回 Task<Task>并忽略内部 Task .你应该做的是使用 await而不是 return并更改 PerformLoadingActionAsync() 的返回类型只是Task :

await Task.WhenAll(evenHandlerTask, commandTask);

一些注意事项:

1. 以这种方式使用事件处理程序非常危险，因为您关心的是 Task。从处理程序返回，但如果有更多处理程序，则只有最后一个 Task如果您正常引发事件，将被退回。如果你真的想这样做，你应该调用 GetInvocationList() ，它允许您调用和 await每个处理程序分别:

   private async Task OnDoLoading(bool mustLoadPreviousRunningState)
   {
       var handler = this.DoLoading;
   
       if (handler != null)
       {
           var handlers = handler.GetInvocationList();
   
           foreach (AsyncEventHandler<bool> innerHandler in handlers)
           {
               await innerHandler(this, mustLoadPreviousRunningState);
           }
       }
   }
   

   如果您知道您永远不会有多个处理程序，则可以使用可以直接设置的委托(delegate)属性来代替事件。

2. 如果你有一个 async方法或 lambda 具有唯一的 await就在它的 return 之前(并且没有 finally s)，那么你不需要让它成为 async , 只需返回 Task直接:

   Task.Factory.StartNew(() => this.OnDoLoading(true))