ArrayList не клонируется

Здравствуйте,
Пытаюсь клонировать ArrayList со всеми переменными. В итоге изменяю значения градусов в новом листе. И автоматически изменяются данные в старом листе. Хотя использую пример, в котором данные старого листа должны остаться неизменными.
ArrayList celsius = new ArrayList(); ArrayList fareng = (ArrayList) celsius.clone();
for(HeatSensor h: fareng) { double far = h.getCelsius(); far = 9*far/5 + 32;
h.setCelsius(far);
System.out.println(h.toString()); }
Чтобы было понятнее, вот что выводится на экран:
================celsius============================ Heat : 70.0, 1/02/2017 Heat : 30.0, 1/02/2017 Heat : 35.0, 3/02/2017
================fahrenheit========================== Heat : 158.0, 1/02/2017 Heat : 86.0, 1/02/2017 Heat : 95.0, 3/02/2017
================celsius============================ Heat : 158.0, 1/02/2017 Heat : 86.0, 1/02/2017 Heat : 95.0, 3/02/2017
Так же имеется другой вопрос относительно этой же программы. найти максимальное значение в ArrayList

Ответ

Метод clone определен в ArrayList как копирование ссылок элементов.
public Object clone() { try { ArrayList v = (ArrayList) super.clone(); v.elementData = Arrays.copyOf(elementData, size); v.modCount = 0; return v; } catch (CloneNotSupportedException e) { // this shouldn't happen, since we are Cloneable throw new InternalError(e); } }
Таким образом, элементы равны с точностью до ссылки celsius.get(i) == fareng.get(i) для любого i, и при изменении элемента в одном списке вы измените его же в другом списке.
Для решения вашего вопроса нужно воспользоваться глубоким копированием, то есть сделать дополнительно и копии самих элементов.
for (HeatSensor h : celsius) { fareng.add(h.clone()); }
P.S. в разработке не рекомендуется использовать метод clone. Он считается плохим дизайном в приложении. Вам стоит воспользоваться копированием через конструктор.
List fareng = new ArrayList<>(celsius.size()); for (HeatSensor h : celsius) { fareng.add(new HeatSensor(h)); }
И сам конструктор:
public HeatSensor(HeatSensor another) { this.celsius = another.celsius; // и так далее по аналогии }