事情的起因是由于一段简单的数据库连接代码引起，这段代码从语法上看，是没有任何问题；但是就是莫名其妙的报错了，这段代码极其简单，就是打开数据库连接，读取一条记录，然后立即更新到数据库中。但是，惨痛的事实证明，老司机也是会翻车的。
前言
事情的起因是由于一段简单的数据库连接代码引起，这段代码从语法上看，是没有任何问题；但是就是莫名其妙的报错了，这段代码极其简单，就是打开数据库连接，读取一条记录，然后立即更新到数据库中。但是，惨痛的事实证明，老司机也是会翻车的。
1. 异常的发生来得太突然
1.1 引起不舒适的代码片段
[HttpPut]
public async void Put([FromBody] TopicViewModel model)
{
var topic = this.context.Topics.Where(f => f.Id == model.Id).FirstOrDefault();
topic.Content = model.Content;
this.context.Update(topic);
var affrows = await this.context.SaveChangesAsync();
}
这是一段不太标准的异步接口，可能你也这么写过， 从结构和语法上看，这段代码没有任何问题，而且正常情况下，它还能执行成功
1.2 报错信息
从报错信息中可以看出，数据库上下文对象被销毁了，是在什么时候销毁的呢，通过跟踪程序，了解到，是在 this.context.Update(topic); ，调用 Update 后执行了 DbContext.Dispose()，为了证明这点，我们重写 DbContext.Dispose() 方法，并简单的输出一句话
1.3 重写 DbContext.Dispose()
public class ForumContext : DbContext
{
public ForumContext(DbContextOptions<ForumContext> options) : base(options)
{
}
public DbSet<Topic> Topics { get; set; }
public DbSet<Post> Posts { get; set; }
public override void Dispose()
{
base.Dispose();
Console.WriteLine("Dispose");
}
}
1.4 再次执行程序，查看结果
通过输出结果红色方框处可以看到，确实是在执行了 Update 以后执行了 Dispose 方法，关于这点，如果我们使用了同步方法，先 Update 再 SaveChanges ，这是没有任何问题的，理论上说，EFCore 中启用了 AutoDetectChangesEnabled，我们在上面的代码中其实无需调用 Update，直接 SaveChangesAsync 即可，也不会抛出异常，同理，如果是在同步方法中，先执行 Update 再 SaveChanges ，也是没有任何问题的
1.5 同步 SaveChanges
[HttpPut]
public void Put([FromBody] TopicViewModel model)
{
var topic = this.context.Topics.Where(f => f.Id == model.Id).FirstOrDefault();
topic.Content = model.Content;
this.context.Update(topic);
Console.WriteLine("Updated");
var affrows = this.context.SaveChanges();
Console.WriteLine("affrows:{0}", affrows);
}
- 输出结果
从输出结果可知，先执行了 Update,然后执行了 SaveChanges 输出 affrows，最后执行了 Dispose 方法
2. 问题所在
那到底是什么问题引起了程序执行的不确定性呢，答案就是 async/await，我们先来尝试改进一下最初的代码
2.1 改进后的代码
[HttpPut]
public async Task Put([FromBody] TopicViewModel model)
{
var topic = this.context.Topics.Where(f => f.Id == model.Id).FirstOrDefault();
topic.Content = model.Content;
this.context.Update(topic);
Console.WriteLine("Updated");
var affrows = await this.context.SaveChangesAsync();
Console.WriteLine("affrows:{0}", affrows);
}
细心的你已经发现，这段代码和 1.1 之中的没有太多的不同，无非是增加了一些跟踪信息，其中，最关键的是：增加了返回值为：Task ，替换了 void
2.2 再次执行修正的程序
输出结果和 1.5 中的同步方法完全相同，至此，问题解决
3. 问题的解决方案
3.1 问题分析
为什么会发生这种问题呢，原因就是因为使用了异步方法 async/await 时，当没有值需要返回时，使用了 void 造成的，正确的做法是如果没有返回值，则返回 Task，如果有返回值，则使用 Task
；当一个异步方法内部没有返回 Task 的时候，基于任务的异步模式（TAP）并不知道异步任务的状态，当 this.context.Update 执行完成后，发现挂载在内存中的连接已经没有使用，就执行了回收；实际上，此时程序还没有执行完成，但是 TAP 并不知道，所以它不会去阻止这个回收的过程（使用标记），所以 async/await 应该成对出现，并且应该始终返回 Task 或者 Task ，以确保 TAP 能够将上下文进行正确的挂载，否则，当异常发生时，TAP 无非将异常信息挂载到相应的 Task 上，亦无法跟踪其执行状态等信息
3.2 解决方案
请牢记下面的铁律
- 3.2.1 在 EFCore 中，应当始终发挥 AutoDetectChangesEnabled 的特性，不要再更新实体的时候去调用 Update 方法
- 3.2.2 使用 async/await 修饰方法时，应该始终返回 Task 或者 Task
- 适当的使用同步方法，可避免异步踩坑
演示代码下载
https://github.com/lianggx/EasyAspNetCoreDemo/tree/master/Ron.TaskThird