[java] problême par rapports à l'héritage et au cast

Question

Bonjour &#224; tous,

je programme un jeu de cartes dans lequel il ya plusieurs joueurs, humain ou IA (control&#233; par l'ordinateur).

voici les classes joueur utilis&#233;es:

public class Joueur{

    //....

    public boolean isCPU(){

        if(this instanceof JoueurCPU){

            return true;

        }

        else{

            return false;

        }

    }

}



public class JoueurCPU extends Joueur{

    //...

    //ajout d'une m&#233;thode suppl&#233;mentaire

    public void jouer(){

    }

}



Si maintenant on effectue le test suivant, on obtiendra:

Joueur j=new Joueur();

JoueurCPU cpu=new JoueurCPU();

j.isCPU(); //retourne false

j.jouer(); //erreur;, m&#233;thode non d&#233;finie pour cette classe

cpu.isCPU();     //retourne true

jcpu.jouer(); //ok



Mon probl&#232;me est que je stocke mes joueurs dans un vecteur (plus
pr&#233;cis&#233;ment dans une file LIFO, grace &#224; la classe LinkedList). Je
voudrais pour chaque jouer de la file effectuer une action selon le
type, come le montre le listing ci-dessous (exemple):

LinkedList file=new LinkedList();

Joueur j=new Joueur();


JoueurCPU cpu=new JoueurCPU();

//on ajoute les joeurs &#224; la file:

file.add(j); //ajout &#224; l'indice 0

file.add(cpu); //ajout &#224; l'indice1



Joueur temp;

for(int i=0;i<2;i++){

    temp=(Joueur)liste.get(i);

    if(temp.isCPU()){

         //cas d'un joueur cpu

         temp.jouer

    }

    else{

          //cas d'un joueur humain

     }

}



C'est la que je me pose la question. Est-ce que le code est juste? Je
ne connais pas grand chose au cast et je n'ai pas trouv&#233; de cours assez
complets. Je me demande si temp ne sera pas toujours consid&#233;r&#233; comme
une instance de la classe Joueur. A moins qu'il fasse faire
temp=(JoueurCPU)liste.get(i) et recaster en focntion du type de joueur.

OU autre possibilit&#233;: red&#233;finir la m&#233;thode isCPU dans la class JoueurCPU.

on aurra par exemple dans la classe Joueur "return false" et dans la classe JoueurCPU "return true".



Si quelqu'un peut m'eclairer la dessus svp.

en vous remerciant

super_toinou · Answer

alors ton code n a pas l air trop faux mais c est ton approche qui est mauvaise je pense
a mon avis t as un joueur CPU et un humain


tu dervait donc avoir une classe


public abstract class Joueur{


....


public abstract jouer();


}

 pis une classe 

public class JoueurCPU extends Joueur{

public void jouer(){
//f&#233; les trucs pr un joueur CPU
}
}
et
public class JoueurHumain extends Joueur{

public void jouer(){
//f&#233; les trucs pr un joueur humain en gros ton else
}
}

 pis ds ta fifo tu f&#233; 
Joueur j1 = new JoueurCPU();
Joueur j2 = new JoueurHumain();

t ajoute et qd tu r&#233;cup tu f&#233; juste
joueur = (Joueur) liste.get(i);


temp.jouer();


et ca suffit, le polymorphisme joue pr toi ;)

++ Toinou

johanb · Answer

nikel, merci bien super toinou :D

je vais faire le coup des 2 classes qui h&#233;ritent de Joueur.

le seul probleme c'est que la m&#233;thode jouer de la classe JoueurHumain
ne sert &#224; rien (elle sera vide), pask'en fait j'ai mon application
(class Application) qui attend que le joeur humain ai cliqu&#233; sur une
carte dans la pr&#233;sentation (class Presentation).



en gros si c'est &#224; un joueur humain de jouer, on attend un clic de
l'interface, sinon on appelle la m&#233;thode jouer du JoueurCPU. Et le
probleme c'est que la classe Joueur n'a aucun lien avec l'interface dc
il ne peut pas r&#233;agir aux &#233;v&#232;nements.

en gros &#231;a serait:

 for(int i=0;i<2;i++){
     temp=(Joueur)liste.get(i);
     if(temp.isCPU()){
          //cas d'un joueur cpu: appel de la m&#233;thode jouer

        //celle ci g&#232;re directement le dialogue avec l'interface pour piocher des cartes...
          temp.jouer
     }
     else{
           //cas d'un joueur humain

            //on attend un &#233;v&#232;nement sur la pr&#233;sentation
      }
 }



Moi pas pens&#233; avoir &#233;t&#233; clair hug...d&#233;sol&#233;

Mais je garde ta soluce pour la m&#233;thode isCPU(), en abstract dans la classe Joueur,

retourne faux dans la classe JoueurHumain et vrai dans la classe JoueurCPU.



encore merci ! :D

johanb · Answer

une autre petite question...

si je fais:

Joueur j;



for(i=0;<=nombreJoueurs;i++){



//au hasard on cr&#233;e soit

j=new JoueurHumain;

//ou soit

j=new JoueurCPU;



file.add(j);

}


yaura pas de probl&#232;me? ou je dois faire:

JoueurHumain j;

JoueurCPU cpu;

for(i=0;<=nombreJoueurs;i++){



//au hasard fait soit

j=new JoueurHumain

file.add(cpu)

//ou soit

cpu=new JoueurCPU

file.add(cpu);

}


Perso je pr&#233;f&#232;re la 1ere solution mais je ne sais pas vraiment si &#231;a marche

encore d&#233;sol&#233; :$

super_toinou · Answer

alors sur ton post juste dessus la 1ere solution ne pose pas de pb ca va marcher sans soucis.

Par contre conceptuellement, ta m&#233;thode isCpu() n a pas gd chose a faire ds la classe JoueurHumain, ok ca va marcher 


mais tu peux tres bien faire le test comme ca :

for(int i=0;i<2;i++){
    temp=(Joueur)liste.get(i);
    if(temp ){
         //cas d'un joueur cpu: appel de la m&#233;thode jouer
       //celle ci g&#232;re directement le dialogue avec l'interface pour piocher des cartes...
         temp.jouer
    }
    else{
          //cas d'un joueur humain
          //on attend un &#233;v&#232;nement sur la pr&#233;sentation
     }
}

super_toinou · Answer

sinon je viens de penser a un truc, si ta partie

super_toinou · Answer

merde ca a plant&#233;, en gros je disai que tu pouvais utiliser les maps pr stocker les joueurs et avoir donc la possibilit&#233; de faire soit des map.get('"humain") ou map.get("cpu") soit parcourir ts les objets de ta map et de faire la solution du instanceof !!!

johanb · Answer

j'ai pas bien compris la solution avec la map (j'en ai jamais manipul&#233;), puis j'ai plus beaucoup de temps pour utiliser &#231;a :p



Sinon pour ta soluce plus haut j'ai pas compris: pourquoi "if(temp)" ?

super_toinou · Answer

sorry j reply un peu tard
le if temp c t pour if (temp

[java] problême par rapports à l'héritage et au cast

8 réponses

Votre réponse

Discussions similaires