Почему все элементы моего списка кажутся одинаковыми? - PullRequest
Новая
       20

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

2 голосов
/ 01 декабря 2010

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

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 одинаковы (они равны первому элементу списка).

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

Ответы [ 2 ]

5 голосов
/ 01 декабря 2010

Как указано в ответе 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 голосов
/ 01 декабря 2010

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

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

...