为什么 ReSharper 建议我将 for 循环转换为 LINQ 表达式？

Question

在 Visual Studio Re-Sharper 中不断建议我将 for 循环转换为 linq 表达式，但这是什么原因？

哪个更快？

以下是一些示例循环，其中 resharper 建议进行 linq 转换：

foreach (XmlNode legendEntryNode in _legendEntryNodes)
{
    var xmlElement = legendEntryNode["FeatureType"];

    if (xmlElement == null || !xmlElement.InnerText.Equals(featuretype)) continue;

    var xmlNodeList = legendEntryNode.SelectNodes("Themes/Theme");

    if (xmlNodeList != null)

    foreach (XmlNode themeNode in xmlNodeList)
    {
        var element = themeNode["Value"];

        if (element == null || !element.InnerText.Equals(v)) continue;

        var xmlElement1 = themeNode["Icon"];

        if (xmlElement1 != null)
        {
            string iconname = "<ms:ICON>" + xmlElement1.InnerText + "</ms:ICON>";

            var element1 = themeNode["Highlight"];

            if (element1 != null)
            {
                string highlightname = "<ms:HIGHLIGHT>" + element1.InnerText + "</ms:HIGHLIGHT>";
                gml = gml.Insert(c, iconname + highlightname);

                c += (iconname.Length + highlightname.Length);
            }
        }
        break;
    }
}

还有这个更简单的例子：

for (int i = 0; i < getPointsRequest.Attribs.Length; i++)
{
    string attribName = getPointsRequest.Attribs[i].AttributeName;

    if (!String.IsNullOrEmpty(attribName))
    {
        sqlQuery += "<ms:" + attribName + ">||\"" + attribName + "\"||</ms:" + attribName + ">";
    }
}

Answer 1

速度通常与大部分代码无关 - 您应该以最简单的方式编写代码，然后对其进行测量以确保它足够快。 p>

如果您的 for 循环真的只是查询，那么 LINQ绝对 是获得更易读代码的好方法。它不是普遍适用的，但您至少应该牢记经常。

通常可以将 for 循环转换为要延迟评估的查询，然后是 foreach 循环，该循环对查询返回的每个值执行一些操作。这有助于将这两个方面分开，让您在阅读代码时一次只专注于一个方面。重要的是保持 LINQ 查询 查询，而不是在其中使用副作用 - 它被设计为具有一种功能性方法，它确实不会与副作用很好地混合。

如果你有一些具体的例子，我们可以就哪些循环转换为使用 LINQ 有意义，哪些没有意义提供更多意见。

Answer 2

这样没有性能提升，但有一些好处

• 使代码更具可读性。
• 减少行数。
• 易于维护。
• 在某些情况下，您不需要临时变量，而 for 循环中可能需要这些变量。使用 Linq，您可以链接查询。

更多详情可以参考：

• LINQ query operators: lose that foreach already
• The “Anti-For” Campaign
• Life After Loops

希望对你有所帮助。

Answer 3

速度上可能没有差异，但是使用 Linq 通常会产生更简洁的代码。 这并不是说您应该始终接受 R# 转换为 Linq 表达式的建议。有时复杂但易于理解的 foreach 循环会转换为有效但不易理解的 Linq 表达式。

Answer 4

一般来说，ReSharper 的建议只是建议，没有警告。因此，您可以自行决定采用哪种方式：LINQ 还是 foreach。
我对建议“使用'var'”有同样的问题。只有当我认为读者可以更好地阅读该声明时，我才会点击该建议。
在编写代码时，可读性是我的首要任务之一。

Answer 5

我会说有时不转换是有原因的。 ReSharper 没有提供重构以将 LINQ 表达式（返回）转换为 for 循环，这可能不太令人钦佩。我有几次将循环转换为表达式，然后想在循环中放入一些进一步的操作（通常是调试操作）；我必须手动将它们转换回来。

我会警告不要在没有充分理由的情况下转换 for 循环。很多时候它确实并没有提高可读性，而且没有任何其他强有力的理由这样做（正如其他人正确指出的那样，大多数循环对速度不是很关键）。

我认为一些 for 循环比 LINQ 等价物更具可读性，因为它们在视觉上将循环的动作分解成小块。我会说它往往是小循环（三或四行），通过将它们变成一行的表达式来得到最大的改进。

[对不起，这篇文章主要是意见，但可读性是一个主观主题。不要拖钓！]

Answer 6

嗨，Linq 实际上是在内部调用一个 for 循环。我想归结为 Linq 表达式通常更容易阅读/维护。如果您真的关心性能，可以将两者进行很好的比较：http://geekswithblogs.net/BlackRabbitCoder/archive/2010/04/23/c-linq-vs-foreach---round-1.aspx

Answer 7

作为其他人的参考，这里是 Resharper 建议的 for 循环和 for 循环的示例

for (int x = 0; x < grid.Length; x++)
        {
            var intCount = grid[x].Select((a, b) => new {Value = a, Index = b})
                .GroupBy(y => y.Value)
                .Where(y => y.Count() > 1).Select(item => item.Key).ToArray();

            if (intCount.Count() > 1)
                return false;
        }

为了解释这段代码，这个 for 循环将获取数组中的所有重复项。获取所有重复项后，检查项数是否大于 1，然后返回 false。

这是 LINQ 中建议的 for 循环：

  return grid.Select(t => t.Select((a, b) => new {Value = a, Index = b}).
            GroupBy(y => y.Value).Where(y => y.Count() > 1).
            Select(item => item.Key).ToArray()).All(intCount => intCount.Count() <= 1);

可能没有性能提升，但是从示例中可以看出，LINQ 查询更干净、易于阅读、行数更少（在这种情况下，只有一行代码，我只是在粘贴后对其进行了调整在这里）并且也易于调试。