Problème pour faire bouger deux objets simultanement

Version imprimable

Bonjour,
Je suis entrain de réaliser un mini-jeu(pong-like), mes deux raquettes(correspondantes aux deux joueurs) bougent en fonction des boutons pressées au clavier, mais le problème est que on ne peut les faire bouger simultanément, c'est soit l'une soit l'autre.
Je vous donne un aperçu de mon code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 public void init() { addKeyListener(this); } public void keyReleased(KeyEvent e){} public void keyTyped(KeyEvent e) {} public void keyPressed(KeyEvent e) { /* ici je définie les actions en fonction des touches pressées pour la raquette player1 : ZQSD pour la raquette player1 : UP DOWN LEFT RIGHT */ }
Je voudrais savoir comment faire pour avoir deux Keylistener : un que gère les touches pressées par le player1, et lautre pour le player2.

Est-ce-que je dois appeler une seconde fois addKeylistener?
Mais dans ce cas comment attribuer un écouteur à un joueur, et l'autre écouteur à l'autre joueur.

J'éspère avoir été à peu près clair, merci pour votre aide.

20/04/2007, 12h58
L Ankou
Ca c'est un problème intéressant !

Le prob ce situe donc dans le fait de faire
Code:

1 2 3 4 5 if(touche pressée = joueur){ gérer joueur 1 }else{ gérer joueur 2 }
Tu peux peut être mettre un thread dans gérer joueur 1 et gérer joueur 2?
20/04/2007, 13h04
LinuxUser

Désolé mais je ne sais pas ce que c'est qu'un thread.
20/04/2007, 13h27
LinuxUser

Je me suis renseigné sur les threads, j'ai à peu près compris comment ça march(création, démarrage,...), mais je n'ai pas compris comment attribuer une tâche à un thread.
20/04/2007, 13h30
®om

Citation:

Envoyé par L Ankou

Tu peux peut être mettre un thread dans gérer joueur 1 et gérer joueur 2?

Cela ne changerait rien a priori, car les évènements sont générés à partir de l'edt, donc 1 seul thread!
A priori, si les 2 appuient sur une touche en même temps, je pense que ça te génèrera 2 events, mais forcément de manière séquentielle (mais très rapidement)...
20/04/2007, 13h37
LinuxUser

