Cloner un objet

Version imprimable

Cloner un objet

Bonjour à tous !

J'ai un petit problème pour cloner un objet. J'ai lu le tuto de Yann D'Isanto dans les cours de Java , mais rien n'y fait.

J'ai plusieurs questions :

- Est-ce que l'on peut cloner un objet dans une méthode de la classe de cet objet ? (une méthode qui appartient à cet objet) ou il faut le faire dans une classe extérieure ? (ce qui m'arrangerait pas du tout)

- Et ensuite, voici mon problème. J'ai l'impression que le clone de mon objet Echiquier ne marche pas, pourtant, aucune exception ne survient pendant l'opération. En fait, il attribue mon objet Echiquier à l'objet EchiquierCloné, c'est à dire qu'il modifie l'objet original. :cry:

Voici mon code :

Code:

1
2
3
4
5
6
7
8
9
10
 
public boolean uneMethode() {
...
// TODO: Le clone de l'échiquier marche pas ! Ca modifie le véritable échiquier ?!
// On clone l'échiquier
Echiquier echiquierClone = (Echiquier) this.clone();
// On joue un coup virtuellement
echiquierClone.joueCoup(position, autrePosition);
...
}

Cette méthode si situe dans ma classe Echiquier, et cette classe étend Cloneable.
Voici les attributs principaux de ma classe Echiquier :

Code:

1
2
3
4
5
 
// La taille du plateau est une constante
public static final int TAILLE_PLATEAU = 8;
// L'échiquier possède un tableau de cases
private Case plateau[][] = new Case[TAILLE_PLATEAU][TAILLE_PLATEAU];

Et ce tableau de Cases est initialisé comme ça dans mon constructeur :

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
32
33
 
int camp;  // 0 => blanc ; 1 => noir
for(camp = 0; camp < 2; camp++) {
    final int dernierNumero = TAILLE_PLATEAU - 1;
    final int ligneArriere = camp * dernierNumero; 
    final int ligneAvant = 1 + camp * (TAILLE_PLATEAU - 3); 
    final boolean couleurNoire = (camp == 1); 
 
    // Figures
    plateau[ligneArriere][0] = new Case(ligneArriere, 0, new Tour(couleurNoire, "T")); 
    plateau[ligneArriere][7] = new Case(ligneArriere, 7, new Tour(couleurNoire, "T"));
    plateau[ligneArriere][2] = new Case(ligneArriere, 2, new Fou(couleurNoire, "F"));
    plateau[ligneArriere][5] = new Case(ligneArriere, 5, new Fou(couleurNoire, "F"));
    plateau[ligneArriere][3] = new Case(ligneArriere, 3, new Dame(couleurNoire, "D"));
    plateau[ligneArriere][4] = new Case(ligneArriere, 4, new Roi(couleurNoire, "R"));
    plateau[ligneArriere][1] = new Case(ligneArriere, 1, new Cavalier(couleurNoire, "C"));
    plateau[ligneArriere][6] = new Case(ligneArriere, 6, new Cavalier(couleurNoire, "C"));
    // Pions
    plateau[ligneAvant][0] = new Case(ligneAvant, 0, new Pion(couleurNoire, ""));
    plateau[ligneAvant][1] = new Case(ligneAvant, 1, new Pion(couleurNoire, ""));
    plateau[ligneAvant][2] = new Case(ligneAvant, 2, new Pion(couleurNoire, ""));
    plateau[ligneAvant][3] = new Case(ligneAvant, 3, new Pion(couleurNoire, ""));
    plateau[ligneAvant][4] = new Case(ligneAvant, 4, new Pion(couleurNoire, ""));
    plateau[ligneAvant][5] = new Case(ligneAvant, 5, new Pion(couleurNoire, ""));
    plateau[ligneAvant][6] = new Case(ligneAvant, 6, new Pion(couleurNoire, ""));
    plateau[ligneAvant][7] = new Case(ligneAvant, 7, new Pion(couleurNoire, ""));
 
     // Aperçu du tableau de positions des pièces
     //  ...   ...   ...  ...
     // (2,0) (2,1) (2,2) ...
     // (1,0) (1,1) (1,2) ...
     // (0,0) (0,1) (0,2) ...
}

