Java небезопасные или непроверенные выражений: Клонирование ArrayList

Question

я получил непроверенную ошибку выражения при компиляции и найти строку, вызвавшую быть

ArrayList<Integer> items = (ArrayList<Integer>) this.items.clone(); 

Я пытаюсь выполнить глубокую копию моего объекта, поэтому я Клонирование свойства объекта и списка массивов указанным выше способом. Как я могу исправить это предупреждение?

• я мог бы использовать @SuppressWarnings("unchecked"), но это просто скрывает проблему (Тхо я не ожидал никто)
• Если я клонировать вручную цикл через все элементы к нему будет медленнее, я думаю

Будет правильный способ сделать это?

Answer 1

Если ваши элементы являются целыми, выполнение «глубокой копии» действительно не является проблемой, так как нет причин, по которым вам нужно будет скопировать объект Integer. Просто используйте new ArrayList<Integer>(this.items).

Но для справки ни клоун(), ни конструктор копии ArrayList не сделают глубокую копию. Это связано только с тем, что ваши типы элементов не нуждаются в глубоком копировании, что удовлетворяет ваши потребности.

Answer 2

Аналогичное поведение можно получить с помощью new ArrayList<Integer>(this.items). В любом случае, это мелкой копия.

API

Answer 3

Благодаря обратной совместимости требуется, когда они введены дженериков в API Java, нет никакого способа вокруг использования литья и @SuppressWarnings("unchecked") в некоторых ситуациях.

Кроме того, см. here по причине, почему использование clone() следует использовать с осторожностью: оно делает мелкую копию, что отлично подходит для первобытных людей, но опасно для объектов.

Answer 4

Вы сказали, что пытаетесь сделать глубокую копию, но, как обсуждалось, here, я сомневаюсь, что вы сможете это сделать, используя clone(). Так, как и другие плакаты, это несколько более опасный способ использовать clone(), и вы не сможете получить глубокую копию, которую вы искали.

Answer 5

Целые числа неизменны, поэтому не имеет значения, сделаете ли вы глубокую копию или нет.

использовать класс Коллекции утилиты в java.util:

import java.util.Collections; 
... 
ArrayList<Integer> items = new ArrayList<Integer>(this.items.size()); 
Collections.copy(items, this.items); 

Answer 6

Как уже отмечалось, клонирование ArrayList не клонировать его элементы. Если вы хотите сделать глубокую копию содержимого, есть аккуратный трюк: сериализовать и десериализовать массив. (Это работает, потому что ArrayList и Integer оба реализуют Serializable.) Однако это не избавляет от необходимости подавлять непроверенные предупреждения о конверсиях.

// Write the object out to a byte array 
ByteArrayOutputStream bos = new ByteArrayOutputStream(); 
ObjectOutputStream out = new ObjectOutputStream(bos); 
out.writeObject(this.items); 
byte[] bytes = bos.toByteArray(); 

// Retrieve an input stream from the byte array and read 
// a copy of the object back in. 
ObjectInputStream in = new ObjectInputStream(
    new ByteArrayInputStream(bytes)); 
ArrayList<Integer> items = (ArrayList<Integer>) in.readObject(); 

Если весь ваш объект может быть объявлен Сериализуемым, вы могли бы использовать это вместо операции клонирования, чтобы сделать вашу глубокую копию. Кроме того, см. this article, чтобы избежать затрат на копирование байтов из ByteArrayOutputStream.