Sobre el warning, la única manera de elimintarlo es usando la anotación @SupressWarnings
. Esto sucede porque estás haciendo un casteo a un tipo genérico. En Java, no existe otra manera de hacer casteo hacia abajo a un objeto definido con genéricos (en este caso, de Object
a ArrayList<Integer>
) y eliminar la advertencia. No tiene nada de malo en usar @SupressWarnings
, siempre y cuando sepas lo que estás haciendo. En caso que removieses el genérico, te saldría otro warning de uso de variables crudas (raw type), y de todas maneras tendrías que agregar esta anotación.
El método clone
es un pobre intento de Java para implementar el patrón prototype. Digo que es pobre porque técnicamente no ofrece mucho valor y solo es una copia superficial. Solo hay pocos casos donde se debería llamar al método clone, como en un arreglo.
Para tu caso, si solo quieres una copia superficial con una nueva lista con los mismos objetos, es decir, que si modificas el estado de uno los objetos de la nueva lista también se verá el cambio de estado en la lista original (porque son el mismo objeto), entonces basta con lo siguiente:
List<Integer> copia = new ArrayList<>(listaOrigen);
Si lo que buscas es una copia profunda, es decir, que incluso los objetos en la nueva lista sean diferentes a los de la original y cada uno mantenga su propio estado, deberás asegurarte que cada elemento dentro de tu lista original sea inmutable o que implementen su propio método de clonado profundo como constructor de copia, serialización/deserialización u otro.