Discussion :

[ville_de_lumiere]

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 :

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

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.

[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.

[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.

[joel.drigo]

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.

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 ?

[ville_de_lumiere]

merçi joel,

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

Code :

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

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

mais j'ai fais la grossiere erreur de mettre:

Code :

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

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.