Comment diriger un laser (imageIcon) vers le haut d'un JPanel

Version imprimable

Bonjour a tous,

je suis en plein dans un codage d'un petit jeu ou le but est d'eliminer le max d'aliens venant d'en haut(du Jpanel), ça j'ai reussis a faire, mais le sens inverse non:Je dois envoyer des lasers vers eux.

voiçi un bout de code et dites moi svp la ou ça cloche grave.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 JLabel laser= new JLabel(imgLaser);
            JLabel vaisseau= new JLabel(imgVaisseau);
 
         public void keyPressed(KeyEvent e) 
        {
            if(e.getKeyCode()==KeyEvent.VK_SPACE)
            {
                //position initiale du laser a vaisseau a getY()car c'est le vaisseau qui "porte" le laser, jusqu'a le point y du container qui == 0.
                for(int i=vaisseau.getY(); i<=(container.getBounds().y); i--)
                {
                    //"getX()-20" car je le centre par rapport au vaisseau.
                    laser.setBounds(vaisseau.getX()-20, vaisseau.getY()+20, 10, 20);
                    try {
                        Thread.sleep(300);
                    } catch (InterruptedException ex) {
 
                    }
                }
            }
        }

Merçi par avance pour votre aide.

02/07/2014, 14h35
joel.drigo

Salut,

Le problème dans ton code est que tu fais une boucle avec temporisation de 300ms dans le thread de SWING (appelé EDT = Event Dispatch Thread). Ce thread est utilisé par le système évènementiel (donc sur le KeyPressed, en l'occurence), mais également pour l'affichage. Un thread ne peut faire qu'une seule chose à la fois : c'est d'ailleurs pour faire plusieurs choses en même temps qu'on utilise des threads. Donc si tu fais une boucle dans l'EDT, tant qu'elle n'est pas terminée, il n'est pas possible d'avoir une mise à jour de l'affichage : comme le but de ta boucle est d'afficher ton laser à plusieurs endroits différents, avec 300ms d'attente entre chaque, l'affichage se met à jour uniquement à la fin, quand le mouvement est terminé, au bout d'un certain temps. Pour faire ce genre de chose, il faut faire le déplacement dans un thread à part, et ne faire que la mise à jour de l'affichage dans le thread graphique : en fait, on change les coordonnées dans le thread à part, et on appelle repaint(), ce qui a pour effet de "réveiller" l'EDT pour qu'il mette à jour l'affichage.
02/07/2014, 15h01
ville_de_lumiere

merçi pour ta réponse,

1) Si je me debarasse du thread.sleep et que je mette un repaint, ça arrangera un peu mon pb??

2) et la boucle for, elle est fausse?? rien ne decremente dans l'histoire??

Merçi.
02/07/2014, 15h09
joel.drigo

Citation:

Envoyé par ville_de_lumiere

1) Si je me debarasse du thread.sleep et que je mette un repaint, ça arrangera un peu mon pb??

Non. Il faut faire la boucle dans un thread autre que l'EDT, sinon ça changera pas grand chose. L'EDT sera toujours bloqué le temps de la boucle (moins longtemps sans les "sleeps", certe), et l'affichage ne sera pas mis à jour durant la boucle. Donc tu ne verras donc toujours que le dernier déplacement.

Citation:

Envoyé par ville_de_lumiere

2) et la boucle for, elle est fausse?? rien ne decremente dans l'histoire??

Je ne sais pas, j'ai pas regardé ton for. Normalement, lorsqu'on itère sur un index avec un pas négatif, on le fait tant que l'index est supérieure à une borne de fin, qui devrait être inférieur à celle de début (sinon on n'entre pas dans la boucle). Est-ce le cas de ta boucle ?
02/07/2014, 18h06
ville_de_lumiere

merçi joel,

Effectivement l'index etait superieur a la borne de fin:

Code:

vaisseau.getY() > container.getBounds.y ==> 0

mais j'ai fais la grossiere erreur de mettre:

Code:

container.getBounds.y<=0:

donc le laser n'aurait pas parcouru tout le long du jpanel, vrai??
Sinon pour le thread je me suis rendu compte de mon erreur du fait de son placement ou de son effacement.

Merçi encore.