"このホットコードパスでStringBuilderまたはforeachを使用しないでください"

Question

オープンソースSignalRプロジェクトのソースコードを参照しています。this diff codeはと表示されています。このホットコードではStringBuilderまたはforeachを使用しないでくださいパス」：

-   public static string MakeCursor(IEnumerable<Cursor> cursors) 
+   public static string MakeCursor(IList<Cursor> cursors) 
      { 
-    var sb = new StringBuilder(); 
-    bool first = true; 
-    foreach (var c in cursors) 
+    var result = ""; 
+    for (int i = 0; i < cursors.Count; i++) 
       { 
-     if (!first) 
+     if (i > 0) 
        { 
-      sb.Append('|'); 
+      result += '|'; 
        } 
-     sb.Append(Escape(c.Key)); 
-     sb.Append(','); 
-     sb.Append(c.Id); 
-     first = false; 
+     result += Escape(cursors[i].Key); 
+     result += ','; 
+     result += cursors[i].Id; 
       } 
-    return sb.ToString(); 
+    return result; 
      } 

私はforeachのは時々あまり効率的ではなかった理由を理解し、なぜそれがために置き換えられます。

しかし、私はStringBuilderが文字列を連結する最も効率的な方法であることを知り、経験しました。だから、私はなぜそれを標準的な連結に置き換えることにしたのだろうと思っています。

ここで、そして一般的にStringBuilderの使用については何が間違っていますか？

Answer 1

私はコードの変更を行いました。はい、割り当ての数に大きな違いがありました（GetEnumerator（））がvsではないことを示しています。このコードが毎秒何百万回もあると想像してください。割り当てられた列挙子の数はばかげているので避けることができます。

編集： 現在任意の割り当て（直接ライターへの書き込み）を避けるために、コントロールを反転：あなたは `foreach`がfor``未満効率的であることを思わせる https://github.com/SignalR/SignalR/blob/2.0.2/src/Microsoft.AspNet.SignalR.Core/Messaging/Cursor.cs#L36

Answer 2

あなたは何回連結していますか？数が多い場合は、StringBuilderを使用します。ほんの少数の場合は、StringBuilderを作成するオーバーヘッドがすべての利点を上回ります。

Answer 3

これを変更した人が実際に違いを測定してくれることを願っています。

• 毎回新しいストリングビルダをインスタンス化する際にオーバーヘッドがあります。 これはメモリ/ガーベジコレクションにも圧力をかけます。
• コンパイラが簡単なの連結
• それはコンパイラが、彼らは境界内にある「知っている」としてforeach文で行われていない れる境界のチェックが必要な場合がありますので、FORが実際に遅くなるかもしれないための「stringbuilderlike」コードを生成することができます。

Answer 4

は私が

  StringBuilder sb = new StringBuilder(); 
      bool first = true; 
      foreach (Cursor c in cursors) 
      { 
       if (first) 
       { 
        first = false; // only assign it once 
       } 
       else 
       { 
        sb.Append('|'); 
       } 
       sb.Append(Escape(c.Key) + ',' + c.Id); 
      } 
      return sb.ToString(); 

に私のお金を置くしかし、私は私のお金とdfowlerからアップデートを置きます。彼の答えの中のリンクを見てください。

Answer 5

これは、機能に提供されたCursorの数によって異なります。

4つの文字列をconcatinatingするとき、2つのapporaches間のほとんどの比較は、文字列連結よりもStringBuilderを好むようです。明白な理由がない場合は、私の問題/アプリケーションの2つのアプローチの性能比較など、あまり良い結果が得られなかった場合は、StringBuilderをお勧めします。私は、（多くの）再割り当てを避けるために、StringBuilderにバッファをあらかじめ割り当てることを検討します。

件名の説明については、String concatenation vs String Builder. PerformanceおよびConcatenating with StringBuilders vs. Stringsを参照してください。