我们在Where子句中使用了一个lambda表达式,它不返回“expected”结果.
快速总结
在analyzeObjectRepository对象中,有一些对象也包含一个名为Parent的属性中的父关系.我们正在查询此analysisObjectRepository以返回一些对象.
详情
下面的代码应该是返回根,包含ID值的特定对象的第一个孩子(直接子代)和子孙.
在下面的代码中,常识说,所有结果,使任何3个独立OR条件为真应返回,如结果.
List<AnalysisObject> analysisObjects =
analysisObjectRepository
.FindAll()
.Where(x => x.ID == packageId ||
x.Parent.ID == packageId ||
x.Parent.Parent.ID == packageId)
.ToList();
但上面的代码只返回子孙,而不返回根对象(有一个null的父值),这使得
x.ID == packageId
条件是真的.
只有第二个对象
x.Parent.ID == packageId
和第三
x.Parent.Parent.ID == packageId
返回条款.
如果我们只使用下面的代码编写代码来返回根对象,那么它将被返回,所以我们完全确定analyzeObjectRepository包含所有的对象
List<AnalysisObject> analysisObjects =
analysisObjectRepository
.FindAll()
.Where(x => x.ID == packageId )
.ToList();
但是,当我们将其重写为代理时,我们得到预期的结果,返回所有预期的对象.
List<AnalysisObject> analysisObjects =
analysisObjectRepository
.FindAll()
.Where(delegate(AnalysisObject x)
{
return
(x.ID == packageId) ||
(x.Parent != null && x.Parent.ID == packageId) ||
(x.Parent != null &&
x.Parent.Parent != null &&
x.Parent.Parent.ID == packageId); })
.ToList();
题
我们在lambda表达式中缺少一些东西吗？这是一个非常简单的3部分OR条件,我们认为任何使三个条件中的任一条件为真的对象都应该被返回.我们怀疑具有null Parent值的根对象可能会导致问题,但无法准确计算出来.
任何帮助都会很棒.
解决方法
第一：
x.ID == packageId || x.Parent.ID == packageId || x.Parent.Parent.ID == packageId
第二：
(x.ID == packageId) || (x.Parent != null && x.Parent.ID == packageId) || (x.Parent != null && x.Parent.Parent != null && x.Parent.Parent.ID == packageId)
对lambda的调用将抛出一个异常,其中的任何一个ID不匹配,父对象为null或不匹配,并且祖父母为空.将空检查复制到lambda中,它应该正常工作.
编辑评论后问题
如果您的原始对象不是List< T>,那么我们无法知道FindAll()的返回类型,以及这是否实现了IQueryable接口.如果是这样,那么这可能解释了差异.因为lambdas可以在编译时被转换成Expression< Func>>但是匿名委托不能,那么在使用lambda版本时使用IQueryable,而在使用匿名委托版本时可能会使用LINQ对对象.
这也将解释为什么你的lambda不会导致NullReferenceException.如果要将该lambda表达式传递给实现IEnumerable< T>但是不是IQueryable< T>的lambda(与其他方法没有任何差异)的运行时间评估会在第一次遇到ID不等于目标的对象时抛出NullReferenceException,父或祖父是空值.
增加了3/16/2011 8:29 AM EDT
考虑以下简单的例子：
IQueryable<MyObject> source = ...; // some object that implements IQueryable<MyObject>
var anonymousMethod = source.Where(delegate(MyObject o) { return o.Name == "Adam"; });
var expressionLambda = source.Where(o => o.Name == "Adam");
这两种方法产生完全不同的结果.
第一个查询是简单的版本.匿名方法导致一个委托,然后传递给IEnumerable< MyObject>.在扩展方法中,将根据您的委托检查源的整个内容(使用普通的编译代码手动在内存中).换句话说,如果你熟悉C#中的迭代器块,那就像这样做：
public IEnumerable<MyObject> MyWhere(IEnumerable<MyObject> dataSource,Func<MyObject,bool> predicate)
{
foreach(MyObject item in dataSource)
{
if(predicate(item)) yield return item;
}
}
这里的重点是您实际上是在客户端的内存中进行过滤.例如,如果你的源是一些sql ORM,查询中就不会有WHERE子句;整个结果集将被带回客户端并在那里过滤.
使用lambda表达式的第二个查询被转换为Expression< Func< MyObject,bool>>>并使用IQueryable< MyObject> .Where()扩展方法.这导致一个也被输入为IQueryable< MyObject>的对象.所有这些都通过将表达式传递给底层提供程序.这就是为什么你没有得到NullReferenceException.查询提供程序完全取决于如何翻译表达式(而不是实际编译的函数,它可以调用,它是表示使用对象的表达式的逻辑)到它可以使用的东西.
一个简单的方法来看待区别(或至少有一个区别),就是在调用lambda版本之前调用AsEnumerable().这将强制您的代码使用LINQ对对象(这意味着它像IEnumerable< T>像匿名委托版本,而不是IQueryable< T>像lambda版本当前一样),你会得到预期的异常.
TL; DR版本
很长一段时间,你的lambda表达式被转换成对数据源的某种查询,而匿名方法版本正在评估内存中的整个数据源.无论做什么,将lambda转换成查询都不代表您期望的逻辑,这就是为什么它不会产生您期望的结果.