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

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吗?


当前回答

您看过聚合扩展方法了吗?

var sa = (new[] { "yabba", "dabba", "doo" }).Aggregate((a,b) => a + "," + b);

其他回答

这里它使用纯LINQ作为单个表达式:

static string StringJoin(string sep, IEnumerable<string> strings) {
  return strings
    .Skip(1)
    .Aggregate(
       new StringBuilder().Append(strings.FirstOrDefault() ?? ""), 
       (sb, x) => sb.Append(sep).Append(x));
}

而且非常快!

为什么使用Linq?

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

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

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

你可以这样一行:

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结尾的字符串,你会失去分隔符…我觉得这不太可能。如果是这样的话,你还有其他奇特的角色可以选择。

您看过聚合扩展方法了吗?

var sa = (new[] { "yabba", "dabba", "doo" }).Aggregate((a,b) => a + "," + b);

我总是使用扩展方法:

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