Discussion :

[blaise_laporte]

Bonjour les gens

J'ai un probleme pour cloner une ArrayList.

La ligne de code suivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
((Courbe)o).listePoint = (ArrayList<PointMesure>)(this.listePoint.clone());

Me provoque un warning à la compilation :

courbe\Courbe.java:124: warning: [unchecked] unchecked cast
found : java.lang.Object
required: java.util.ArrayList<courbe.PointMesure>
((Courbe)o).listePoint = (ArrayList<PointMesure>)ob;

ce que je comprend pas étant donné que je cast ma liste cloner...

Pourriez vous eclairer ma lanterne?

Merci
Blaise

[adiGuba]

Salut,


Le warning est normal puique ta méthode clone() renvoit un objet : rien ne garantit que l'objet retrouné est bien une ArrayList<PointMesure> (en particulier si tu hérite de ArrayList et que tu redéfinis cette méthode).

Or l'utilisation des Generics te garantit l'absence totale de ClassCastException si la compilation s'effectue sans warning (et sans erreur bien sûr).

Pour éviter ce warning tu peux utiliser l'annotation @SuppressWarning (attention il faut le JDK _05 ou _06 il me semble) pour ignorer ce warning, par exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
    @SuppressWarnings("unchecked")
    public static <T> ArrayList<T> clone(ArrayList<T> list) {
        return (ArrayList<T>) list.clone();
    }

Puis :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

((Courbe)o).listePoint = clone(this.listePoint);

Sinon tu peux tout simplement recréer une nouvelle ArrayList sans passer par le cloneage :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

((Courbe)o).listePoint = new ArrayList<PointMesure>(this.listePoint);

a++

[blaise_laporte]

L'enuit du code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

((Courbe)o).listePoint = new ArrayList<PointMesure>(this.listePoint);

c'est que je me retrouve avec les references des objets contenuent dans ma listePoint, et que si je modifie la liste cloner, je modifie l'original également (me semble t'il)

Le warning est normal puique ta méthode clone() renvoit un objet : rien ne garantit que l'objet retrouné est bien une ArrayList<PointMesure>.

Justement, je recherche un moyen de caster l'objet retourné par clone() pour lui signifie que c'est une arraylist et qu'il arrete de m'enquiquiner avec ce warning (non mais!)

Mais bon, je sens que je vais opter pour la méthode du @SuppressWarning.

Merci

[adiGuba]

c'est que je me retrouve avec les references des objets contenuent dans ma listePoint, et que si je modifie la liste cloner, je modifie l'original également (me semble t'il)

Quelque soit la méthode (clone() ou création d'une nouvelle instance) :

• Si tu ajoutes/supprimes des éléments de la liste, la liste original ne sera pas modifié (et vice/versa).
• Si tu modifies des attributs des objets de la liste, les objets de la listes original seront aussi modifié (et vice/versa) puisque la méthode clone() ne copie pas les éléments de la liste mais seulement leurs références...

a++

[blaise_laporte]

Quelque soit la méthode (clone() ou création d'une nouvelle instance) :

• Si tu ajoutes/supprimes des éléments de la liste, la liste original ne sera pas modifié (et vice/versa).
• Si tu modifies des attributs des objets de la liste, les objets de la listes original seront aussi modifié (et vice/versa) puisque la méthode clone() ne copie pas les éléments de la liste mais seulement leurs références...

a++

en effet, j'ai pensé à remplacer tout les elements de ma nouvelle liste par des clones des elements de mon ancienne liste (Je crois qu'on appelle ça un clonage en profondeur?). Du coup, je peux parfaitement faire

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

((Courbe)o).listePoint = new ArrayList<PointMesure>(this.listePoint);

comme tu me l'a conseiller...

J'suis bête

Merci