2010-12-01 83 views
2

我有以下代碼:爲什麼我的列表中的所有元素似乎都是一樣的?

Integer[] lastExchange = new Integer[nColors]; 
Integer[] newExchange = new Integer[nColors]; 
while (true) { 
    ... 
    for (int i=0; i<nColors; i++) { 
     lastExchange[i] = newExchange[i]; 
    } 
    ... 
    exchanges.add(lastExchange); 
    output.log.fine("Exchange:" + lastExchange[0] + "," + lastExchange[1]); 
} 
for (Integer[] exchange : exchanges) { 
    output.log.fine("Exchange:" + exchange[0] + "," + exchange[1]); 
} 

我有兩個輸出(一個在while循環另一個在for循環)。第一個輸出顯示我將不同的數組添加到列表中。雖然當我在第二個循環中進行雙重檢查時,我發現exchange列表中的所有元素都是相同的(它們與列表中的第一個元素相同)。

有人知道我在做什麼錯嗎?

+0

這些點是相當重要的。你如何創建/設置lastExchange? – Jens 2010-12-01 13:24:40

回答

5

正如unwind的答案所述,您在循環的每次迭代中都添加對同一個數組的引用。您需要在每次創建新陣列

// It's not clear where newExchange is actually populated 
Integer[] newExchange = new Integer[nColors]; 
while (true) { 
    Integer[] lastExchange = new Integer[nColors]; 
    ... 
    for (int i=0; i<nColors; i++) { 
     lastExchange[i] = newExchange[i]; 
    } 
    ... 
    exchanges.add(lastExchange); 
    output.log.fine("Exchange:" + lastExchange[0] + "," + lastExchange[1]); 
} 

或者,如果你只是克隆陣列:

Integer[] newExchange = new Integer[nColors]; 
while (true) { 
    Integer[] lastExchange = newExchange.clone(); 
    ... 
    exchanges.add(lastExchange); 
    output.log.fine("Exchange:" + lastExchange[0] + "," + lastExchange[1]); 
} 
2

lastExchange是什麼類型的?如果它是一個對象的引用,那麼問題可能就是這個;您只需將相同的引用添加到可變對象,然後再對其進行修改和添加。

由於第一個循環在對象(大概)被修改之前先打印對象,它會打印適當的(不同的)值。

相關問題