Discussion :

[kokoboy]

Bonjour, je suis confronté a un probleme. J'espere que vous pourrez m'aider.

Alors voila mon probleme :

J'ai une classe Plateau qui possède un attribut de type: Case [][]
J'ai une classe Case qui possède un attribut de type: Piece
et une classe Piece qui possède un attribut de type: Plateau


Dans l'implementation initiale toute mes pieces pointent sur le meme objet plateau (appelé damier)

Mon probleme vient de l'implementation de la methode clone() pour ma classe Plateau. En effet pour cloner ma classe Plateau je dois lancer les clones de mes Cases.
Le clone de Case lancera quant a lui le clone de Piece , et pour cloner un objet Piece je devrai cloner son attribut de type Plateau...
Ce qui va m'entrainer dans une boucle infini!

Quelqu'un saurait il comment cloner mon Objet Plateau sans partir dans cette boucle?


Merci d'avance .

[divxdede]

1ere question, pourquoi ta classe piece tient une reference sur le plateau ?

[yann2]

Bonjour,

redéfinie la methode clone() dans la classe Piece

[kokoboy]

pourquoi ta classe piece tient une reference sur le plateau ?

Ma classe Piece possede une reference sur Plateau car elle a besoin d'acceder a des methodes de mon objet damier(Type Plateau).

redéfinie la methode clone() dans la classe Piece

Je veux bien mais je dois bien copier mon attribut de type Plateau, comment faire sans utiliser son clone().

[GrandFather]

Bonjour,

il faut éviter, d'une manière générale, les références cycliques entre objets. Ca te pose des problèmes pour le clonage, mais ca peut aussi en poser au garbage collector pour récupérer la mémoire inutilisée.

Tu devrais donc retirer la référence à l'objet Plateau dans tes objets Piece, et les rajouter dans les paramètres de fonctions. Exemple:

Remplace :

Code :

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

1
2
3
4
5
6
class Piece {
 
  private Plateau monPlateau;
 
  void deplacePiece() { ... }
}

par :

Code :

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

1
2
3
4
class Piece {
 
  void deplacePiece(Plateau unPlateau) { ... }
}

[kokoboy]

Merci d'avoir repondu aussi vite.

Je vais effectivement deplacer la reference dans les methodes.