Et enfin, voici la méthode clone() dans toujours la même classe (Echiquier) :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
// Clone l'objet echiquier
    // TODO: Marche pas !
    @Override
    public Object clone() {
        Echiquier echiquier = null;
        try {
            echiquier = (Echiquier) super.clone();
        } catch (CloneNotSupportedException exception) {
            System.err.println("" + exception.getMessage());
        }
        // On clone le plateau de jeu
        echiquier.plateau = (Case[][]) plateau.clone();
        // TODO: Il faut peut être ici cloné aussi les pièces du plateau
        // mais comment faire ?
 
        // On retourne l'échiquier cloné
        return echiquier;
    }

-Est ce que les classes Case, Pion, Tour, Fou, Dame, Roi et Cavalier doivent aussi étendre Cloneable ? (Ce n'est pas le cas pour l'instant)

Merci pour les personnes qui prendront le temps de regarder mon portable et me proposer une solution !

19/09/2008, 15h21
BugFactory

Je pense que vous devriez parcourir "plateau" pour cloner les "Case" une par une, car les celles-ci contiennent des données spécifiques à un échiquier. Pour Pion, Tour, Fou, Dame, Roi et Cavalier, celà dépend : si elles contiennent des données spécifiques à un échiquier, alors oui.

Voici de nouveau ma méthode clone() de la classe Echiquier :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
// Clone l'objet echiquier
    // TODO: Marche pas !
    @Override
    public Object clone() {
        Echiquier echiquier = null;
        try {
            echiquier = (Echiquier) super.clone();
        } catch (CloneNotSupportedException exception) {
            System.err.println("" + exception.getMessage());
        }
        // On clone le plateau de jeu
        echiquier.plateau = (Case[][]) plateau.clone();
        // TODO: Il faut peut être ici cloné aussi les pièces du plateau
        for (int i = 0; i < TAILLE_PLATEAU; i++) {
            for (int j = 0; j < TAILLE_PLATEAU; j++) {
                echiquier.plateau[i][j] = (Case) plateau[i][j].clone();
            }
        }
        // On retourne l'échiquier cloné
        return echiquier;
    }

Mais si je fais ça, que la classe Case implémente l'interface Cloneable ou non, ça me fait une erreur pour la ligne en gras :
"clone() has protected access in java.lang.Object"
Pourquoi cette erreur ?? :cry:

19/09/2008, 15h38
BugFactory

Est-ce que Case implémente l'interface Cloneable?

Citation:

Envoyé par womannosky

Code:

(Case) plateau[i][j].clone();

Un peu ambigu comme écriture, mieux vaut ajouter les parenthèses :

Code:

((Case) plateau[i][j]).clone();

Au fait, quel est le type de plateau?

