Использование LINQ для объединения строк

Какой самый эффективный способ написания старой школы:

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?

Этот ответ показывает использование LINQ ( Aggregate) в соответствии с запросом в вопросе и не предназначен для повседневного использования. Поскольку это не использует, StringBuilderэто будет иметь ужасную производительность для очень длинных последовательностей. Для обычного использования кода, String.Joinкак показано в другом ответе

Используйте совокупные запросы, как это:

string[] words = { "one", "two", "three" };
var res = words.Aggregate(
   "", // start with empty string to handle empty list case.
   (current, next) => current + ", " + next);
Console.WriteLine(res);

Это выводит:

, раз два три

Агрегат - это функция, которая принимает коллекцию значений и возвращает скалярное значение. Примеры из T-SQL включают min, max и sum. И VB, и C # поддерживают агрегаты. И VB, и C # поддерживают агрегаты в качестве методов расширения. Используя точечную запись, вы просто вызываете метод объекта IEnumerable .

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

Дополнительная информация - MSDN: агрегированные запросы

Если вы действительно хотите Aggregateиспользовать вариант с использованием StringBuilderпредложенного в комментарии CodeMonkeyKing, который будет примерно таким же, как обычный, с String.Joinхорошей производительностью для большого количества объектов:

 var res = words.Aggregate(
     new StringBuilder(), 
     (current, next) => current.Append(current.Length == 0? "" : ", ").Append(next))
     .ToString();

return string.Join(", ", strings.ToArray());

В .Net 4, есть новая перегрузка на string.Joinкоторый принимает IEnumerable<string>. Код будет выглядеть так:

return string.Join(", ", strings);

Зачем использовать Linq?

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

Это прекрасно работает и принимает любое, IEnumerable<string>насколько я помню. Здесь не нужно Aggregateничего более медленного.

Вы смотрели на метод совокупного расширения?

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

Реальный пример из моего кода:

return selected.Select(query => query.Name).Aggregate((a, b) => a + ", " + b);

Запрос - это объект со свойством Name, представляющим собой строку, и я хочу, чтобы имена всех запросов в выбранном списке были разделены запятыми.

Вот комбинированный подход Join / Linq, на котором я остановился, посмотрев на другие ответы и проблемы, затронутые в аналогичном вопросе (а именно, что Aggregate и Concatenate терпят неудачу с 0 элементами).

string Result = String.Join(",", split.Select(s => s.Name));

или (если sне строка)

string Result = String.Join(",", split.Select(s => s.ToString()));

• просто
• легко читать и понимать
• работает для общих элементов
• позволяет использовать объекты или свойства объекта
• обрабатывает регистр элементов 0-длины
• может использоваться с дополнительной фильтрацией Linq
• хорошо работает (по крайней мере, по моему опыту)
• не требует (ручного) создания дополнительного объекта (например StringBuilder) для реализации

И, конечно же, Join заботится о досадной последней запятой, которая иногда проникает в другие подходы ( for, foreach), именно поэтому я искал решение Linq в первую очередь.

Вы можете использовать StringBuilderв Aggregate:

  List<string> strings = new List<string>() { "one", "two", "three" };

  StringBuilder sb = strings
    .Select(s => s)
    .Aggregate(new StringBuilder(), (ag, n) => ag.Append(n).Append(", "));

  if (sb.Length > 0) { sb.Remove(sb.Length - 2, 2); }

  Console.WriteLine(sb.ToString());

(Это Selectтолько для того, чтобы показать, что вы можете делать больше вещей LINQ.)

быстрые данные о производительности для случая StringBuilder и Select & Aggregate для более чем 3000 элементов:

Модульный тест - Длительность (секунды) LINQ_StringBuilder - 0.0036644
LINQ_Select.Aggregate - 1.8012535

    [TestMethod()]
    public void LINQ_StringBuilder()
    {
        IList<int> ints = new List<int>();
        for (int i = 0; i < 3000;i++ )
        {
            ints.Add(i);
        }
        StringBuilder idString = new StringBuilder();
        foreach (int id in ints)
        {
            idString.Append(id + ", ");
        }
    }
    [TestMethod()]
    public void LINQ_SELECT()
    {
        IList<int> ints = new List<int>();
        for (int i = 0; i < 3000; i++)
        {
            ints.Add(i);
        }
        string ids = ints.Select(query => query.ToString())
                         .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);
}

