重複するローカル変数の問題を洗練されたソリューションですか？

Question

Integer[] lastExchange = new Integer[nColors]; 
Integer[] exchangeToAdd = new Integer[nColors]; 
lastExchange = getValue(); 
exchangeToAdd = getValue(); 
exchanges.add(exchangeToAdd); 

Integer[] newExchange = new Integer[nColors]; 
while (true) { 
    newExchange = getValue(lastExchange); 
    Integer[] exchangeToAddForLoop = new Integer[nColors]; 
    for (int i=0; i<nColors; i++) { 
     lastExchange[i] = newExchange[i]; 
     exchangeToAddForLoop[i] = newExchange[i]; 
    } 
    exchanges.add(exchangeToAddForLoop); 
} 

は、私はこのコードをどうしようとしています何

を追加しましたか？私はexchangesというリストを埋め込む必要があります。リストの最初の要素はlastExchangeです。コードの私の問題は、常に変数の2つの複製を作成する必要があることです（コードがエレガントではないと思う理由ですが、より良い解決策が見つけられません）。たとえば、最初はlastExchangeを作成し、次にと同じ値を持つexchangeToAddを作成します。ループでも同じことが起こります。私はlastExchangeを作成し、exchangeToAddForLoopを作成します。後者が変更されるため、lastExchangeをリストに追加できないため、これを行います。

はここ2

を追加しました私の問題です。私はそのようなコードを持っている：

Integer[] e = getValue(); 
Integer[] e1 = getValue(); // <-- I do not like that. 
exchanges.add(e1);   // <-- I do not like that. 
while (true) { 
    Integer[] e_new = getValue(e); 
    Integer[] e2 = new Integer[nColors]; // <-- I do not like that. 
    for (int i=0; i<nColors; i++) { 
     e[i] = e_new[i]; 
     e2[i] = e_new[i]; // <-- I do not like that. 
    } 
    exchanges.add(e2); // <-- I do not like that. 
} 

を、私はeの計算に加えてe1とe2を計算する必要があります。

Answer 1

これは、少なくとも2つの方法で無粋コードです：あなたのローカル変数のほとんどは、その後すぐに上書きされた値

が割り当てられている

• あなたnewExchange変数は、より深くネストされた宣言することができました。

だからどんな振る舞いを変更せずに、ここではよりよいバージョンです：

Integer[] lastExchange = getValue(); 
Integer[] exchangeToAdd = getValue(); 
exchanges.add(exchangeToAdd); 

while (true) { 
    Integer[] newExchange = getValue(lastExchange); 
    Integer[] exchangeToAddForLoop = new Integer[nColors]; 
    for (int i=0; i<nColors; i++) { 
     lastExchange[i] = newExchange[i]; 
     exchangeToAddForLoop[i] = newExchange[i]; 
    } 
    exchanges.add(exchangeToAddForLoop); 
} 

次は、私たちは、あなたがこのコードのいずれかがやっていることを意図しているものを私たちに語っていない問題に来て、またどのようなあなたは "重複ローカル変数問題"を意味します。ああ、コメントで指摘したように、あなたのループは決して終了しません。

Answer 2

あなたのコードについて議論することなく、重複する可変エラーがある場合はいつでも{}を使用できます。

これはコンパイルされません

  int a=0; 
      a++; 


      int a=0; 
      a++; 

この処理が行われますJonの簡素化@

 { 
      int a=0; 
      a++; 
     } 
     { 
      int a=0; 
      a++; 
     } 

Answer 3

は、最も安全ですが、私はそれをさらに簡素化することができると思います。

exchanges.add(getValue()); 

while (true) { // forever?? 
    // do you need null values or can you use int[] 
    int[] newExchange = getValue(exchanges.get(exchanges.size()-1); 
    // do you need to add a copy, if not then clone() can be dropped. 
    exchanges.add(newExchange.clone()); 
}