Discussion :

[Koudou]

Bonjour,

J'ai besoin de cloner des objets contenant un identifiant donné à la création.

La construction se fait en gros de cette manière :

Code :

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

1
2
3
public Item(String id) {
 this.identifiant = id;
}

Je vois à priori deux façons de cloner cet objet :

• créer une méthode clone(String id) qui ne respecte pas l'interface Cloneable.
• appeler systématiquement item.clone() puis item.setId(id) ce qui est dangereux puisque rien ne garanti qu'on n'oubliera pas de changer l'identifiant.

Quel est le moyen le plus propre pour pouvoir cloner ce type d'objet ? Merci.

[slim_java]

bonjour.
FAQ : Pourquoi et comment redéfinir la méthode equals() ?

[thebloodyman]

Salut,

La première méthode est meilleure puisque t'as l'atomicité du traitement et en plus un nommage explicite de la méthode.
Maintenant, je n'appellerais pas la méthode clone(String id). Ca peut porter à confusion. J'utiliserais plutôt qqch comme cloneWithNewId(String id).

Remarque : si tu as beaucoup de types d'objet à cloner et que le traitement est toujours le même (tu changes que l'id) , je te conseillerais de sortir la méthode clone dans une classe utilitaire et d'utiliser les apis faites pour : beanutils de Spring par exemple.

[tchize_]

pourquoi tu n'implémente pas cloen de Cloneable, et que tu définis dedans l'id du clone au même id que le parent?

[thebloodyman]

tchize_,

pourquoi tu n'implémente pas cloen de Cloneable, et que tu définis dedans l'id du clone au même id que le parent?

J'ai l'impression que justement il ne voulait pas l'id soit "cloné" dans le nouvel objet.

[Koudou]

Merci de vos réponses.

Effectivement par définition un identifiant est unique et immuable (un String final donné à la construction de l'objet) donc je voudrais pouvoir cloner l'objet tout en passant un identifiant en argument.

Je vais suivre le conseil de thebloodyman et créer une méthode cloneWithNewId(String newId).

Merci

[tchize_]

par définition un clone est identique

[Alain Defrance]

+1 tchize_

On peut cloner une instance (c'est à dire avoir une copie), puis être amené à faire varier l'état du clone (ici l'id).

[thebloodyman]

On peut cloner une instance (c'est à dire avoir une copie), puis être amené à faire varier l'état du clone (ici l'id).

Quel intérêt si on connait l'id dès la création du nouvel objet ?

[thelvin]

Oui fondamentalement il s'agit juste de fournir une méthode utilitaire qui prend en paramètre un nouvel id et qui
- d'une part, clone l'objet, par appel de clone() ou par ce qu'on veut ;
- d'autre part, change l'id de l'objet cloné.

Je vois pas vraiment ce qui se discute, là.

[Koudou]

Comme je l'ai dit, par définition un identifiant est unique et invariable.

Dans le cas qui m'intéresse l'identifiant est déclaré final et initialisé à la création il n'est donc pas possible de le changer par la suite.

Une méthode set(String id) me paraît dangereuse parce qu'on pourrait changer l'identifiant d'un objet alors que celui-ci est peut-être en cours d'utilisation.

[thelvin]

Eh bien, un constructeur dans lequel tu passes l'objet à cloner et l'id à utiliser.

C'est ce que je voulais dire par "appel de clone() ou par ce qu'on veut."