我有以下代码:
public void DequeueRecipe(AuthIdentity identity,params Guid[] recipeIds) { using (var session = GetSession()) { var recipes = (from r in recipeIds select new Models.Recipes {RecipeId = r}).ToArray(); var dbRecipes = session.QueryOver<Models.QueuedRecipes>() .Where(Expression.Eq("UserId",identity.UserId)) .Where(Expression.InG("Recipe",recipes)) .List<Models.QueuedRecipes>(); using (ITransaction transaction = session.BeginTransaction()) { dbRecipes.ForEach(r => session.Delete(r)); // <-- Warning transaction.Commit(); } } }
reSharper给了我警告:
Access to disposed closure
在线上:
dbRecipes.ForEach(r => session.Delete(r));
(会话变量加下划线作为罪魁祸首).
虽然ForEach方法采用lamba表达式来创建一个围绕变量会话的闭包,但我没有看到在执行此代码时可能会处理它的方法.也许reSharper认为ForEach可以并行执行某种任务,或者保存Action<>在以后的某个时间,从技术上来说,它可能会被释放,而匿名函数仍然可以在内存中访问.
我安全忽略这个警告吗?有没有办法可以重新格式化我的代码以防止出现此警告?这段代码确实存在危险吗?
我习惯于reSharper比我更聪明,所以我想知道究竟发生了什么.
解决方法
因为session包含在using语句中,所以LINQ执行可以在枚举之前执行.
Resharper警告这可能会导致异常,因为在枚举dbRecipes时,会话可能已被释放.
老实说:我不确定上面的代码是否会以警告的方式失败.
原文地址:https://www.jb51.cc/csharp/100135.html
版权声明:本文内容由互联网用户自发贡献,该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 dio@foxmail.com 举报,一经查实,本站将立刻删除。