Преобразование List <int> в строку, разделенную запятыми


116

Есть ли способ взять список и преобразовать его в строку, разделенную запятыми?

Я знаю, что могу просто зациклить и построить его, но почему-то я думаю, что некоторые из вас, ребята, могут это сделать более круто?

Я действительно хочу изучить эти типы «уловок», поэтому, пожалуйста, объясните или дайте ссылку на документацию по методу, который вы используете.

Ответы:


189
List<int> list = ...;
string.Join(",", list.Select(n => n.ToString()).ToArray())

7
Умный, но медленный и раздутый, так как выделяет одну строку на элемент. Использование StringBuilder было бы намного эффективнее.
Стивен Судит, 06

3
Из того, что я видел в Интернете (быстрый поиск), String.Join работает быстрее, чем использование StringBuilder.
Юрий Факторович


6
Я думаю, что Стивен имеет в виду часть n.ToString (), а не String.Join.
Larsenal 06

9
Larsenal: но StringBuilder.Append (Int32) в любом случае внутренне вызывает ToString для целого числа. StringBuilder не позволяет волшебным образом избежать затрат на выделение строки для каждого элемента; он просто красиво убирает его из поля зрения.
itowlson 07

115

Простое решение

List<int> list = new List<int>() {1,2,3};
string.Join<int>(",", list)

Я использовал это только сейчас в своем коде, работая очень быстро.


1
Спасибо! это прекрасный подход
Ирфан Ашраф

2
Это лучший подход, чем принятый ответ. При таком подходе вам не нужно импортировать Linq, и это быстрее.
JoKeRxbLaCk

Прохладно! Я никогда не знал string. Join имеет общие перегрузки. Спасибо.
mrmashal


6

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

http://blogs.msdn.com/ericlippert/archive/2009/04/15/comma-quibbling.aspx

и комментарий StackOverflow:

Вызов Эрика Липперта - «придираться к запятым», лучший ответ?


Спасибо за ссылку. Эта проблема конкатенации строк оказалась более сложной и познавательной, чем я ожидал!
Стивен Судит,

4

Для большей крутизны я бы сделал это методом расширения для IEnumerable <T>, чтобы он работал на любом IEnumerable:

public static class IEnumerableExtensions {
  public static string BuildString<T>(this IEnumerable<T> self, string delim = ",") {
    return string.Join(delim, self)        
  }
}

Используйте его следующим образом:

List<int> list = new List<int> { 1, 2, 3 };
Console.WriteLine(list.BuildString(", "));

Две возможные оптимизации: 1) Независимо от каждого элемента добавьте разделитель, а затем удалите лишний после завершения цикла. 2) Укажите емкость для StringBuilder.
Стивен Судит, 07

1
Если вы откопаете Reflector, то окажется, что Join суммирует длины для предварительного расчета размера буфера, а также «запускает насос», добавляя первую строку вне цикла, а затем внутри цикла безоговорочно добавляя разделитель перед следующая строка. В сочетании с некоторыми небезопасными / внутренними уловками это должно быть очень быстро.
Стивен Судит, 07

@ Стивен: последовал твоему совету.
cdiggins

1
Вы жестко кодируете разделитель в своем расширении и игнорируете переданное значение в качестве разделителя и пропускаете точку с запятой. Должно бытьreturn string.Join(delim, self);
Эндрю

1

Кажется достаточно быстрым.

IList<int> listItem = Enumerable.Range(0, 100000).ToList();
var result = listItem.Aggregate<int, StringBuilder, string>(new StringBuilder(), (strBuild, intVal) => { strBuild.Append(intVal); strBuild.Append(","); return strBuild; }, (strBuild) => strBuild.ToString(0, strBuild.Length - 1));

1

Моя «умная» запись:

        List<int> list = new List<int> { 1, 2, 3 };
        StringBuilder sb = new StringBuilder();
        var y = list.Skip(1).Aggregate(sb.Append(x.ToString()),
                    (sb1, x) =>  sb1.AppendFormat(",{0}",x));

        // A lot of mess to remove initial comma
        Console.WriteLine(y.ToString().Substring(1,y.Length - 1));

Просто не придумал, как условно добавить запятую.


1
Пожалуйста, не пишите Selectс побочными эффектами в лямбде. В этом случае вы даже не используете y, так что ваш, Selectпо сути, просто foreach- так что напишите его как таковой.
Павел Минаев

Я не предлагал это как хорошее решение. OP хотел чего-то более интересного, чем foreach.
Ларсенал 07

Да, но злоупотребление , Selectкак foreachпроходит мимо «интересным» и в, ну, «злоупотребление». Более интересным подходом здесь было бы использование Enumerable.Aggregateс StringBuilderв качестве начального значения - попробуйте.
Павел Минаев

Отличная идея. Мне нужно выйти, но я могу дать этому поворот.
Ларсенал 07

0

вы можете использовать библиотеку System.Linq; Это более эффективно:

using System.Linq;
string str =string.Join(",", MyList.Select(x => x.NombreAtributo));
Используя наш сайт, вы подтверждаете, что прочитали и поняли нашу Политику в отношении файлов cookie и Политику конфиденциальности.
Licensed under cc by-sa 3.0 with attribution required.