ループ内の変数を再初期化する正しい場所はどこですか？

Question

どちらが正しいアプローチであり、なぜですか？ループが完了した後

string initializeme = string.Empty; 
StringBuilder AppendToMe = new StringBuilder(); 

for(int i=0; i < 10; i++) 
{ 
    initializeme = string.Empty; //Is this the right place to initialize? 

    if(expressionThatEvalsTrue) 
     initializeme = SomeMethodReturningString(); 

    if(!string.IsNullOrEmpty(initializeme) 
     AppendToMe.Append(initializeme); 
} 

または

string initializeme = string.Empty; 
StringBuilder AppendToMe = new StringBuilder(); 

for(int i=0; i < 10; i++) 
{ 
    if(expressionThatEvalsTrue) 
     initializeme = SomeMethodReturningString(); 

    if(!string.IsNullOrEmpty(initializeme) 
     AppendToMe.Append(initializeme); 

    initializeme = string.Empty; //Is this the right place to initialize? 

} 

Answer 1

あなたはこの変数を使用するのでしょうか？私が見ることができる唯一の違いは、その変数の最後の値を保持する必要がある場合、最初の変数は2番目の変数を保持していないことです。

Answer 2

3つの要因がここにあります

• あなたはラムダ式や匿名メソッドを使用してループ内の変数をキャプチャするつもりですか？その場合は、の単一変数をキャプチャするか、繰り返しごとに1つずつキャプチャしますか？
• ループの後に値が必要ですか？
• ループの次の反復で現在の値が必要ですか？

私は一般的に可能な最小のスコープ内の変数を宣言すると、すぐにそれらを初期化しよう：

for(int i=0; i < 10; i++) 
{ 
    string initializeme = expression ? SomeMethodReturningString() : "";  
    Console.WriteLine(initializeme);  
} 

Answer 3

行うための最善の方法、それはこのようなものです：

for (int i = 0; i < 10; i++) 
{ 
    string initializeme = string.Empty; 
    if (expressionThatEvalsFalse) 
     initializeme = SomeMethodReturningString(); 

    Console.WriteLine(initializeme); 
} 

あなたが使用している場合initializemeがループ外にある場合は、ループ外で宣言する必要があります。

Answer 4

私はこれを好む：

for(int i=0; i<10; i++) 
{ 
    var initializeme = expression 
         ? SomeMethodReturningString() 
         : string.Empty; 
    Console.WriteLine(initializeme); 
} 

を、私はこのスタイルを好む理由は、それが自己完結していることです：変数が一つだけの場所に設定されています。ループ内で実行すると、変数のスコープも小さくなります。これもまた好ましいものです。三項演算子をパラメータIMOとして使用すると、関数呼び出しの読みやすさが低下するため、この場合は余分な変数を使用するのが好きです。

Answer 5

私は、ループ内で変数を宣言すると、コンパイラは、このから遅いコードを作成しないように十分にスマートであることを前提としています：

for(int i=0; i<10; i++) 
{ 
    string initializeme = string.Empty; 
    if(expressionThatEvalsFalse) 
     initializeme = SomeMethodReturningString(); 

    Console.WriteLine(initializeme); 
} 

Answer 6

現代のコンパイラは、すべての変数がある中で、静的単一代入にコードを変換します正確に1回割り当てられているので、ループの各繰り返しのように、都合の良いときに新しい変数を宣言しないという言い訳はありません。

Answer 7

コードスニペットでは、私は最初に初期化する必要はありません。 2つのスニペットは、initializemeが値（スニペット1）または空の文字列（スニペット2）に初期化されるループの後で異なります。以下のコードのプレミスは、ループ内のinitializerを必要とすることです。

StringBuilder AppendToMe = new StringBuilder(); 
for(int i=0; i < 10; i++) 
{  
    if(expressionThatEvalsTrue) 
     AppendToMe.Append(SomeMethodReturningString());   
} 

Answer 8

私の意見でもどちらの場合も、私は次のことを行います。

それが空白に設定されますothewiseあなたexpressionThatEvalsTrueは、あなたがメソッドから文字列を取得真である場合

initializeme = expressionThatEvalsTrue ? 
    SomeMethodReturningString() : string.Empty 

この方法では

initializeme = string.Empty; //Is this the right place to initialize? 

if(expressionThatEvalsTrue) 
    initializeme = SomeMethodReturningString(); 

を交換してください。