写老派最有效的方法是什么:

StringBuilder sb = new StringBuilder();
if (strings.Count > 0)
{
    foreach (string s in strings)
    {
        sb.Append(s + ", ");
    }
    sb.Remove(sb.Length - 2, 2);
}
return sb.ToString();

...在LINQ吗?


当前回答

我对字符串进行了基准测试。使用BDN在15个字符串数组上Join vs .Aggregate:

Method Mean Error StdDev Gen0 Allocated
String_Join 92.99 ns 9.905 ns 0.543 ns 0.0560 352 B
LING_Aggregate 406.00 ns 74.662 ns 4.092 ns 0.4640 2912 B

阵列越大,间隔越大

其他回答

我要稍微欺骗一下,给出一个新的答案,它似乎总结了这里所有最好的东西,而不是把它粘在注释里。

你可以这样一行:

List<string> strings = new List<string>() { "one", "two", "three" };

string concat = strings        
    .Aggregate(new StringBuilder("\a"), 
                    (current, next) => current.Append(", ").Append(next))
    .ToString()
    .Replace("\a, ",string.Empty); 

编辑:你要么先检查一个空的枚举对象,要么添加一个.Replace("\a",string.Empty);到表达式的末尾。我想我可能是太聪明了。

来自@a.friend的答案可能会稍微更好一些,我不确定Replace与Remove相比在底层做了什么。唯一的另一个警告是,如果你想连接以\a结尾的字符串,你会失去分隔符…我觉得这不太可能。如果是这样的话,你还有其他奇特的角色可以选择。

我代码中的真实例子:

return selected.Select(query => query.Name).Aggregate((a, b) => a + ", " + b);

查询是一个具有Name属性的对象,该属性是一个字符串,我想要所选列表上所有查询的名称,用逗号分隔。

这个答案显示了问题中要求的LINQ(聚合)的使用情况,而不是用于日常使用。因为它没有使用StringBuilder,所以对于很长的序列,它的性能会很糟糕。对于常规代码,使用String。如另一个答案所示

像这样使用聚合查询:

string[] words = { "one", "two", "three" };
var res = words.Aggregate(
   "", // start with empty string to handle empty list case.
   (current, next) => current + ", " + next);
Console.WriteLine(res);

这个输出:

, one, two, three

聚合是一个接受值集合并返回标量值的函数。T-SQL中的例子包括min、max和sum。VB和c#都支持聚合。VB和c#都支持聚合作为扩展方法。使用点表示法,可以简单地调用IEnumerable对象的方法。

记住,聚合查询是立即执行的。

更多信息- MSDN:聚合查询


如果你真的想使用聚合,使用CodeMonkeyKing在注释中提出的StringBuilder的变体,这将是与常规字符串相同的代码。Join包括对大量对象的良好性能:

 var res = words.Aggregate(
     new StringBuilder(), 
     (current, next) => current.Append(current.Length == 0? "" : ", ").Append(next))
     .ToString();

我总是使用扩展方法:

public static string JoinAsString<T>(this IEnumerable<T> input, string seperator)
{
    var ar = input.Select(i => i.ToString());
    return string.Join(seperator, ar);
}

为什么使用Linq?

string[] s = {"foo", "bar", "baz"};
Console.WriteLine(String.Join(", ", s));

这是完美的工作,并接受任何IEnumerable<string>,据我所知。不需要在这里聚合任何东西,这是非常慢的。