Discussion :

[Loceka]

Bonjour,

Je travaillais sur des Vecteurs contenant des tableaux et j'avais besoin de changer juste quelques valeurs d'un des tableaux, valeurs qui pouvaient s'averer "néfastes".
J'ai donc fait un clone du vecteur de base pour avoir une copie de sauvegarde au cas où il y'aurait des erreurs dans l'autre et je suis tombé sur une constatation étrange :

lorsqu'on clone un objet (un vecteur en tout cas), le clone se contente de faire une copie de "premier degré" des objets qu'il contient puis des références pour les sous-objets.

En effet, voilà ce qui m'est arrivé :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
Vector v = new Vector() // normal...
v.add(new String[]{"A","B","C"});
 
// Clonage de v
backUpV = (Vector)v.clone();
 
// Changement de "A" en "D" dans le vecteur v
( (String[])v.elementAt(0) )[0] = "D";
 
/* ou bien (c'est equivalent dans les faits) :
 * String[] contenuV = ((String[])v.elementAt(0));
 * contenuV[0] = "D";
 * v.setElementAt(contenuV,0);
 */
 
// Affichage du resultat
System.out.println(
     "premiere lettre de v = "+( (String[])v.get(0) )[0]+"\n"+
     "premiere lettre de backUpV = "+( (String[])backUpV.get(0) )[0]
);

Code :

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

1
2
3
4
5
6
7
Le résultat attendu est :
premiere lettre de v = D
premiere lettre de backUpV = A
 
Mais contre toute attente il s'affiche :
premiere lettre de v = D
premiere lettre de backUpV = D

Une alternative qui fonctionne est d'écrire :

Code :

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

1
2
3
4
5
6
7
String[] contenu = new String[v.size()];
// Transfert du contenu de v dans 'contenu'
for (int i=0; i<contenu.length; i++)
   contenu[i] = ((String[])v.elementAt(0))[i];
 
contenu[0] = "D";
v.setElementAt(contenu,0);

Est-ce un "bug" dans la méhode clone() ou bien est-ce désiré ?

Dans tout les cas c'est étrange (et ennuyeux aussi...)

Loceka.

[adiGuba]

Salut,


C'est normal puisque les Vecteurs ne contiennent qu'une référence vers l'objet qu'ils contiennent.

Donc les deux vecteurs contiennent le même objet (le tableau). Lorsque tu le modifies il est modifié dans les deux vecteurs...

Dans le second cas tu crée un nouveau tableau donc tu as deux objets et lorsque tu en modifies un tu ne modifies pas l'autre...

a++

[zekey]

Ben dans la doc de l'API c'est dit:

Code :

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

1
2
 
Returns a clone of this vector. The copy will contain a reference to a clone of the internal data array, not a reference to the original internal data array of this Vector object.

C'est pas marqué qu'il duplique les objets contenus.
Pourquoi irait-il cloné recursivement tous les objets et les sous objets et les sous-sous objets..... et mettre toutes les références a jour. Imagine le bordel

[Loceka]

Pourquoi irait-il cloné recursivement tous les objets et les sous objets et les sous-sous objets..... et mettre toutes les références a jour.

Pour faire un vrai clone, totalement indépendant de la source par exemple.

Je croyais que c'était le comportement par défaut de la méthode clone (un peu comme la serialization, qui sérialize tous les objets en appelant leur méthode de sérialization).

Mais si c'est pas un bug alors tant mieux.

[Pill_S]

Je croyais que c'était le comportement par défaut de la méthode clone

justement pas. d'ailleurs, dans la javadoc de "Object", on peut lire ceci:

Creates and returns a copy of this object. The precise meaning of "copy" may depend on the class of the object. The general intent is that, for any object x, the expression:

x.clone() != x

will be true, and that the expression:

x.clone().getClass() == x.getClass()

will be true, but these are not absolute requirements. While it is typically the case that:

x.clone().equals(x)

will be true, this is not an absolute requirement.

en gros, chaque classe peut implémenter la méthode clone de la manière qu'elle le souhaite... il faut gérer au cas par cas

[Gfx]

J'en parle un peu ici : http://www.progx.org/index.php?section=articles&article=Design%20Patterns/article4

Tu peux parfois contourner le probleme en faisant le clonage a l'aide de la serialisation.