使用 if - else if 结构将 foreach 重构为 LINQ

Question

我有这个 foreach 循环：

var includedElements = new HashSet<int>();
foreach(var e in elements)
{
    var include = false;
    if(isTable(e.Key))
    {
        if(tables.ContainsKey(e.Key)
        {
            if(tables[e.Key].Elements
               .Any(subElem => shouldBeIncluded(subElem.Key) ) )
            {
                include = true;
            }
        }
    }
    else if(shouldBeIncluded(e.Key))
    {
        include = true;
    }
    if(include){
        includedElements.Add(e.Key);
        DoSomeMoreStuff(e);
    }
}

我尝试将其重构为 LINQ：

var query = 
    from e in elements
    where 
    ( 
        isTable(e.Key)
        && tables.ContainsKey(e.Key)
        && tables[e.Key].Elements
                .Any(subElem => shouldBeIncluded(subElem.Key) )
    ) || (
        !isTable(e.Key)
        && shouldBeIncluded(e.Key)
    )
    select e;
foreach(e in query){
    includedElements.Add(e.Key);
    DoSomeMoreStuff(e);
}

我不确定的是这里的 or 子句。在我的脑海中，我需要包含 !isTable(e.Key) 来处理外部 if/else if 结构。
我的重构思路正确吗？这两个代码示例是否产生相同的逻辑功能？

这是一种我只需调用一次 isTable 就可以逃脱的方法吗？正如我现在所拥有的，我需要在 || 的另一侧将其称为倒置。

Answer 1

是的你是对的。 if isTable 没有副作用（除了检查某些内容之外不执行任何操作）并且基于参数是确定性的（因此使用 e.Key 调用它两次总是会产生相同的值）。尽管如此，它可能（它可能是一个过早的优化......谁知道呢？）可能更好地保持它与原始if更加相似并使用三元运算符 (? :)，这样就不必重新检查 isTable

var query = 
    from e in elements
    where 
        isTable(e.Key) ? 

            tables.ContainsKey(e.Key) && tables[e.Key].Elements
                .Any(subElem => shouldBeIncluded(subElem.Key) ) 
        :

            shouldBeIncluded(e.Key)
    select e;

我会补充一点，如果您讨厌三元运算符，您可以使用 let 关键字：

var query = 
    from e in elements
    let isT = isTable(e.Key)
    where 
        ( isT && tables.ContainsKey(e.Key) && tables[e.Key].Elements
            .Any(subElem => shouldBeIncluded(subElem.Key) ) )
            ||
        ( !isT && shouldBeIncluded(e.Key) )
    select e;

缓存 isTable(e.Key)

Answer 2

你是对的。 else if 暗示 if 条件不匹配，因此

if(A) { 
    if(B) { 
        if(C) { 
            include = true; 
        } 
    } 
}
else if(D) {
    include = true;
}

相当于

if(A) { 
    if(B) { 
        if(C) { 
            include = true; 
        } 
    } 
}

if(!A && D) {
    include = true;
}

which 相当于

if ((A && B && C) || (!A && D)) {
    include = true;
}

which 正是您在 LINQ 中编写的内容。