用于检测 String.Concat 的使用/滥用的工具（应使用 StringBuilder）

Question

众所周知，不应使用 StringBuilder 来代替少量连接：

string s = "Hello";
if (greetingWorld)
{
    s += " World";
}

s += "!";

但是，在大小较大的循环中，StringBuilder 是显而易见的选择：

string s = "";
foreach (var i in Enumerable.Range(1,5000))
{
    s += i.ToString(); // <- bad idea!
}

Console.WriteLine(s);

是否有一个工具可以在原始 C# 源代码或已编译的程序集以识别源代码中何处调用了String.Concat？ （如果您不熟悉，s += "foo" 会映射到 IL 输出中的 String.Concat。）显然，我无法实际搜索整个项目并评估每个 += 以确定左值是否是字符串。

理想情况下，它只会指出 for/foreach 循环内的调用，但我什至会忍受所有注意到 每个 String.Concat 的误报。另外，我知道有一些重构工具会自动重构我的代码以使用 StringBuilder，但我此时只对识别 Concat 用法感兴趣。

我经常在我的代码上运行 Gendarme 和 FxCop，但这两个工具都无法识别我所描述的内容。但是，如 @Cristian 指出，旧版本的 FxCop 用于检查这一点。也许有一种方法可以从旧版本的 FxCop 中提取该规则并告诉新版本（1.36）使用它？

Answer 1

也许 NDepend CQL（代码查询语言）对此足够表达。不确定是否是这样。

Answer 2

FxCop 有对此的一些建议。检查 这篇文章

例如根据此代码中的文章：

static string BadConcatenate(string[] items)
{
    string strRet = string.Empty;

    foreach(string item in items)
    {
        strRet += item;
    }

    return strRet;
}

FxCop reports

"Change StringCompareTest.BadConcatenate(String[]):String to use StringBuilder 
  instead of String.Concat or +

编辑

规则 CA1807 似乎已被删除因为高噪音或不再适用的分析。看起来编译器不会自动替换它，在同一链接中，他们详细阐述了这两种方法的性能。