Linq“选择”理智问题

Question

我的逻辑类似于下面的代码。似乎为了检查这样的空引用，我必须有两个单独的 Select 语句。 linq ninja 能告诉我如何改进这样的查询吗？

List<C> cList = Globals.GetReferences(irrelevantThing) // GetReferences returns List<A>, which are references to B
            .Select(a => a.GetB()) // GetB returns type B, can be null
            .Where(b => b != null && someOtherConditions)
            .Select(c => new C(b)) // C Constructor cannot have a null parameter
            .ToList();

谢谢。

编辑：稍微清理了示例代码。

Answer 1

好吧，首先我会更改参数名称 - 在第一个 Select 之后，您会得到一个 B，那么为什么不将其称为 b 呢？

List<B> bList = Globals.GetReferences(irrelevantThing)
            .Select(a => a.GetB()) // GetB returns type B, can be null
            .Where(b => b != null && someOtherConditions)
            .Select(b => new B(b))
            .ToList();

那时，我想知道为什么你有第二个Select...你已经有了一个 B，那么你为什么要创建一个新的呢？ B(B old) 构造函数有什么作用？

如果您确实需要其他选择，那对我来说似乎没问题。我的意思是，如果 GetB 很便宜，你总是可以使用：

List<B> bList = Globals.GetReferences(irrelevantThing)
            .Where(a => a.GetB() != null && someOtherConditions)
            .Select(a => new B(a.GetB()))
            .ToList();

...但说实话，我不确定我会不会。

Answer 2

我发现使用查询语法和 let 运算符看起来稍微更好一些：

   var queryB = from a in Globals.GetReferences(irrelevantThing)
                let b = a.GetB()
                where b != null && someOtherConditions
                select new B(b);

   var bList = queryB.ToList()

不过取决于您的偏好...

仅供参考，在扩展方法语法中，上面的内容转换为

   var queryB = Globals.GetReferences(irrelevantThing)
            .Select(a => new { a, b = a.GetB() })
            .Where(x => x.b != null && someOtherConditions)
            .Select(x => new B(x.b))
            .ToList();

更新：刚刚意识到您也可以使用select into 子句从字面上翻译确切的原始查询：

   var queryB = from a in Globals.GetReferences(irrelevantThing)
                select a.GetB() into b
                where b != null && someOtherConditions
                select new B(b);

不过仍然更喜欢 let...