Probléme de repaint()

**guillaume59680** · 20/03/2010, 12h48

Bonjour à tous.

Je suis actuellement entrain de développer un Tetris en java. Pour cela j'ai une boucle principale qui tourne en continue grâce à un Thread. Dans cette boucle j'appelle repaint a différents endroit pour mettre à jour mon affichage.
Cependant, je souhaite ajouté un KeyListener qui me permet de déplacer ma pièce de Tetris à l'écran. Dans la méthode KeyPressed j'ai donc mis un repaint pour repeindre la pièce une fois qu'elle a été déplacée.

Mon problème est le suivant : Lorsque j'appuie sur les touches du clavier, la pièce est bien déplacée (dans ma matrice) mais l'affichage ne suit pas. Le repaint est fait trop tard et ça se voit. Il y à un petit temps de latence entre le moment ou j'appuie sur la touche et le moment ou a pièce déplacée est ré affichée à l'écran.
Comment faire pour supprimer cette attente ?!

Voila mon code :

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
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
 
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JButton;
import javax.swing.JLayeredPane;
import java.awt.Color;
import java.awt.*;
import java.awt.Graphics;
import java.awt.MenuBar;
import java.awt.event.*;
import javax.swing.JTextField;
 
public class Fenetre extends JFrame implements ActionListener, Runnable
{        
    JLayeredPane JLP = new JLayeredPane();
    MenuBar menu = new MenuBar();
    Menu option = new Menu("Options");
    MenuItem item1 = new MenuItem("Nouvelle partie", new MenuShortcut(KeyEvent.VK_N));
    MenuItem item2 = new MenuItem("Quitter", new MenuShortcut(KeyEvent.VK_Q));
    Dessin Jjeu = new Dessin();
 
    Thread actif;
    public boolean I, X = true;
    public int refreshTime;
 
    JButton y = new JButton("azerty");
 
    public Fenetre()
    {
 
        refreshTime = 1000;
 
        setTitle("Tetris");
        setSize(450, 648);
        setLocationRelativeTo(null);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);               
        setVisible(true);
        setResizable(false);
        addKeyListener(new RecepteurClavier());
 
        setMenuBar(menu);
        menu.add(option);
        option.add(item1);
        option.add(item2);
 
        item1.addActionListener(this);
        item2.addActionListener(this);
 
        Jjeu.setBounds(0, 0, 300, 600);
        y.setBounds(300,400, 30, 70);
 
        add(JLP);
        JLP.add(Jjeu);
 
        requestFocus();
    } 
 
    public void start()
    {
        if (actif == null)
        {
            actif = new Thread(this);
            actif.start();
        }
    }
 
    public void run()
    {
 
        int i = 0; 
        Jjeu.grille.P.creerPiece();        
 
       while(X)
       {  
           Jjeu.grille.P.y = Jjeu.grille.P.y + 1;          
           I = Jjeu.grille.bloque_bas();
 
           if(I == true)
           {               
               Jjeu.grille.P.y = Jjeu.grille.P.y - 1;
               if(Jjeu.grille.P.y == -1)
               {
                   System.out.println("Fin de partie");
                   X = false;
               }
               else
               {
                   Jjeu.grille.afficher_Piece();
                   Jjeu.repaint();
                   try 
                    {
                        actif.sleep(refreshTime);
                    }           
                    catch (InterruptedException e) {}
                    Jjeu.grille.P.creerPiece();
                    System.out.println("Tout numero :"+i);
                    i=i+1;
                } 
            }
            else
            {   
                    Jjeu.grille.afficher_Piece();
                    Jjeu.repaint();                      
                    try 
                    {
                        actif.sleep(refreshTime);
                    }           
                    catch (InterruptedException e) {}
                    Jjeu.grille.effacer_Piece();
                    System.out.println("Tout numero : "+i);
                    i=i+1;
 
            }                                  
        }
   }  
 
     public void actionPerformed(ActionEvent e)
     {
         if (e.getSource()== item1) 
         {
             start();
         }
         else if (e.getSource()== item2) 
              {
                  dispose();
              }
     }
 
