Discussion :

[Pragmateek]

Salut!

Pour copier le contenu d'un "ArrayList<ArrayList<Double>>", "matrice":

Code :

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

1
2
3
4
5
6
ArrayList<ArrayList<Double>> temp=new ArrayList<ArrayList<Double>>();
for(int i=0;i<matrice.get(0).size();i++){
	temp=(ArrayList<ArrayList<Double>>)(matrice.clone());//
	for(int j=0;j<temp.size();j++){
		temp.set(j,(ArrayList<Double>)(matrice.get(j).clone()));//
	}

.
Mais les lignes:

Code :

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

temp=(ArrayList<ArrayList<Double>>)(matrice.clone());

et

Code :

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

temp.set(j,(ArrayList<Double>)(matrice.get(j).clone()));

provoquent le warning:

Code :

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

Type safety: The cast from Object to ArrayList<ArrayList<Double>> is actually checking against  the erased type ArrayList

Pourtant c'est bien ainsi que doit être casté le retour de "clone", non?

Merci de votre aide.

[Aldebaranl2]

Je pense que nous montrer comment est construite ta matrice aiderait bien (quel type, quelle archi, etc..).

Deuxiemement, il est préférable d'utiliser une boucle "forEach" plutot qu'une boucle "for" dans ce cas :

for (ArrayList<ArrayList<Double>> al : matrice.get(0) {
............
}

et pareil pour la seconde

[osopardo]

Le transtypage et les types paramétrés ne font pas bon ménage pour le compileur, le seul moyen de se débarrasser de cet avertissement semble être la surcharge des méthodes clone pour qu'elles te retournent les types appropriés

[Pragmateek]

Mais sinon il n'y a pas de risque de mauvais transtypage?
Le résultat est bien celui attendu?

[had35]

Je pense que le warning vient du fait que le compilateur ne vérifie pas le type du template s'il est surcasté en Object. Le seul risque, mis à part l'inadvertance, est que si tu distribues ta librairie, un utilisateur ne comprenne pas la logique et outrepasse la méthode clone() pour retourner un objet qui ne serait pas du type requis. Le compilateur n'y verrait rien si cet objet est surcasté en Object.

Code :

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

1
2
3
4
 
Object clone() {
   return new ArrayList<Float>();
}

La méthode d'osopardo est la plus fiable

[adiGuba]

Salut,


Il serait peut-être plus simple de créer une nouvelle ArrayList tout simplement :

Code :

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

temp = new ArrayList<ArrayList<Double>>(matrice);

a++