Probleme : clonage de tableau d'objets

Version imprimable

bonjour a tous,

je planches depuis déjà un petit moment sur un probleme que je n'arrive pas a resoudre, je vous expose mon probleme si quelqu'un aurait une piste je suis preneur.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 public class MaClass { MonObjet [] copy_MonTabObjet; public void test( MonObjet [] MonTabObjet) { copy_MonTabObjet = (MonObjet[]) MonTabObjet.clone(); System.out.println("original = "+MonTabObjet[0].MonNom); System.out.println("COPY = "+copy_MonTabObjet[0].MonNom); copy_MonTabObjet[0].MonNom = "Paulo"; System.out.println("original = "+MonTabObjet[0].MonNom); System.out.println("COPY = "+copy_MonTabObjet[0].MonNom); } }
L'execution du code source ci-dessous me donne le resulat suivant :cry: :

original = Sebastien
COPY = Sebastien
original = Paulo
COPY = Paulo

alors que souhaiterais avoir :D :

original = Sebastien
COPY = Sebastien
original = Sebastien
COPY = Paulo

est-ce quelq'un aurait une idée pour que je puisse disposé d'un "vrai clone" de mon tableau MonTabObjet de fait que je puisse modifier des données dans le tableau cloner sans modifier l'original.

Merci d'avance pour toute vos suggestions.

Sébastien

25/09/2006, 16h43
le y@m's

Je ne crois pas que lorsque tu clone un tableau tous ses éléments le soient aussi (et je pense qu'il en est de même pour la méthode System.arraycopy() ).
Il va te valoir boucler sur ton tableau pour cloner tes objets un par un ;).
25/09/2006, 16h59
sebastien2222

dans le cas ou j'ai des variables de type chaine de caracteres cela fonctionne mais si dans ma classe MonObjet j'ai des objets comment je pourrais faire ? (une idée?? je seche ..)
25/09/2006, 17h08
le y@m's

Citation:

Envoyé par sebastien2222

dans le cas ou j'ai des variables de type chaine de caracteres cela fonctionne mais si dans ma classe MonObjet j'ai des objets comment je pourrais faire ? (une idée?? je seche ..)

Cela vient d'une mauvaise implémentation du clonage de tes objets.
En effet, pour les attributs immuables (comme les Strings) il n'y a pas de problème, par contre les attributs non immuables doivent être, eux aussi, clonés.

[edit] Tu as plus d'informations dans la Javadoc de la méthode clone() ;) [/edit]
25/09/2006, 17h15
sebastien2222

Citation:

Cela vient d'une mauvaise implémentation du clonage de tes objets

peux tu préciser ? je suis pas vraiment un expert (java et objet)

si dans mon objet MonObjet j'ai un seul et unique objet nommé AutreObjet qui contient lui meme des chaines de caracteres quel serait la solution en terme d'implémentation pour que le clonage fonctionne.

désolé pour les questions trivial mais je debute dans le "metier".

Citation:

Envoyé par sebastien2222

si dans mon objet MonObjet j'ai un seul et unique objet nommé AutreObjet qui contient lui meme des chaines de caracteres quel serait la solution en terme d'implémentation pour que le clonage fonctionne.

Dans la méthode clone() du premier objet tu doit aussi cloner l'attribut qui est un objet (et qui doit donc être Cloneable).

Voici un exemple :
Considéront une famille (classe Familly) qui possède plusieurs membres (des personnes, classe Person) qui eux ont un nom et un age. Nous voulons pouvoir cloner une famille et donc que la classe Familly implemente l'interface Cloneable (et donc la méthode clone()).
Cependant, la famille contient des personnes, il faut donc que la class Person implémente, elle aussi, l'interface Cloneable.

Person
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 public class Person implements Cloneable { private String name; private int age; public Person(String name, int age) { this.name = name; this.age = age; } public Object clone() { // on peut utiliser directement les attributs 'name' et // 'age' car leurs classes respectives sont immuable return new Person(name, age); } }
Familly
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 public class Familly implements Cloneable { private Person father; private Person mother; private ArrayList<Person> childs; public Familly(Person father, Person mother, ArrayList<Person> childs) { this.father = father; this.mother = mother; this.childs = childs; } public Object clone() { // on doit cloner les attributs 'father', 'mother' et 'childs' // car leurs classes respectives ne sont pas immuables return new Familly((Person) father.clone(), (Person) mother.clone(), (ArrayList<Person>) childs.clone()); } }

25/09/2006, 18h04
sebastien2222

bon ok je comprends ton explication par contre le type de retour de clone() n'est pas corecte ? normalement clone doit retourner un type Object alors que dans ton exemple il retourne un type Familly dans le cas de la classe famille.

donc dans la pratique cela ne fonctionne pas ? ou alors je suis passé a coté ?
25/09/2006, 19h15
le y@m's

C'est faisable, mais il est vrai qu'il vaut mieux renvoyer un Object (j'ai corrigé mon post ;)).
25/09/2006, 19h27
®om

Citation:

Envoyé par sebastien2222

bon ok je comprends ton explication par contre le type de retour de clone() n'est pas corecte ? normalement clone doit retourner un type Object alors que dans ton exemple il retourne un type Familly dans le cas de la classe famille.

donc dans la pratique cela ne fonctionne pas ? ou alors je suis passé a coté ?

Tu as le droit, depuis java 1.5, de modifier le type de retour d'une fonction dans une classe fille, à condition que le nouveau type soit un sous-type de l'"ancien" type...
26/09/2006, 09h08
sebastien2222

merci y@m's, j'en suis arrivé à la meme conclusion hier soir, merci d'avoir pris de temps pour m'expliquer tt ca et bonne journée a toi.

A+

Sébastien

Après avoir relu la Javadoc, je me suis rendu compte que mes implémentations de la méthode clone() ne respectaient pas les conventions :aie:.
Voici un code beaucoup plus juste ;) :

Person.java

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
public class Person implements Cloneable {
    private String name;
    private int age;

    public Person(String name, int age) {
        this.name = name;
        this.age = age;
    }

    public Object clone() {
        Object o = null;
        try {
          // On récupère l'instance à renvoyée par l'appel de la 
          // méthode super.clone()
          o = super.clone();
        } catch(CloneNotSupportedException cnse) {
          // Ne devrait jamais arriver car nous implémentons 
          // l'interface Cloneable
          cnse.printStackTrace(System.err);
        }
        // on renvoie le clone
        return o;
    }
}

Familly.java

Code:

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
27
28
29
30
31
public class Familly implements Cloneable {
    private Person father;
    private Person mother;
    private ArrayList<Person> childs;

    public Familly(Person father, Person mother, ArrayList<Person> childs) {
        this.father = father;
        this.mother = mother;
        this.childs = childs;
    }

    public Object clone() {
       Familly familly = null;
        try {
          // On récupère l'instance à renvoyée par l'appel de la 
          // méthode super.clone()
          familly = (Familly) super.clone();
        } catch(CloneNotSupportedException cnse) {
          // Ne devrait jamais arriver car nous implémentons 
          // l'interface Cloneable
          cnse.printStackTrace(System.err);
        }
        // on clone les attributs de type Person car ils ne sont pas immuables
        familly.father = (Person) father.clone();
        familly.mother = (Person) mother.clone();
        // on clone l'attribut de type ArrayList car il n'est pas immuable
        familly.childs = (ArrayList<Person>) childs.clone();
        // on renvoie le clone
        return familly;
    }
}