public class RecepteurClavier implements KeyListener
{
                public void keyPressed(KeyEvent event)
                {
                        int source = event.getKeyCode();
 
                        if(source==KeyEvent.VK_UP)
                        {
                            System.out.println("HAUT !");
                            }
                            else if(source==KeyEvent.VK_DOWN)
                            {
                               System.out.println("BAS !");
                            }
                            else if(source==KeyEvent.VK_RIGHT)
                            {  
 
                                Jjeu.grille.effacer_Piece();
                                Jjeu.grille.P.x = Jjeu.grille.P.x + 1;
                                Jjeu.grille.afficher_Piece();
                                Jjeu.repaint();
                                System.out.println("DROITE !");
                                try 
                                {
                                    actif.sleep(refreshTime);
                                }           
                                catch (InterruptedException e) {}
 
                            }
                            else if(source==KeyEvent.VK_LEFT)
                            {   
 
                                Jjeu.grille.effacer_Piece();
                                Jjeu.grille.P.x = Jjeu.grille.P.x - 1;
                                Jjeu.grille.afficher_Piece();
                                Jjeu.repaint();
                                System.out.println("GAUCHE !");
                                try 
                                {
                                    actif.sleep(refreshTime);
                                }           
                                catch (InterruptedException e) {}                                
                            }
                 }
 
 
                 public void keyReleased(KeyEvent event){}
 
 
                 public void keyTyped(KeyEvent event){}
 
}
 
}

J'ai vu des posts où des personnes ont le même problème et j'ai vu plusieurs fois la méthode paintImmediatly(), mais je n'arrive pas à l'utilisée ... Est-ce la solution à mon problème ?

Merci d'avance pour vos réponses =)

**tchize_** · 20/03/2010, 13h33

Quel est le problème exactement?

1) Il y a un délai entre l'appui de l'a touche et le réaffichage
2) Tu vois la piece disparaitre puis réapparitre?
3) tu vois l'affichage se redessiner progressivement?

Ensuite, que fait ta méthode afficherPiece appelée un peu partout?

**guillaume59680** · 20/03/2010, 13h56

Pour répondre à ta question, il y à un délai entre l'appui de la touche et le ré affichage. Plus exactement, un délai entre l'appui de la touche et le lancement de la fonction repaint(). En effet j'ai mis un affichage dans la fonction paintComponent et celui ci est fait après avoir afficher le "DROITE !" se trouvant dans le KeyListener.

En ce qui concerne la fonction afficherPiece :

Pour notre Tetris nous avons donc une matrice (un simple tableau) et un plus petit tableau, PieceChoisie (qui contient la pièce actuellement en déplacement).
La fonction afficherPiece ne fait qu'une seule chose, elle copie le tableau de la pièce choisie vers notre grande matrice. Elle ne touche donc pas du tout à l'affichage, elle ne fait que modifiée notre matrice qui elle nous sert pour l'affichage.

**tchize_** · 20/03/2010, 14h32

Envoyé par guillaume59680

En effet j'ai mis un affichage dans la fonction paintComponent et celui ci est fait après avoir afficher le "DROITE !" se trouvant dans le KeyListener.

repaint n'est pas immédiat. Cette méthode poste un message dans l'EDT demandant, lorsque ce sera possible, de faire un réaffichage. Si il y a un délai notable pour l'utilisateur, c'est probablement que l'EDT est surchargé. Soit ta méthode paintComponent est lente (ce qui explique le délai), soit tu demande beaucoup trop souvent, dans ton thread, des réaffichage.

**tchize_** · 20/03/2010, 14h34

je viens de voir,

ceci

Code :

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

1
2
3
4
5
                                try 
                                {
                                    actif.sleep(refreshTime);
                                }           
                                catch (InterruptedException e) {}

n'a rien à faire dans ton listener et est probablement ta cause de ralentissement (tu endort le thread EDT!) La méthode sleep est une méthode statique qui endort le thread courant, ca n'endort certainement pas ton thread appelé 'actif'

**guillaume59680** · 20/03/2010, 14h45

Effectivement le try, catch utilisé dans mon KeyListener fait attendre une seconde au programme, tout comme celui se trouvant dans la boucle principale.
En le supprimant le problème disparait .

Merci beaucoup de ton aide, sans toi j'y serais encore. ><

Probléme de repaint()

Agents de placement/Fenêtres Java

Vue hybride

Discussions similaires

Partager

Partager