Ваш код, не действительно совместимый с полный формат CSV . Если Вы просто генерируете CSV от данных, которые не имеют никаких запятых, продвижения/конечных пробелов, вкладок, новых строк или кавычек, это должно быть прекрасным. Однако в большинстве реальных сценариев обмена данными, Вам действительно нужна полная реализация.

Для поколения к надлежащему CSV, можно использовать это:

public static String EncodeCsvLine(params String[] fields)
{
    StringBuilder line = new StringBuilder();

    for (int i = 0; i < fields.Length; i++)
    {
        if (i > 0)
        {
            line.Append(DelimiterChar);
        }

        String csvField = EncodeCsvField(fields[i]);
        line.Append(csvField);
    }

    return line.ToString();
}

static String EncodeCsvField(String field)
{
    StringBuilder sb = new StringBuilder();
    sb.Append(field);

    // Some fields with special characters must be embedded in double quotes
    bool embedInQuotes = false;

    // Embed in quotes to preserve leading/tralining whitespace
    if (sb.Length > 0 && 
        (sb[0] == ' ' || 
         sb[0] == '\t' ||
         sb[sb.Length-1] == ' ' || 
         sb[sb.Length-1] == '\t' ))
    {
        embedInQuotes = true;
    }

    for (int i = 0; i < sb.Length; i++)
    {
        // Embed in quotes to preserve: commas, line-breaks etc.
        if (sb[i] == DelimiterChar || 
            sb[i]=='\r' || 
            sb[i]=='\n' || 
            sb[i] == '"') 
        { 
            embedInQuotes = true;
            break;
        }
    }

    // If the field itself has quotes, they must each be represented 
    // by a pair of consecutive quotes.
    sb.Replace("\"", "\"\"");

    String rv = sb.ToString();

    if (embedInQuotes)
    {
        rv = "\"" + rv + "\"";
    }

    return rv;
}

не Мог бы быть самый эффективный код в мире, но он был протестирован. Реальный мир сосет по сравнению с быстрым примером кода :)

, Почему бы не пользоваться одной из библиотек CSV с открытым исходным кодом там?

я знаю, что это походит на излишество для чего-то, что кажется настолько простым, но как можно сказать комментариями и фрагментами кода, существуют больше, чем кажется на первый взгляд. В дополнение к обработке полного соответствия CSV Вы в конечном счете захотите обработать и чтение и запись CSVs..., и можно хотеть управление файлом.

я использовал , Открывают CSV на одном из моих проектов прежде (но существует много других для выбора из). Это, конечно, сделало мою жизнь легче. ;)

Не забывайте нашего старого друга "для". Это не столь привлекательно как foreach, но это имеет преимущество способности запуститься во втором элементе.

public string ReturnAsCSV(ContactList contactList)
{
    if (contactList == null || contactList.Count == 0)
        return string.Empty;

    StringBuilder sb = new StringBuilder(contactList[0].Name);

    for (int i = 1; i < contactList.Count; i++)
    {
        sb.Append(",");
        sb.Append(contactList[i].Name);
    }

    return sb.ToString();
}

Вы могли также перенестись, вторые Добавляют в, "если" это тестирует, содержит ли свойство Name двойную кавычку или запятую, и если так, выйдите из них соответственно.

Вы могли вместо этого добавить запятую как первую вещь в Вашем foreach.

if (sb.Length > 0) sb.Append(",");

Вы могли также сделать массив c. Имя данные и использование Строка. Соединение метод для создания строки.

public string ReturnAsCSV(ContactList contactList)
{
    List<String> tmpList = new List<string>();

    foreach (Contact c in contactList)
    {
        tmpList.Add(c.Name);
    }

    return String.Join(",", tmpList.ToArray());
}

Это не могло бы быть столь же производительно как подход StringBuilder, но это определенно выглядит более чистым.

кроме того, Вы могли бы хотеть рассмотреть использование .CurrentCulture. TextInfo. ListSeparator вместо трудно кодированной запятой - Если Ваш вывод будет импортированным в другие приложения, у Вас могли бы быть проблемы с ним. ListSeparator может отличаться через различные культуры и MS Excel по крайней мере, соблюдает эту установку. Так:

return String.Join(
    System.Globalization.CultureInfo.CurrentCulture.TextInfo.ListSeparator,
    tmpList.ToArray());

мне нравится идея добавить запятую путем проверки, пуст ли контейнер, но разве который не означает больше обработки, поскольку это должно проверить длину строки на каждом возникновении?

Вы преждевременно оптимизируете, хит производительности был бы незначителен.

Просто мысль, но не забывают обрабатывать и кавычки запятой (") в значениях полей, иначе Ваш файл CSV может повредить потребительское средство чтения.

Я использовал этот метод прежде. Свойство Length StringBuilder не только для чтения настолько вычитающее это, каждый имеет в виду усеченный последний знак. Но необходимо удостовериться, что длина не является нулем для запуска с (который произошел бы, если список пуст), потому что установка длины к меньше, чем нулю является ошибкой.

public string ReturnAsCSV(ContactList contactList)
{
    StringBuilder sb = new StringBuilder();

    foreach (Contact c in contactList)       
    { 
        sb.Append(c.Name + ",");       
    }

    if (sb.Length > 0)  
        sb.Length -= 1;

    return sb.ToString();  
}

Как насчет некоторой обрезки?

public string ReturnAsCSV(ContactList contactList)
{
    StringBuilder sb = new StringBuilder();

    foreach (Contact c in contactList)
    {
        sb.Append(c.Name + ",");
    }

    return sb.ToString().Trim(',');
}

Как насчет того, чтобы отследить, являетесь ли Вы на первом объекте и только добавляете запятую прежде объект, если это не первое.

public string ReturnAsCSV(ContactList contactList)
{
    StringBuilder sb = new StringBuilder();
    bool isFirst = true;

    foreach (Contact c in contactList) {
        if (!isFirst) { 
          // Only add comma before item if it is not the first item
          sb.Append(","); 
        } else {
          isFirst = false;
        }

        sb.Append(c.Name);
    }

    return sb.ToString();
}