3

Здравствуйте,

Пытаюсь клонировать ArrayList со всеми переменными. В итоге изменяю значения градусов в новом листе. И автоматически изменяются данные в старом листе. Хотя использую пример, в котором данные старого листа должны остаться неизменными.

ArrayList<HeatSensor> celsius = new ArrayList<HeatSensor>();
ArrayList<HeatSensor> fareng = (ArrayList<HeatSensor>) 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

1
  • хорошо. так и сделаю
    – Alex
    10 фев 2017 в 18:07

2 ответа 2

4

Метод 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<HeatSensor> fareng = new ArrayList<>(celsius.size());
for (HeatSensor h : celsius) {
     fareng.add(new HeatSensor(h));
} 

И сам конструктор:

public HeatSensor(HeatSensor another) {
     this.celsius = another.celsius; // и так далее по аналогии
}
1
  • Спасибо за подробное объяснение!
    – Alex
    10 фев 2017 в 20:02
2

Реализуйте метод clone() в классе HeatSensor и клонируйте списки следующим образом:

ArrayList<HeatSensor> celsius = new ArrayList<>();
ArrayList<HeatSensor> fareng = new ArrayList<>();
for (HeatSensor h : celsius)
        fareng.add(h.clone());

Ваш вариант не работает, т.к. клонируются списки, но объекты остаются одни и те же.

1
  • Спасибо большое!
    – Alex
    10 фев 2017 в 20:02

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.