扩展方法中的lambda什么时候做得太多?

我意识到这部分是主观的,但我对社区意见普遍感到好奇,并且还没有成功地找到解决这个问题的现有问题。

我正在和一位同事就L2EF查询中的特定select语句进行某种宗教性的辩论。

.Select(r =>
{
    r.foo.Bar = r.bar;
    r.foo.Bar.BarType = r.Alpha;
    if (r.barAddress != null)
    {
        r.foo.Bar.Address = r.barAddress;
        r.foo.Bar.Address.State = r.BarState;
    }
    if (r.baz != null)
    {
        r.foo.Bar.Baz = r.baz;
        if (r.bazAddress != null)
        {
            r.foo.Bar.Baz.Address = r.bazAddress;
            r.foo.Bar.Baz.Address.State = r.BazState;
        }
    }
    return r.foo;
})

Caveats:

• 这是实体的Linq
• 这是 之后 数据库中已执行并返回的工作
• 输入参数 r 是匿名的

就我个人而言,我认为(a)select子句不应该改变值,它应该只是投影。他的反驳是,他没有改变任何东西,他只是确保所有的东西都在数据库查询的结果中正确初始化。其次,我认为一旦他开始进入完整的代码块并返回语句,是时候定义一个方法,甚至是 Func<T, U> 并不是所有这些都是内联的。这里的复杂因素是,输入也是匿名的,所以需要定义一个类型。但是,如果不是具体的话,我们仍然在讨论一般的观点。

那么,lambda表达式什么时候做的太多?你在沙地的什么地方画模糊的线?