Justement je voudrais savoir comment faire pour gerer deux evenement simultanés (ou presque) sachant que dans ma méthode keyPressed j'ai declaré un seul evenement KeyEvent e, et que par conséquent il n'y à qu'une raquette qui bouge à la fois.
Je voudrais savoir comment faire pour avoir deux écouteur aux clavier ou deux evenments(si c'est possible) à la fois.
20/04/2007, 13h47
®om

Citation:

Envoyé par juve1897

Justement je voudrais savoir comment faire pour gerer deux evenement simultanés (ou presque) sachant que dans ma méthode keyPressed j'ai declaré un seul evenement KeyEvent e, et que par conséquent il n'y à qu'une raquette qui bouge à la fois.
Je voudrais savoir comment faire pour avoir deux écouteur aux clavier ou deux evenments(si c'est possible) à la fois.

avec un if(la touche correspond au joueur 1) {...} else {...}
ça devrait être ok...

ça sera à la suite, mais à la vitesse où ils seront, tu ne devrais pas avoir de problème...

C'est à peu pres ce que je fais, le problème c'est que si par exemple le joueur 1 décide de monter(ou une autre action) le joueur 2 reste immobile pendant ce temps, les deux joueurs ne peuvent bouger simultanément.
Je vais plutot te donner un aperçu du code :

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
34
35
 
public void keyPressed(KeyEvent e)
  {
      if((player1.r.y+50)<(getHeight()))
	    if(e.getKeyCode()==KeyEvent.VK_S)
	      player1.r.y += 10;
 
	if(player1.r.y>=0)  
	  if(e.getKeyCode()==KeyEvent.VK_Z)
	    player1.r.y -= 10;
 
	if(player1.r.x-10>=0)
	  if(e.getKeyCode()==KeyEvent.VK_Q)
	    player1.r.x -= 10;
 
	if((player1.r.x+5)<(getWidth()/2-10))
	    if(e.getKeyCode()==KeyEvent.VK_D)
	      player1.r.x += 10;
 
	if((player2.r.y+50)<(getHeight()))
	  if(e.getKeyCode()==KeyEvent.VK_DOWN)
	    player2.r.y += 10;
 
	if(player2.r.y>=0)  
	  if(e.getKeyCode()==KeyEvent.VK_UP)
	    player2.r.y -= 10;
 
	if((player2.r.x-10)>=(getWidth()/2))
	  if(e.getKeyCode()==KeyEvent.VK_LEFT)
	    player2.r.x -= 10;
 
	if(player2.r.x+5<=getWidth()-10)
	    if(e.getKeyCode()==KeyEvent.VK_RIGHT)
	      player2.r.x += 10;
  }

Sachant que r est un champ de ma classe Raquette, et est un point qui represente le coin haut-gauche de la raquette.

Le problème est que ce code se base sur la répétition des touches quand on laisse appuyé. Et je pari que l'OS ne répète qu'un caractère quand on maintient deux touches appuyées.

La méthode usuelle pour les jeux est de se baser sur l'état des touches en jouant sur keyReleased et keyPressed. Soit en gérant leur état, soit avec quelque chose comme :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 public void keyPressed(KeyEvent e) { if(e.getKeyCode()==player1.keyDown) player1.setDirection(Player.DirectionEnum.DOWN); [...] } public void keyReleased(KeyEvent e) { if(e.getKeyCode()==player1.keyDown && player1.direction==Player.DirectionEnum.DOWN) player1.setDirection(Player.DirectionEnum.NONE); [...] }
Il faudra par ailleurs une boucle qui mette à jour à intervale régulier les positions des joueurs. Je pense qu'il faudra créer un thread pour ça, et donc synchroniser Player.setDirection, ou du moins déclarer Player.direction "volatile". Mais ça permettra de faire des déplacements plus petits plus souvant, d'où une meilleure animation.

20/04/2007, 16h43
xavlours

Citation:

Envoyé par Sivrît

Il faudra par ailleurs une boucle qui mette à jour à intervale régulier les positions des joueurs. Je pense qu'il faudra créer un thread pour ça, et donc synchroniser Player.setDirection, ou du moins déclarer Player.direction "volatile". Mais ça permettra de faire des déplacements plus petits plus souvant, d'où une meilleure animation.

Jusque là j'étais d'accord, mais là, c'est se faire du mal pour rien. Les javax.swing.Timers sont faits pour ça et simplifieront beaucoup la vie de juve1897.
21/04/2007, 15h28
LinuxUser
J'utilise déjà un Timer, il me sert visualiser le mouvement des différents objet(balle,raquettes). Mais par contre je vois pas comment l'utiliser pour gérer le mouvement des raquettes independaemment.Moi je l'utilise de la manière suivante :
Code:

1 2 3 4 5 6 7 8 9 Timer minuteur; public void start() { minuteur=new Timer(100,this); minuteur.start(); } public void actionPerformed(ActionEvent evt) {repaint();}
22/04/2007, 19h52
xavlours

Tu peux utiliser un timer par joueur, mais ça ne sera pas plus simple que d'utiliser un Timer global. En gros, l'idée, c'est lors des keyPressed et KeyReleased, de changer l'état de la commande du joueur (de immobile vers haut, par exemple). Ensuite, le Timer vérifie à chaque exécution l'état de la commande des joueurs et les déplace en fonction.
22/04/2007, 21h41
LinuxUser

Le probleme c'est que ne sait pas ce que fait la méthode keyReleased, si tu pouvais m'expliquer en gros comment elle marche et à quoi elle sert je pourais peut etre resoudre mon probleme. Merci de ton aide.
Sinon qu'est ce que tu pense de mon code, ou plus precisement de ma methode keyPressed et de mon Timer?

Ben en gros keyPressed est appelée lorsqu'une touche est enfoncée, keyReleased lorsqu'une touche est relachée. La différence avec le code que tu as posté dans tes 2 avant-derniers messages, c'est que la raquette ne bouge pas dans keyPressed, elle bouge dans l'actionPerformed du Timer, suivant une direction qui elle est modifiée dans le keyPressed et le keyReleased.

En rapide, ça ressemble à ça :
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 public void keyPressed(KeyEvent e) { if(e.getKeyCode() == KeyEvent.VK_UP) player1.yOrder += 1; // on enregistre l'ordre du joueur else if ... } public void keyReleased(KeyEvent e) { if(e.getKeyCode() == KeyEvent.VK_UP) player1.yOrder -= 1; // on oublie l'ordre du joueur else if ... } /** Methode exécutée par le timer */ public void actionPerformed(ActionEvent e) { player1.raquette.y += player1.yOrder; // on applique l'ordre du joueur // on s'assure de ne pas sortir de l'écran if(player1.raquette.y > maxY) player1.raquette.y = maxY; else if(player1.raquette.y < minY) player1.raquette.y = minY; ... // faire la même chose en x et pour le joueur 2 repaint(); }