projet1 méthode clone()

bonjour à tous

j'ai un problem avec la méthode
protected Object clone ()

[code]
protected Object clone() {
Pion[][] copiePlateau = plateau.clone();
Joueur unJoueurCourant = joueurCourant.clone();
Joueur unJoueurGagnant = joueurGagnant.clone();
Joueur[] lesJoueurs = joueurs.clone();
Etat copie = new Etat(copiePlateau,unJoueurCourant,unJoueurGagnant,lesJoueurs);
return copie;
[code]

quand je le compile il me donne érreur message

[code]

Etat.java:89: clone() has protected access in java.lang.Object
Joueur unJoueurCourant = joueurCourant.clone();
^
Etat.java:89: incompatible types
found : java.lang.Object
required: gxxxxx.projet1.Joueur
Joueur unJoueurCourant = joueurCourant.clone();
^
Etat.java:90: clone() has protected access in java.lang.Object
Joueur unJoueurGagnant = joueurGagnant.clone();
^
Etat.java:90: incompatible types
found : java.lang.Object
required: gxxxxx.projet1.Joueur
Joueur unJoueurGagnant = joueurGagnant.clone();
^
Note: /home/my/gxxxxx/projet1/Position.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
4 errors

[/code]

donc je fait un casting

[code]

protected Object clone() {
Pion[][] copiePlateau = plateau.clone();
Joueur unJoueurCourant = (Joueur)joueurCourant.clone();
Joueur unJoueurGagnant = (Joueur)joueurGagnant.clone();
Joueur[] lesJoueurs = joueurs.clone();
Etat copie = new Etat(copiePlateau,unJoueurCourant,unJoueurGagnant,lesJoueurs);
return copie;

[/code]

il m'affiche encore érreur

[code]


Etat.java:89: clone() has protected access in java.lang.Object
Joueur unJoueurCourant = (Joueur)joueurCourant.clone();
^
Etat.java:90: clone() has protected access in java.lang.Object
Joueur unJoueurGagnant = (Joueur)joueurGagnant.clone();
^
Note: /home/my/gxxxx/projet1/Position.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
2 errors

[code]

merci d'avance

protected Object clone() {
Pion[][] copiePlateau = plateau.clone();
Joueur unJoueurCourant = (Joueur)joueurCourant.clone();
Joueur unJoueurGagnant = (Joueur)joueurGagnant.clone();
Joueur[] lesJoueurs = joueurs.clone();
Etat copie = new Etat(copiePlateau,unJoueurCourant,unJoueurGagnant,lesJoueurs);
return copie;

Je ne sais pas quel est l'énoncé, mais :
- pourquoi protected ? Si c'est une méthode "à toi" (utilisée en interne), private suffit, non ?
- pourquoi dire qu'elle retourne un Object ? A tous les coups tu retournes un Etat, donc autant être plus spécifique et dire qu'elle retourne un Etat.

Cela pourrait également résoudre les erreurs citées. Le compilateur n'est pas toujours très précis, mais ici il pointe les deux bouts de code qui est sont en tort, un peu d'anglais suffit généralement à trouver ce qui cloche

PS: si ton projet/dossier est vraiment nommé "gxxx", renomme-le avec ton numéro d'étudiant, histoire que le prof qui va corriger le code sache à qui il a affaire

didi a écrit:

Code:




Etat.java:89: clone() has protected access in java.lang.Object
      Joueur unJoueurCourant = (Joueur)joueurCourant.clone();
                                                    ^
Etat.java:90: clone() has protected access in java.lang.Object
      Joueur unJoueurGagnant = (Joueur)joueurGagnant.clone();
                                                    ^
Note: /home/my/gxxxx/projet1/Position.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
2 errors



merci d'avance

Tu es dans le même package que la classe Joueur ? Si oui, je ne vois pas comme ça, si non, la méthode clone de joueur t'es inaccessible..

Je ne comprend pas très bien la méthode clone. J'ai posé plusieurs fois la question et on m'as répondu qu'elle fait une copie de l'objet courant.
Mais quand on n'est dans l'objet tous ces attributs sont courant.

DjMan42 a écrit:Je ne comprend pas très bien la méthode clone. J'ai posé plusieurs fois la question et on m'as répondu qu'elle fait une copie de l'objet courant.
Mais quand on n'est dans l'objet tous ces attributs sont courant.

Elle retourne une copie de l'object auquel elle appartient.

Pour simplifier, c'est la même représentation (attributs avec les mêmes valeurs, etc..) mais stockés à des endroits différents en mémoire. Ainsi, si tu supprimes l'objet de base (cas d'un tableau ou d'une map), sa copie est "safe".

Bon, pour tout compliquer, on rajoute le garbage collector et tout les petites joyeusetés de Java.

Bien plus "caché" comme implémentation qu'en C++... Mais on t'en parlera donc plus en profondeur quand tu verras le C++ et tu comprendras vraiment ce qui est copié et comment... et les pièges dans lequel il ne faudra pas tomber

Un grand merci à tous. Je comprend à présent.

[quote="nvs"]
lorsque Wark évoque la suppression de l'objet de base, il se trompe, dans la mesure où c'est impossible en java si une copie réfère également cet objet. ce qu'il a probablement voulu dire c'est que si tu modifies (ou supprimes) les objets qui sont attributs de l'original, les attributs du clone ne sont pas modifiés. ils correspondent à une duplication de ceux de l'original au moment du clonage, puis ils vivent chacun leur propre vie.
/quote]

Oui oui, d'où j'ai parlé du garbage collector (sans redéfinir sa fonction). Ceci dit, j'aurais effectivement dû prendre la modification en exemple. :-)

Ce lien n'est pas mal non plus ...
http://ydisanto.developpez.com/tutoriels/j2se/cloneable/

... et une p'tite note sur mon blog à ce sujet.