質問

昔ながらの文を書く最も効率的な方法は何ですか:

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

この出力:

one, two, three

集計は、値のコレクションを受け取り、スカラー値を返す関数です。 T-SQLの例には、min、max、sumが含まれます。 VBとC#の両方が集約をサポートしています。 VBとC#は両方とも、拡張メソッドとして集約をサポートします。ドット表記を使用して、 IEnumerable オブジェクト。

集計クエリはすぐに実行されることに注意してください。

詳細- MSDN:Aggregateクエリ


本当に<=>を使用したい場合は、 CodeMonkeyKing がコメントで提案した<=>を使用してバリアントを使用します。多数のオブジェクトの良好なパフォーマンスを含む通常の<=>とほぼ同じコード:

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

他のヒント

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

.Net 4には、 IEnumerable&lt; string&gt; を受け入れるstring.Join 。コードは次のようになります。

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

Linqを使用する理由

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

これは完全に機能し、覚えている限りでは IEnumerable&lt; string&gt; を受け入れます。ここでは Aggregate は必要ありませんが、これは非常に遅いです。

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

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

私のコードからの実際の例:

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

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

Aggregate StringBuilder を使用できます:

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

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

string Result = String.Join(&quot;、&quot ;, split.Select(s =&gt; s.Name));

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

string Result = String.Join(&quot;、&quot ;, split.Select(s =&gt; s.ToString()));

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

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

StringBuilderとSelect&amp;のクイックパフォーマンスデータ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()).ToArray();
    return string.Join(seperator, ar);
}

super-cool LINQ way 」とは、拡張メソッドを使用することで、LINQが関数型プログラミングをより魅力的にする方法について話しているかもしれません。つまり、関数をネスト(1つの内部に)するのではなく、視覚的に線形に(次々に)チェーンできるようにする構文糖衣です。例:

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

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

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

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

他の多くの回答では、 String.Join を読むのが最も速いか、最も簡単であるため、これが道であると述べています。しかし、「 super-cool LINQ way 」の解釈を採用する場合、答えは String.Join を使用することですが、それを可能にするLINQスタイル拡張メソッドでラップすることです視覚的に心地よい方法で機能を連鎖させる。したがって、 sa.Concatenate(&quot ;,&quot;)を作成する場合は、次のようなものを作成するだけです。

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

そして非常に高速です!

少しごまかして、コメント内に貼り付けるのではなく、ここにあるものすべての最良の部分を要約していると思われる新しい回答をスローします。

したがって、次の 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 で終わる文字列を連結したい場合、区切り文字が失われることになります...その可能性は低いと思います。そうであれば、あなたは持っています その他の派手なキャラクター から選択します。

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

linqを使用してIISログファイルを解析するとき、次のようにすばやく実行しましたが、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
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()
    ;
quot;) && !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()のためでした:

<*>

これについては少し前にブログで説明しましたが、私がしたことはまさにあなたが探しているものになります:

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(", ");
ライセンス: CC-BY-SA帰属
所属していません StackOverflow
scroll top