如何正确使用Task.WhenAll()

时间:2012-08-14 10:25:31

标签: c# .net asynchronous .net-4.5 async-await

我正在尝试使用Task.WhenAll来等待多个任务的完成。

我的代码如下 - 它应该启动多个异步任务,每个异步任务检索一个总线路由,然后将它们添加到本地阵列。但是,Task.WhenAll(...)会立即返回,并且本地路由数组的计数为零。这看起来很奇怪,因为我希望每个await中的各种Task语句表示流程已暂停,Task在完成之前不会返回。

List<Task> monitoredTasks = new List<Task>();
foreach (BusRouteIdentifier bri in stop.services)
{
    BusRouteRequest req = new BusRouteRequest(bri.id);

    // Start a new task to fetch the route for each stop
    Task getRouteTask = Task.Factory.StartNew(async () =>
    {
        var route = await BusDataProviderManager.DataProvider.DataBroker.getRoute(req);

            // Add the route to our array (on UI thread as it's observed)
            await dispatcher.RunAsync(Windows.UI.Core.CoreDispatcherPriority.Normal, delegate
            {
                this.routes.Add(route);
            });
    });

    // Store the task in our monitoring list
    monitoredTasks .Add(getRouteTask);
}

Debug.WriteLine("Awaiting WHENALL");
await Task.WhenAll(monitoredTasks );
Debug.WriteLine(string.Format("WHENALL returned (routes count is {0} ", this.routes.Count));

this.OnWillEndFetchingRoutes(new EventArgs());

我显然做错了什么 - 但是什么?

2 个答案:

答案 0 :(得分:7)

这归结为对async-await真正起作用的基本缺乏了解。

内部任务是将流返回到外部任务,然后在等待返回之前完成。

为了达到我的目的,我需要按如下方式进行重构:

List<Task<BusRoute>> routeRetrievalTasks = new List<Task<BusRoute>>();
foreach (BusRouteIdentifier bri in stop.services)
{
    BusRouteRequest req = new BusRouteRequest(bri.id);
    routeRetrievalTasks.Add(BusDataProviderManager.DataProvider.DataBroker.getRoute(req));
}

foreach (var task in routeRetrievalTasks)
{
    var route = await task;
    this.routes.Add(route); // triggers events
}

感谢Dave Smits

答案 1 :(得分:5)

我怀疑问题是你对Task.Factory.StartNew()的号召。我怀疑你最终得到的是Task<Task>,而你只是在发现启动任务的时候。

请改为尝试:

Func<Task> taskFunc = async () =>
{
    var route = await BusDataProviderManager.DataProvider.DataBroker.getRoute(req);

    // Add the route to our array (on UI thread as it's observed)
    await dispatcher.RunAsync(Windows.UI.Core.CoreDispatcherPriority.Normal, delegate
    {
        this.routes.Add(route);
    });

}

Task getRouteTask = Task.Run(taskFunc);