如果lambda内的语句用于整个查询

Question

这个陈述可以写成1个lambda表达式吗？

if (filter.SubFormId.HasValue)
{
    query = query.Where(c => c.SubFormId == filter.SubFormId);
}

Answer 1

对这种查询要非常小心.where子句绑定到变量过滤器,而不是当前值.

原始代码:

var query = from form in forms select form;
var filter = new Filter();
filter.SubFormId = 123;
if (filter.SubFormId.HasValue)
{
    query = query.Where(c => c.SubFormId == filter.SubFormId);
}
filter = null;
foreach(var matchingForm in query)
{ ... }

那段代码崩溃了.查询使用filter的当前值,而不是查询时的值.

Jared的版本并没有更好:

var query = from form in forms select form;
var filter = new Filter();
filter.SubFormId = 123;
query = query.Where(c => 
  !filter.SubFormId.HasValue || c.SubFormId == filter.SubFormId);
filter = null;
foreach(var matchingForm in query)
{ ... }

那也崩溃了.

如果过滤器以其他方式变异,那么您的版本和Jared的版本也会有不同的语义:

原始代码:

var query = from form in forms select form;
var filter = new Filter();
filter.SubFormId = null;
if (filter.SubFormId.HasValue)
{
    query = query.Where(c => c.SubFormId == filter.SubFormId);
}
filter.SubFormId = 123;
foreach(var matchingForm in query)
{ ... }

这符合每一种形式.你从未添加过Where子句.

Jared的版本做了不同的事情:

var query = from form in forms select form;
var filter = new Filter();
filter.SubFormId = null;
query = query.Where(c => 
  !filter.SubFormId.HasValue || c.SubFormId == filter.SubFormId);
filter.SubFormId = 123;
foreach(var matchingForm in query)
{ ... }

这只匹配subformId等于123的表单.Jared 的版本与过滤子表单Id的当前值匹配(如果有的话),无论是否在构建查询时都存在.

请记住,查询是表示查询的对象.执行查询时,将从头开始重新执行查询,并根据查询关闭的变量的当前值来计算查询中的子句,而不是根据查询中存在的变量值的快照进行评估.创建了.查询具有实时,最新的变量视图.

所以我要说我也返回这个查询,它允许在方法之外评估查询.由于过滤器超出范围,这会崩溃吗？

简短回答:

没有.

更长的回答:

这是这个问题的重复:

动作/ Lambda表达式内存管理问题

详情请见我的回答.

答案很长:

您将范围与生命周期混淆 - 这是一个常见错误.范围纯粹是编译时的概念; 局部变量的范围是程序文本的区域,其中该变量可以通过其名称引用.变量的生命周期纯粹是运行时概念; 局部变量的生存期是垃圾收集器知道存储有效的时间段.

范围和寿命经常相关; 虽然控制在逻辑上是在本地范围内的程序文本的一部分中,但是已知它是活着的.查询,lambda,迭代器块和异步块中使用的本地生命周期延长,因此即使控制离开本地范围,本地仍然存在.本地保持活着至少直到查询,lambda等已经死亡.

不幸的是,在某些情况下,当地的居住时间更长; 请参阅上面的链接问题以获取示例.有关此问题的扩展讨论,请参阅此问题:

C#Action,Closure和Garbage Collection