今天在开发过程中发现.在SaveChanges的时候偶尔会抛出异常:Cannot access a disposed object. A common cause of this error is disposing a context that was resolved from dependency injection and then later trying to use the same context instance elsewhere in your application. This may occur if you are calling Dispose() on the context, or wrapping the context in a using statement. If you are using dependency injection, you should let the dependency injection container take care of disposing context instances.
异常说得很明显,通过依赖注入的DBContext上下文已经被其他地方Dispose了.所以无法再次Dispose.
代码逻辑很简单,就是发送邮件后,调用委托通知tracking发送成功.然后保存到数据库.
按理说这段代码没问题,但就是报错了.
一开始以为是哪里没有await,所以导致task开的新线程没有被等待,从而导致提前gc.但代码就这么多,都检查过了没有遗漏的地方.排除
后来怀疑是DBContext的生命周期问题,但DBcontext是ServiceLifetime.Scoped. 同一个request中是单例的.排除
后来经过和同事交流,在代码结尾处加入Task.CompletedTask.等待所有线程结束.
神奇的事情发生了,完美运行.不报错了.
那这样的话,就问题就只可能是定义的Action<int> rollBack委托的问题了.
猛然发现,尽管我在代码中确实加入了async/await关键字
但是这里的异步等待,只是异步等待委托内部的操作.并不等待Action委托本身.也就是说,当我们执行委托里的方法时.开辟了一个新的线程去执行_dbContext.SaveChangesAsync()的方法.但是并没有等待它完成.
这时候主线程会立即执行下一步,也就是返回结果给Controller层. Return Ok()给前端.这个时候DBContex立刻就会调用Dispose.等到委托的方法调用完毕再次Dispose的时候.自然而然的就会抛出异常啦.因为他之前已经被Dispose了.
所有解决办法很简单
方法一 在代码结尾加入await Task.CompletedTask 等待所有线程都结束.再返回.
方法二 讲Action<int> 换成 Func<int,Task> 并在调用委托前 await
经过这次问题,还是暴露出不少问题.
1:对async/await 还是有认识不足的地方.基础知识不扎实,导致了对委托的错误使用.
2:对自己的代码太过自信.没有做完整的测试.事实上 这里的代码我都没测试过就上了DEV环境.认为很简单不会出问题的.做事还是太浮躁.
所以写一篇博客,用以自省
|