Generating Comma Separated Values

Question

Suppose I have a collection of strings:

"foo"
"bar"
"xyz"

And I would like to generate a comma separated values from the list into something like:

"foo, bar, xyz"

Notice the lack of ", " at the end.

I am aware that there are dozens of ways to generate this:

• use for-loop and string.Format() or StringBuilder.
• use integer counter and remove the ending ", " if the value > 0
• don't put ", " on the first run
• etc.

Sample code of what I have right now:

if (strs.Count() > 0)
{
  var sb = new StringBuilder();
  foreach (var str in strs)
    sb.AppendFormat("{0}, ", str);
  return sb.Remove(0, 2).ToString();
}

What is the best code that is highly reusable for the above scenario, and why?

Answer 1

You want to use the string.Join method, which exists in the BCL for this purpose.

Example:

var myArray = new string[] { "one", "two", "three" };
var output = string.Join(", ", myArray);

Or if you're using .NET 3.5, you can do this with any IEnumerable<string> as such:

var output = string.Join(", ", myEnumerable.ToArray());

(Note that this doesn't give the best performance as it requires, although it is clearly still 'O(n)', and should be suitable for almost all cases).

Now, if your enumerable is not of type string (generically an IEnumerable<T>), you can just use the Select method to convert the result into a string, e.g.

var output = string.Join(", ", myEnumerable.Select(e => e.ToString()).ToArray());

---

I'm not sure if you're dealing with values that may potentially contains commas in themselves, but this can be worked around by enclosing them in quotes (") and escaping the quotes, similarly to the CSV format.

var output = string.Join(", ", items.Select(x => x.Contains(",") ?
    "\"" + x.Replace("\"", "\"\"") + "\"" : x);

Of course, splitting them back up again is a slightly triciker task, which requires a bit of regex.

Answer 2

Use String.Join

Answer 3

string finalstr = String.Join(",", strs);

Answer 4

If you have an array of strings, go with Noldorin's solution.

But if it's some other collection type, I might do this:

if (strs.Count() > 0)
{
  var sb = new StringBuilder();
  foreach (var str in strs)
    sb.AppendFormat("{0} {1}", (0 == sb.Length ? "" : ","), str);
  return sb.ToString();
}

Answer 5

String.Join is the right answer, but in the case of an IEnumerable, Linq is often shorter than a for loop:

someStringCollection.Aggregate((first, second) => first + ", " + second);