LINQを使用して文字列を連結する


345

オールドスクールを書くための最も効率的な方法は何ですか:

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で?


1
何かを行うための他の超クールなLINQの方法を発見しましたか?
ロバートS.

3
選択した回答と他のすべてのオプションがLinq to Entitiesで機能しません。
Binoj Antony

3
@Binoj Antony、データベースで文字列連結を行わないでください。
エイミーB

6
@ Pr0fess0rX:それができないため、そしてそうすべきではないため。他のデータベースについては知りませんが、SQL Serverでは、(n)varcahr(max)に制限する(n)varcahrのみを連結できます。ビジネスロジックをデータレイヤーに実装しないでください。
the_drow

完全なソースコードと高性能を備えた最終的なソリューションはありますか?
Kiquenet 2012

回答:


527

この回答はAggregate、質問で要求されているLINQ()の使用法を示しており、日常的な使用を目的としていません。これはaを使用しないため、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:集計クエリ


CodeMonkeyKingによって提案されたコメントAggregateを使用してバリアントを使用したい場合は、通常のコードとほぼ同じで、多数のオブジェクトのパフォーマンスが向上します。StringBuilderString.Join

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

4
最初の例は「1、2、3」を出力せず、「、1、2、3」を出力します(先頭のコンマに注意してください)。
Mort

最初の例では、でシードする""ため、で使用される最初の値currentは空の文字列です。したがって、1つ以上の要素の場合、常に, 文字列の先頭に移動します。
マイケル・ヤンニ

@Mort私はこれを修正しました
sergtk

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

.Net 4では、それを受け入れるための新しいオーバーロードstring.JoinありますIEnumerable<string>。コードは次のようになります。

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

2
OK、それでこのソリューションはLinqを使用していませんが、私にはかなりうまく機能しているようです
Mat Roberts

33
ToArray is linq :)
エイミーB

18
これが最も正解です。これは、質問と受け入れられた回答の両方よりも速く、Aggregateを使用するたびに長い説明が必要なAggregateよりも明確です。
PRMan 2016年


125

Linqを使用する理由

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

これは完璧に機能し、IEnumerable<string>私の知る限りすべてを受け入れます。Aggregateここには何も必要ありません。


19
学習LINQはクールであってもよく、LINQは、目的を達成するためのキュートな手段かもしれないが、最終的な結果を得るため、実際にLINQを使用すると、あからさまな愚かな、控えめに言って、悪いことは、もしないだろう
ジェイソンホオジロ

9
.NET 4.0にはIEnumerable <string>およびIEnumrable <T>オーバーロードがあり、使いやすくなります
Cine

3
Cineが指摘するように、.NET 4.0には過負荷があります。以前のバージョンにはありません。String.Join(",", s.ToArray())ただし、古いバージョンでも引き続き使用できます。
Martijn、2011年


@ Shog9マージにより、ここでの回答は重複した作業のようになり、タイムスタンプはまったく役に立たなくなります。
nawfal 2014年

77

Aggregate拡張メソッドを見ましたか?

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

23
これは、おそらくString.Join()より遅く、コードで読み取るのが困難です。ただし、「LINQの方法」の質問には答えますか?:-)
Chris Wenham

5
ええ、私は自分の意見で答えを汚したくありませんでした。:P
ロバートS.

2
確かに、実際にはかなり遅いです。連結の代わりにStringBuilderでAggregateを使用しても、String.Joinよりも遅くなります。
Joel Mueller、

4
10.000.000回の反復でテストを行い、aggregateは4.3秒かかり、string.joinは2.3秒かかりました。したがって、perf diffは一般的なユースケースの99%にとって重要ではないと言えます。したがって、データを処理するためにすでに多くのlinqを実行している場合は、通常、その優れた構文を壊してstring.join imoを使用する必要はありません。gist.github.com/joeriks/5791981
joeriks


56

私のコードの実際の例:

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

クエリは、文字列であるNameプロパティを持つオブジェクトであり、選択したリストにあるすべてのクエリの名前をコンマで区切って表示します。


2
パフォーマンスに関するコメントを考慮して、この例はダイアログが閉じたときに1回実行されるコードからのものであり、リストには10​​個を超える文字列が含まれることはほとんどありません。
Daniel Earwicker、2008年

Linq to Entitiesでこれと同じタスクを実行する手がかりはありますか?
Binoj Antony

1
素晴らしい例。これを実際のシナリオに組み込んでいただきありがとうございます。まったく同じ状況で、連結が必要なオブジェクトのプロパティがありました。
Jessy Houle、2009年

1
myList <T>の文字列プロパティを選択する最初の部分を理解するのを手伝ってくれた
Nikki9696

1
より大きなアレイでのこのアプローチのパフォーマンスについて書いてください。
Giulio Caccin 2015年

