このC＃ループは、文字列のリストが奇妙なのはなぜですか？

Question

私は自分自身を世界で最も優れた開発者とは考えていませんが、少なくとも文字列のリストをループすることができると思いました！からの余分な反復がある

First<br /> 
First<br /> 
Second<br /> 
First<br /> 
First<br /> 
Second<br /> 
Third<br /> 

：

public string liststrings() { 

     List<string> strings = new List<string>(); 
     strings.Add("First"); 
     strings.Add("Second"); 
     strings.Add("Third"); 

     string output = string.Empty; 

     for (int i = 0; i < strings.Count(); i++) 
     { 
      output += output + strings[i] + "<br />"; 
     } 

     return output; 
    } 

この機能は、次のHTMLを返します。

は、ここに私の機能ですか？

はFYI：私は、主にVBスクリプトの背景から来て、私は問題なくVBスクリプト内の配列でこれを行うことができます。リストやC＃の構文が違うと、これは違うのですか？

ありがとうございました。

Answer 1

出力を2回追加しています。

output += output + strings[i] + "<br />"; 

と同等です：

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

output = output + output + strings[i] + "<br />"; 

：

output += strings[i] + "<br />"; 

より良いオプションは、StringBuilderかもしれません。

Answer 2

出力の前の値を自身に2回追加するという事実から、余分な繰り返しが生じています。

output += output + strings[i] + "<br />"; 

私はあなたが私がStringBuilderクラスになり何をしているかから

output = output + strings[i] + "<br />"; 

または

output += strings[i] + "<br />"; 

しかし正直にしたいと思います。

Answer 3

あなたは代わりに以下を試し、各反復に戻って自分自身に文字列全体を追加する：

output += strings[i] + "<br />"; 

Answer 4

あなたはstrings.Count後に（）を削除する必要があります。また、あなたは他の記事で述べたように、また

foreach(string str in strings) 
{ 
    // etc. 
} 

を使用することができ、あなたが二回出力を追加していること：

output += output + "something" 

output = output + output + "something" 

Answer 5

と同等であることが使用することをお勧めします文字列を連結する代わりにStringBuilderオブジェクトを使用します。

... 
StringBuilder sb = new StringBuilder(); 
for(int i = 0; i < strings.Length; i++) 
{ 
    sb.WriteLine("{0}<br />", strings[i]); 
} 
return sb.ToString(); 

それが高速であるため、純粋な配列のため.Lengthプロパティを使用するには注意してください。