Discussion :

[onlytoine]

Hello,

Comment implémentez-vous le clonage profond de vos objets ? Pourquoi ?

Globalement j'ai deux choix : surcharger la méthode clone() de mes objets ou bien écrire une méthode de copie générale externe s'occupant de tous les objets...

Merci !

Toine

[dinobogan]

La méthode de copie générale doit connaitre tous les objets, et faire de l'introspection pour les données internes privées sans accesseurs, et c'est lent. En plus, lors de la création d'un nouvel objet, il faut aussi modifier le super objet de copie générale.
C'est pour ces raisons que l'objet lui-même doit se débrouiller pour se cloner. Il sait comment il fonctionne, il a accès à toutes les données directement.

[onlytoine]

Ok, je suis aussi partisan de cette technique.

Maintenant, il parait qu'il ne faut pas employer de constructeur dans la méthode clone, avez en "pattern", des règles pour écrire cette méthode ?

[natha]

Clone ça marche très bien.
Il ne faut effectivement pas appeler de constructeur à cause de l'héritage, c'est pour ça qu'il faut toujours appeler super.clone() si vous surcharger clone.

[adiGuba]

Salut,



Grosso-modo j'utilise cette forme là :

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
21
22
23
24
25
26
class MonObject implements Cloneable {
 
	private String name;
	private Date date;
 
	public MonObject(String name) {
		this.name = name;
		this.date = new Date();
	}
 
	public Object clone() {
		try {
			// On clone l'objet courant :
			MonObject clone = (MonObject)super.clone();
 
			// On clone les eventuels attributs a protéger
			clone.date = (Date)clone.date.clone();
 
			// Et on retourne la valeur obtenu :
			return clone;
		} catch (CloneNotSupportedException e) {
			// Cela ne devrait jamais arriver :
			throw new RuntimeException(e);
		}
	}
}

• On passe clone() en public pour faciliter son utilisation (mais cela dépend aussi du besoin).
• On commence par appeler super.clone(), qui doit retourner le type réel de l'instance (donc éventuellement une classe fille).
• On duplique aussi les éventuels attributs a protéger via encapsulation. C'est inutile pour les types immuables (comme String).
• Enfin perso je reduis l'exception au silence, puisqu'elle ne devrait jamais arriver. Cela permet une utilisation plus simple du clone() (il n'y a pas d'obligation à gérer les RuntimeExceptions).

A noter qu'avec Java 5.0 on peut utiliser la covariance du type de retour, afin d'éviter de se trainer des Object et des cast :

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
	@Override
	public MonObject clone() {
		try {
			// On clone l'objet courant :
			MonObject clone = (MonObject)super.clone();
 
			// On clone les eventuels attributs a protéger
			clone.date = (Date)clone.date.clone();
 
			// Et on retourne la valeur obtenu :
			return clone;
		} catch (CloneNotSupportedException e) {
			// Cela ne devrait jamais arriver :
			throw new RuntimeException(e);
		}
	}

a++

[onlytoine]

Merci pour les réponses détaillées

CloneNotSupportedException peut se produire dans mon cas car je permet l'ajout de module externe...

Sinon, je suis parti sur la définition d'une interface propre Cloneable étendant la Cloneable du JDK

Maintenant, avez-vous eu des soucis d'implémentation du clonage ? Je pense en particulier au "deep clone"...

Anthony

[Pallando]

J'ai recemment utiliser un clone Hybride
du type
Dans un exemple Area contient des attributs dont un qui est du Tyoe Properties

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
public Object clone(String baseName, Template template) {
		Area res = null;
		try {
			res = (Area) super.clone();
		} catch (CloneNotSupportedException e) {
		}
		res.setId(id + "/" + baseName);
		res.setProperties((Properties) properties.clone(baseName, template));
		return res;
	}

En gros j'écris ma propre méthode clone (qui ne surchage pas clone de Object puisqu'avec un paramètre) mais j'appelle bien la méthode super.clone().
Pour le deep clone, effectivement il faut le faire à la main (dans mon cas écrire le clone(String aaa) dans Properties).
Enfin je ne fait pas remonter l'exception pour ne pas avoir a gérer des cas impossible.

[onlytoine]

Intéressant, le template te sert à quoi dans ton cas ? Une sorte de contexte ?

Pour le "deep clone", je le fais aussi à la main... Il y a bien la possibilité de le faire par sérialisation d'une grappe d'objets ou alors via une méthode supra-intelligente basée sur la réflexion, mais finalement ces techniques restent consommatrices et lentes...

Anthony