Под « супер-классным способом LINQ » вы можете говорить о том, как LINQ делает функциональное программирование намного более приемлемым с использованием методов расширения. Я имею в виду синтаксический сахар, который позволяет связывать функции визуально линейным способом (один за другим) вместо вложения (один внутри другого). Например:

int totalEven = Enumerable.Sum(Enumerable.Where(myInts, i => i % 2 == 0));

можно написать так:

int totalEven = myInts.Where(i => i % 2 == 0).Sum();

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

Многие другие ответы утверждают, что String.Joinэто путь, потому что он самый быстрый или простой для чтения. Но если вы возьмете мою интерпретацию « супер-крутого пути LINQ », то ответ будет использовать, String.Joinно оберните его в метод расширения стиля LINQ, который позволит вам связать ваши функции визуально приятным способом. Так что если вы хотите написать, sa.Concatenate(", ")вам просто нужно создать что-то вроде этого:

public static class EnumerableStringExtensions
{
   public static string Concatenate(this IEnumerable<string> strings, string separator)
   {
      return String.Join(separator, strings);
   }
}

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

Существуют различные альтернативные ответы на этот предыдущий вопрос - который, по общему признанию, был нацелен на целочисленный массив в качестве источника, но получил обобщенные ответы.

Здесь он использует чистый 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));
}

И это чертовски быстро!

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

Таким образом, вы можете в одну строку это:

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, вы потеряете свои разделители ... Я считаю это маловероятным. Если это так, у вас есть другие необычные персонажи на выбор.

Вы можете комбинировать LINQ и string.join()довольно эффективно. Здесь я удаляю элемент из строки. Есть и лучшие способы сделать это, но вот оно:

filterset = String.Join(",",
                        filterset.Split(',')
                                 .Where(f => mycomplicatedMatch(f,paramToMatch))
                       );

Много вариантов здесь. Вы можете использовать LINQ и StringBuilder, чтобы получить производительность, например, так:

StringBuilder builder = new StringBuilder();
List<string> MyList = new List<string>() {"one","two","three"};

MyList.ForEach(w => builder.Append(builder.Length > 0 ? ", " + w : w));
return builder.ToString();

Я быстро и грязно сделал следующее при разборе файла журнала IIS с помощью linq, он сработал примерно на 1 миллион строк (15 секунд), хотя при попытке выполнить 2 миллиона строк возникла ошибка нехватки памяти.

    static void Main(string[] args)
    {

        Debug.WriteLine(DateTime.Now.ToString() + " entering main");

        // USED THIS DOS COMMAND TO GET ALL THE DAILY FILES INTO A SINGLE FILE: copy *.log target.log 
        string[] lines = File.ReadAllLines(@"C:\Log File Analysis\12-8 E5.log");

        Debug.WriteLine(lines.Count().ToString());

        string[] a = lines.Where(x => !x.StartsWith("#Software:") &&
                                      !x.StartsWith("#Version:") &&
                                      !x.StartsWith("#Date:") &&
                                      !x.StartsWith("#Fields:") &&
                                      !x.Contains("_vti_") &&
                                      !x.Contains("/c$") &&
                                      !x.Contains("/favicon.ico") &&
                                      !x.Contains("/ - 80")
                                 ).ToArray();

        Debug.WriteLine(a.Count().ToString());

        string[] b = a
                    .Select(l => l.Split(' '))
                    .Select(words => string.Join(",", words))
                    .ToArray()
                    ;

        System.IO.File.WriteAllLines(@"C:\Log File Analysis\12-8 E5.csv", b);

        Debug.WriteLine(DateTime.Now.ToString() + " leaving main");

    }

Реальная причина, по которой я использовал linq, заключалась в необходимости использовать Distinct (), который мне ранее был необходим:

string[] b = a
    .Select(l => l.Split(' '))
    .Where(l => l.Length > 11)
    .Select(words => string.Format("{0},{1}",
        words[6].ToUpper(), // virtual dir / service
        words[10]) // client ip
    ).Distinct().ToArray()
    ;

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

http://ondevelopment.blogspot.com/2009/02/string-concatenation-made-easy.html

В посте блога описывается, как реализовать методы расширения, которые работают в IEnumerable и называются Concatenate, это позволит вам писать такие вещи, как:

var sequence = new string[] { "foo", "bar" };
string result = sequence.Concatenate();

Или более сложные вещи, такие как:

var methodNames = typeof(IFoo).GetMethods().Select(x => x.Name);
string result = methodNames.Concatenate(", ");