连接对象的字符串表示的好方法？

Question

好的，

我们的代码中有很多 where 子句。我们有多种方法来生成表示 in 条件的字符串。我试图想出一种干净的方法，如下所示：

public static string Join<T>(this IEnumerable<T> items, string separator)
{
    var strings = from item in items select item.ToString();
    return string.Join(separator, strings.ToArray());
}

它可以按如下方式使用：

var values = new []{1, 2, 3, 4, 5, 6};
values.StringJoin(",");
// result should be:
// "1,2,3,4,5,6"

所以这是一个很好的扩展方法，可以完成非常基本的工作。我知道简单的代码并不总是能够快速或高效地执行，但我只是好奇这个简单的代码可能会错过什么。我们团队的其他成员认为：

• 它不够灵活（无法控制字符串表示）
• 可能内存效率不高
• 可能速度不快

有专家插话吗？

问候，

埃里克。

Answer 1

关于第一个问题，您可以添加另一个“formatter”参数来控制每个项目到字符串的转换：

public static string Join<T>(this IEnumerable<T> items, string separator)
{
    return items.Join(separator, i => i.ToString());
}

public static string Join<T>(this IEnumerable<T> items, string separator, Func<T, string> formatter)
{
    return String.Join(separator, items.Select(i => formatter(i)).ToArray());
}

关于后两个问题，我不会担心它，除非您后来遇到性能问题并发现它是一个问题。然而，这不太可能成为瓶颈......

Answer 2

出于某种原因，我认为 String.Join 是根据 StringBuilder 类实现的。但如果不是，那么以下代码可能对于大型输入表现更好，因为它不会为迭代中的每个连接重新创建 String 对象。

public static string Join<T>(this IEnumerable<T> items, string separator)
{
    // TODO: check for null arguments.
    StringBuilder builder = new StringBuilder();
    foreach(T t in items)
    {
        builder.Append(t.ToString()).Append(separator);
    }

    builder.Length -= separator.Length;
    return builder.ToString();
}

编辑：这是分析何时适合使用StringBuilder和String.Join。

Answer 3

为什么不使用 StringBuilder，并自己迭代集合并追加。
否则，您将创建一个字符串数组（var strings），然后进行连接。

Answer 4

您缺少对序列和序列项的空检查。是的，这不是最快和最有效的内存方式。人们可能只是枚举序列并将项目的字符串表示形式呈现到 StringBuilder 中。但这真的重要吗？您遇到性能问题吗？需要优化吗？

Answer 5

这也可以工作：

public static string Test(IEnumerable<T> items, string separator)
{
    var builder = new StringBuilder();
    bool appendSeperator = false;
    if(null != items)
    {
        foreach(var item in items)
        {
            if(appendSeperator)
            {
                builder.Append(separator)
            }

            builder.Append(item.ToString());

            appendSeperator = true;
        }
   }

   return builder.ToString();
}