(Note : j'ai modifié mon message précédent au moment où vous postiez.)
19/09/2008, 15h39
womannosky

Citation:

Envoyé par BugFactory

Je pense que vous devriez parcourir "plateau" pour cloner les "Case" une par une, car les celles-ci contiennent des données spécifiques à un échiquier. Pour Pion, Tour, Fou, Dame, Roi et Cavalier, celà dépend : si elles contiennent des données spécifiques à un échiquier, alors oui.

Ok, je veux bien mais comment faire ? Pouvez vous m'aider ?
19/09/2008, 15h42
womannosky
Ok pour votre dernier message.
Case n'implémente pas l'interface Cloneable pour l'instant, il le faut ?

Votre écriture comme ça :
Code:

1 2 ((Case) plateau[i][j]).clone();
Ca ne fonctionne pas, j'ai le même message

Et plateau est de type Case[][].

Merci pour votre aide
19/09/2008, 15h55
verbose

Oui. Non seulement Case doit implémenter Cloneable, mais en plus tu dois redéfinir la méthode clone() héritée de Object pour cloner tous les attributs complexes s'il y en a et relever la visibilité à public.

Ok, Verbose, merci !
Je viens donc d'implémenter Cloneable dans ma classe Case, et d'y ajouter ceci :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 // Clone la case @Override public Object clone() { Case caseClone = null; try { caseClone = (Case) super.clone(); } catch (CloneNotSupportedException exception) { System.err.println("" + exception.getMessage()); } // On retourne la case cloné return caseClone; }
Grâce à cette méthode, je n'ai plus l'erreur de tout à l'heure. :)

Et comme les cases contiennent des pièces, il faut que je fasse pareil pour la classe Piece ??
Si oui, ça va être interminable...

19/09/2008, 16h05
verbose

Oui, il faut aussi que tu clones les Pieces :aie:

Ma classe échiquier contient un plateau de cases qui contiennent chacune une pièce.
Voici mes trois méthodes clone() :
- Classe Echiquier :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
// Clone l'objet echiquier
    // TODO: Marche pas !
    @Override
    public Object clone() {
        Echiquier echiquierClone = null;
        try {
            echiquierClone = (Echiquier) super.clone();
        } catch (CloneNotSupportedException exception) {
            System.err.println("" + exception.getMessage());
        }
        // On clone le plateau de jeu
        echiquierClone.plateau = (Case[][]) plateau.clone();
 
        // Il faut cloner aussi les pièces du plateau
        for (int i = 0; i < TAILLE_PLATEAU; i++) {
            for (int j = 0; j < TAILLE_PLATEAU; j++) {
                echiquierClone.plateau[i][j] = (Case) ((Case) plateau[i][j]).clone();
            }
        }
        // On retourne l'échiquier cloné
        return echiquierClone;
    }

- Classe Case :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
// Clone la case
    @Override
    public Object clone() {
        Case caseClone = null;
        try {
            caseClone = (Case) super.clone();
        } catch (CloneNotSupportedException exception) {
            System.err.println("" + exception.getMessage());
        }
        // S'il y a une pièce sur la case
        if (piece != null) {
            // On clone la pièce de la case
            caseClone.piece = (Piece) piece.clone();
        }
 
        // On retourne la case cloné
        return caseClone;
    }

- Classe Piece :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
// Clone la pièce
    @Override
    public Object clone() {
        Piece pieceClone = null;
        try {
            pieceClone = (Piece) super.clone();
        } catch (CloneNotSupportedException exception) {
            System.err.println("" + exception.getMessage());
        }
        // On retourne la case cloné
        return pieceClone;
    }

Mes trois classes implémentent l'interface Cloneable, mais celà ne marche toujours pas ! :cry: :cry:
Quand je veux cloner mon échiquier comme ça :

Code:

1
2
3
4
5
6
// TODO: Le clone de l'échiquier marche pas ! Ca modifie le véritable échiquier ?!
// On clone l'échiquier
Echiquier echiquierClone = (Echiquier) this.clone(); // this car je suis dans la classe Echiquier
// On joue le coup sur l'échiquier cloné pour pouvoir tester le chemin libre ensuite
echiquierClone.joueCoup(position, positionAutre);

Et bien, ca me fait une erreur car l'échiquier que je veux cloner est attribué à l'échiquier cloné, donc quand je modifie l'échiquier cloné, je modifie en même temps l'échiquier de base que je ne veux pas cloner !

Ma ligne en gras est-elle correcte sachant que je clone l'objet en cours (this) ?

Salut,

Citation:

Envoyé par womannosky

Et comme les cases contiennent des pièces, il faut que je fasse pareil pour la classe Piece ??
Si oui, ça va être interminable...

Oui : si tu veux cloner des objets, il faut bien implémenter leurs méthodes clone(). Sinon tu partageras les mêmes instances et les modifications impacteront les deux instances...

Ton problème vient que pour les tableaux, il faut cloner TOUS les éléments, y compris les sous-tableaux :
Code:

1 2 3 4 5 6 7 8 9 10 11 // On clone le tableau : echiquier.plateau = echiquier.plateau.clone(); for (int i = 0; i < TAILLE_PLATEAU; i++) { // Les sous-tableaux echiquier.plateau[i] = echiquier.plateau[i].clone(); for (int j = 0; j < TAILLE_PLATEAU; j++) { // ainsi que chaque éléments : echiquier.plateau[i][j] = (Case) plateau[i][j].clone(); } }
Sinon j'aurais deux conseils par rapport à ton code :
Code:

1 2 3 4 5 6 7 8 9 10 11 @Override public Object clone() { Case caseClone = null; try { caseClone = (Case) super.clone(); } catch (CloneNotSupportedException exception) { System.err.println("" + exception.getMessage()); } // On retourne la case cloné return caseClone; }
1/ Afficher un message c'est bien... mais pas très pratique car pas forcément très lisible...
Théoriquement tu ne devrais jamais avoir de CloneNotSupportedException, sauf si tu n'implémentes pas Cloneable. Donc l'exception pourrait être ignoré avec un simple message comme ici, mais cela peut s'avérer peu pratique !

Si l'exception remonte par un malheureux hasard, tu va te contenter d'afficher un message sur la console, et ta méthode clone() va continuer normalement en retournant null.
Du coup ton programme va planter avec un NullPointerException peu explicite, et pour peu que le message précédent passe inaperçu tu vas perdre du temps à en rechercher la cause... :(

Perso je préfère les codes "fail-fast", litérallement : qui plante rapidement.
C'est à dire que plutôt que d'afficher une message et de tenter de continuer comme si de rien n'était, je préfère remonter une exception immédiatement :
Code:

1 2 3 } catch (CloneNotSupportedException exception) { throw new RuntimeException(exception); }
Au moins j'obtiens un stacktrace complet qui me donne toutes les infos neccessaire ;)
La RuntimeException permet de faire planter le programme sans obliger le développeur à les traiter spécifiquement.
Bref CloneNotSupportedException n'est plus une exception qu'on ignore, mais un état invalide qui provoquera un plantage (qui pourra être corriger plus rapidement).

2/ Depuis Java 5 on peut retourner un type différent de celui déclarer dans la classe parente : c'est la covariance (lire : Quelles sont les règles à respecter pour redéfinir/implémenter une méthode ?).
Cela permet d'éviter de s'embêter avec de multiple cast.

Au final cela donnerait :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 @Override public Case clone() { try { Case caseClone = (Case) super.clone(); // Traitement éventuel des attributs return caseClone; } catch (CloneNotSupportedException exception) { throw new RuntimeException(e); } }
a++

PS : Attention également aux références croisées, qui peuvent faire partir le tout en boucle infini...

19/09/2008, 16h54
womannosky
AdiGuba, toujours aussi efficace. Merci pour tes réponses multiples, je vais tenir compte de tes conseils. Surtout que mon code ne plante plus, il me fait un truc bizarre mais bon, ça doit venir de moi. Merci ! :yaisse2:

Pour le :

Code:

throw new RuntimeException(exception);

Je peux l'utiliser dans tous mes catch () ??

Mon échiquier contient une liste de coups :
Code:

1 2 private List<Coup> listeCoups = new ArrayList<Coup>();
Et je voudrais cloner cet objet List, comment faire pour ça ??
Code:

1 2 echiquierClone.listeCoups = (List<Coup>) listeCoups.clone();
Ca ne marche pas :cry:
20/09/2008, 17h45
womannosky
J'ai trouvé seul pour cloner une List. Voici le code :
Code:

1 2 echiquier.listeCoups = new ArrayList<Coup>(listeCoups);
A+