前言
事情的起因是由于一段简单的数据库连接代码引起,这段代码从语法上看,是没有任何问题;但是就是莫名其妙的报错了,这段代码极其简单,就是打开数据库连接,读取一条记录,然后立即更新到数据库中。但是,惨痛的事实证明,老司机也是会翻车的。
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
<https://github.com/lianggx/EasyAspNetCoreDemo/tree/master/Ron.TaskThird>
热门工具 换一换