Почему все элементы моего списка, похоже, совпадают?

Question

У меня есть следующий код:

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]); 
} 

У меня есть два выхода (один в цикле, а другой один в течение цикла). Первый вывод показывает мне, что я добавляю в список разные массивы. Хотя, когда я делаю двойную проверку во втором цикле, я вижу, что все элементы списка exchange одинаковы (они равны первому элементу списка).

Кто-нибудь знает, что я делаю неправильно здесь?

Answer 1

Как говорится в ответе от разговора, вы добавляете ссылку на один и тот же массив на каждой итерации цикла. Вам нужно создать новый массиву каждый раз:

// 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]); 
} 

Answer 2

Что такое lastExchange? Если это ссылка на объект, проблема, вероятно, именно в этом; вы просто добавляете ту же ссылку на изменяемый объект, который затем изменяется и добавляется снова.

Поскольку первый цикл печатает объект до его (предположительно), он печатает правильные (разные) значения.