31

他の回答と同様の質問で対処された問題(つまり、AggregateとConcatenateが0要素で失敗する)を確認した後、Join / Linqを組み合わせたアプローチを次に示します。

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

または(s文字列でない場合)

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

  • シンプルな
  • 読みやすく、理解しやすい
  • 一般的な要素で機能します
  • オブジェクトまたはオブジェクトプロパティの使用を許可します
  • 長さ0の要素のケースを処理します
  • 追加のLinqフィルタリングで使用できます
  • (少なくとも私の経験では)うまく機能します
  • StringBuilder実装するために追加のオブジェクト(例:)を(手動で)作成する必要はありません

そしてもちろん、Joinは他のアプローチ(forforeach)に潜入することがある厄介な最後のカンマを処理します。そのため、最初にLinqソリューションを探していました。


1
括弧の不一致。
ctrl-alt-delor 2013


3
このように使用.Select()すると、この操作中に各要素を簡単に変更できるため、この答えが好きです。たとえば、各アイテムを同じような文字でラッピングするstring Result = String.Join(",", split.Select(s => "'" + s + "'"));
Sam Storie

29

あなたは使用することができますStringBuilderAggregate

  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を実行できることを示すためだけにあります。)


2
いいね+1 ただし、IMOは後で削除するよりも、余分な「、」を追加しない方が良いです。何かnew[] {"one", "two", "three"}.Aggregate(new StringBuilder(), (sb, s) =>{if (sb.Length > 0) sb.Append(", ");sb.Append(s);return sb;}).ToString();
dss539

5
if (length > 0)linqのをチェックせずに取り出すことで、貴重なクロックサイクルを節約できます。
Binoj Antony 2010

1
私はdss539に同意します。私のバージョンはnew[] {"", "one", "two", "three"}.Aggregate(new StringBuilder(), (sb, s) => (String.IsNullOrEmpty(sb.ToString())) ? sb.Append(s) : sb.Append(", ").Append(s)).ToString();
ProfNimrod

22

StringBuilderと3000要素を超えるSelect&Aggregateケースのクイックパフォーマンスデータ:

単体テスト-期間(秒)
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);
    }

このための非LINQルートを
選択するのに役立ち

4
時間の違いは、おそらくStringBuilderと+を使用した文字列連結です。LINQやAggregateとは関係ありません。StringBuilderをLINQ Aggregate(SOでの多くの例)に入れると、それは同じくらい高速になるはずです。
コントロールボックス


12

超クールなLINQの方法」では、LINQが拡張メソッドを使用して関数型プログラミングをより美しくする方法について話しているかもしれません。つまり、入れ子になっているのではなく、視覚的に線形な方法で(次々に)関数を連鎖できる構文上の砂糖です。例えば:

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

このように書くことができます:

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

2番目の例の方が読みやすいことがわかります。また、インデントの問題や式の最後に表示される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);
   }
}

これにより、(少なくともアルゴリズムの複雑さに関して)直接呼び出しと同じくらいのパフォーマンスのコードが提供され、特にブロック内の他のコードが連鎖関数スタイルを使用している場合は、場合によってはコードが読みやすくなります(コンテキストによって異なります)。 。


1
このスレッドのタイプミスの数は奇抜です:seperator => separator、Concatinate => Concatenate
SilverSideDown



5

ここでは、単一の式として純粋な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));
}

そして、そのすごく速い!


3

私は少しごまかして、コメント内に貼り付けるのではなく、ここでのすべての最高のものをまとめているように見える新しい答えを捨てます。

だからあなたはこれを1行にすることができます:

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からの答えは少しパフォーマンスが高いかもしれませんが、Removeと比較してReplaceが内部で何を行うかわかりません。\ aで終わる文字列を連結したい場合、他の唯一の警告は、セパレーターを失うことになります...私はそうは思わないでしょう。その場合は、他にファンシーキャラクターを選択できます。


2

LINQとstring.join()非常に効果的に組み合わせることができます。ここでは、文字列からアイテムを削除しています。これを行うにはもっと良い方法がありますが、ここにあります:

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


1

ここにはたくさんの選択肢があります。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();

builder.Length > 0ForEachでチェックせず、ForEachの後の最初のコンマを削除する方が速いでしょう
Binoj Antony

1

linqを使用してIISログファイルを解析するときに、次のようにすばやくダーティにしました。200万行を試行するとメモリ不足エラーが発生しましたが、@ 100万行(15秒)でかなりうまくいきました。

    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()
    ;


0

私はこれについて少し前にブログに書いています、私が継ぎ目を行ったことはまさにあなたが探しているものと同じです:

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(", ");


ここにコードを連結して、答えを理解しやすくすることはできますか?
Giulio Caccin 2015年
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.