使用异步避免重复代码

Question

如何避免为异步和非异步方法编写两次相同的代码。 我目前正在使用 ASP.NET，所以我目前在请求线程上，我很快了解到他下面的代码（应该显示我的意图），绝对是错误的做法。

应用程序死锁，因为 await 关键字试图返回 .Result 阻塞的同一线程。

我这样做的全部原因是为了避免两次编写相同的“FindAll”代码。

public IEnumerable<Resource> FindAll()
{
   return FindAllAsync().Result;

}

public async Task<IEnumerable<Resource>> FindAllAsync()
{
   return await Context.Resources.ToListAsync();
}

那么你如何解决这个问题？

Answer 1

如何避免为异步和非异步方法编写两次相同的代码。

一般情况下你不能。

有问题的操作要么是自然异步的，要么是自然同步的。在这个例子中（一个数据库请求），它自然是异步的。因此，使 API 异步。就是这样。

Stephen Toub 有一对著名的博文 Should I expose synchronous wrappers for asynchronous methods? 和 Should I expose asynchronous wrappers for synchronous methods? 两个问题的简短回答都是“不”。

您可以通过各种 hack 来公开这两种类型的 API（Stephen 在他的帖子中介绍了每种方法），但与缺点相比，优点微不足道。

Answer 2

同步和异步方法的行为应该不同。通常这意味着同步调用应该调用一个作为请求的一部分阻塞的 API，而异步应该调用“一直异步”的 API

如果你不想创建完全同步的api，在你的情况下，你可以使用ConfigureAwait(false)。

当你用ConfigureAwait(false) 标记Task 时，你实际上是在说“没有必要在同一个await 内运行延续（await 之后的代码），你可以在里面完成您当前的上下文（通常是 ThreadPool 线程）"

至于你的第二种方法，你可以去掉async关键字，省去冗余生成的状态机：

public IEnumerable<Resource> FindAll()
{
     return FindAllAsync().Result;
}

public Task<IEnumerable<Resource>> FindAllAsync()
{
     return Context.Resources.ToListAsync();
}

一些阅读参考：

Stephan Cleary - Dont block on async code

Best practice to call ConfigureAwait(false)

Task.ConfigureAwait(false) MSDN