374

古い学校を書くための最も効率的な方法は何ですか:

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

4

17 に答える 17

557

この回答は、質問で要求された LINQ ( Aggregate) の使用法を示しており、日常的な使用を意図したものではありません。これは 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();
于 2008-10-20T08:53:13.910 に答える
410 
return string.Join(", ", strings.ToArray());

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

return string.Join(", ", strings);
于 2008-10-20T13:28:34.447 に答える
132

なぜLinqを使用するのですか？

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

それは完璧に機能しIEnumerable<string>、私が覚えている限りは何でも受け入れます。Aggregateここでは、はるかに遅いものは必要ありません。

于 2008-09-23T18:18:34.477 に答える
79

Aggregate拡張メソッドを見たことがありますか？

var sa = (new[] { "yabba", "dabba", "doo" }).Aggregate((a,b) => a + "," + b);
于 2008-09-23T18:09:49.740 に答える
58

私のコードの実際の例:

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

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

于 2008-10-20T08:52:59.587 に答える
32

これは、他の回答と同様の質問で対処された問題(つまり、集約と連結が 0 要素で失敗すること) を見て解決した、結合/Linq アプローチを組み合わせたものです。

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

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

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

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

forそしてもちろん、Join は、他のアプローチ ( , ) に忍び込む厄介な最後のコンマを処理しますforeach。これが、そもそも Linq ソリューションを探していた理由です。

于 2012-10-04T19:02:53.463 に答える
29

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 を実行できることを示すためのものです。)

于 2010-05-19T20:42:05.397 に答える
22

3000 要素を超える StringBuilder と 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);
    }
于 2010-05-10T22:46:43.030 に答える
19

私は常に拡張メソッドを使用します：

public static string JoinAsString<T>(this IEnumerable<T> input, string seperator)
{
    var ar = input.Select(i => i.ToString());
    return string.Join(seperator, ar);
}
于 2009-05-08T17:02:34.813 に答える
13

「超クールな LINQ の方法」とは、LINQ が拡張メソッドを使用して関数型プログラミングをより快適にする方法について話している可能性があります。つまり、関数を入れ子にする (一方を他方の内側に) のではなく、視覚的に直線的な方法で (次々に) チェーンできるようにする構文糖衣です。例えば：

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

次のように記述できます。

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

2 番目の例の方が読みやすいことがわかります。また、インデントの問題や、式の最後に表示されるLispの閉じ括弧を減らして、より多くの関数を追加できることも確認できます。

他の多くの回答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);
   }
}

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

于 2012-05-16T12:37:37.140 に答える
5

この前の質問にはさまざまな代替回答があります-ソースとして整数配列をターゲットにしていたことは確かですが、一般化された回答を受け取りました。

于 2008-10-20T09:14:32.680 に答える
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));
}

しかもめちゃくちゃ速い！

于 2012-09-03T04:34:09.773 に答える
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);、式の最後に an を追加する必要があります。少し賢くなりすぎたのかもしれません。

@a.friend からの回答は、わずかにパフォーマンスが高い可能性があります。Replace が Remove と比較して内部で何を行うのかはわかりません。何らかの理由で \a で終わる文字列を連結したい場合、他の唯一の注意点は、セパレーターを失うことになります...それはありそうもないと思います。その場合は、他の派手なキャラクターから選択できます。

于 2011-09-01T17:42:52.357 に答える
2

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

filterset = String.Join(",",
                        filterset.Split(',')
                                 .Where(f => mycomplicatedMatch(f,paramToMatch))
                       );
于 2011-02-02T09:00:14.363 に答える
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()
    ;
于 2011-12-14T18:41:55.440 に答える
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();
于 2010-04-21T20:02:07.253 に答える