Question

我使用mvc 5并按照this sample生成数组任务。

我确定数据库至少包含1行查询。我想我遇到了Task的问题，因为它向我提出了错误消息

tasks参数包含空值。

当我尝试时：

// I'm sure `cg` was not null and `type` was not empty
var cg = new List<string>();
var type = "";

var db = new MyDbContext();
var list = new List<TopicViewModels>();

if (cg != null && cg.Count > 0)
{
   var tasks = new Task<List<TopicViewModels>>[13];
   byte i = 0;

   while (i < cg.Count)
   {
      string _cg = cg[i];

      tasks[i] = Task.Run(async () => 
      {
         return await db.Topics.Where(m => m.Type == type && m.Category == _cg)
            .ToListAsync();
      });

      i++;
   }

   var continuation = Task.WhenAll(tasks); //the tasks argument included a null value

   // never go to this loop...
   foreach (var topics in continuation.Result)
   {
      topics.ForEach(x => list.Add(x));
   }
}

我设置断点来检查数组tasks，tasks[0]不为空。它的正确添加。

你能解释一下为什么吗？

更新（基于@YacoubMassad评论）

await Task.WhenAll(tasks); //same error here...

//never go to this loop, too...
foreach (var task in tasks)
{
   //
}

更新2：（基于@DavidPine回答）

if (cg != null && cg.Count > 0)
{
   var tasks = new List<Task<List<TopicViewModels>>>();
   cg.ForEach(x =>
   {
      tasks.Add(Task.Run(async () =>
      {
         return await db.Topics.Where(m => m.Type == type && m.Category == x)
            .ToListAsync();
      }));
   });

   foreach (var topics in await Task.WhenAll(tasks.ToArray()))
   {
      topics.ForEach(x => list.Add(x));
   }
}

Answer 1

这里有一些问题：

1。使用Task或Task<T>时，如果可用，则始终使用async / await，即;你正在使用.NET 4.5。

2. 当您调用Task.WhenAll(tasks)时，数组中的任务可能为null。您需要检查并正确处理它。

3. 您需要等待所有时间以确保工作完成，await Task.WhenAll(tasks)。

var cg = new List<string>();
var type = "";

var db = new MyDbContext();
var list = new List<TopicViewModels>();

if (cg != null && cg.Count > 0)
{
   var tasks = new Task<List<TopicViewModels>>[13];
   byte i = 0;

   while (i < cg.Count)
   {
      string _cg = cg[i];

      tasks[i] = Task.Run(async () => 
      {
         // Isn't it possible that the where clause filters out and returns null?
         return await db.Topics
                        .Where(m => m.Type == type && m.Category == _cg)
                        .ToListAsync();
      });

      i++;
   }

   // Use await keyword to ensure that work is done
   var continuation = await Task.WhenAll(tasks.Where(t => t != null).ToArray());

   // never go to this loop...
   foreach (var topics in continuation.Result)
   {
      topics.ForEach(x => list.Add(x));
   }
}

Answer 2

您的代码中存在一些问题。

首先，您使用查询数据库的自然异步方法，您不需要在线程池线程上执行，这是多余的，尤其是在已经使用线程池线程来处理请求的ASP.NET中。 p>

其次，您没有等待Task.WhenAll返回的任务，当您使用foreach在Task.Result循环中进行迭代时，您会同步阻止。

你的代码应该是：

var queryTasks = cg.Select(cg => db.Topics.Where(m => m.Type == type && m.Category == cg).ToListAsync());
return await Task.WhenAll(queryTasks);

请注意，EF DbContext不是线程安全的，并且不允许对其执行并发操作。如果是这种情况，您需要独立await每个查询。

tasks参数包含null值

2 个答案: