Problème de rafraichissement d'un dessin

Version imprimable

Bonjour à tous,
je développe actuellement un petit logiciel ou j'ai des objets runnable qui ont une position définie dans leur classe.
Ces objets sont dessinés dans un JPanel en récupérant le graphics.
Dans ma méthode run de l'objet, j'efface le dernier point de l'objet avant de dessiner le nouveau point.
Seul hic, des fois le précédent point est mal effacé et reste malgré le rafraichissement.
Honnetement, je vois pas trop d'où cela peut venir: une idée?
Pour illustrer cela, voici le bout de code de ma méthode run() de ma classe:
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 public void run() { while(true){ //Beginning of the process //First part of Display Management Graphics g=this.getJp().getGraphics(); Color c = g.getColor(); g.drawOval(this.getX(),this.getY(),1,1); //Move management int dx,dy; dx=(int) (Math.round((this.getSpeed()*Math.random()-this.getSpeed()*Math.random()))); dy=(int) (Math.round((this.getSpeed()*Math.random()-this.getSpeed()*Math.random()))); this.setX(this.getX()+dx); this.setY(this.getY()+dy); System.out.println("Id: "+this.getId()+" Ship: "+this.getName()+" X: "+this.getX()+" Y: "+this.getY()); //Second part of Display Management g.setColor(Color.red); g.drawOval(this.getX(),this.getY(),1,1); g.setColor(c); //End of the process try { //Wait 1 second Thread.sleep(1000); } catch (InterruptedException e) { // TODO Auto-generated catch block e.printStackTrace(); } } }
Merci pour votre aide.

Essaie :

Code:

1
2
3
4
5
6
7
8
			Graphics g=this.getJp().getGraphics();
			// remplir le graphics avec la couleur de fond
			Color c = g.getColor();
			g.setColor(couleurDeFond);
			g.fillRect(0, 0, largeur, hauteur);
 
			g.setColor(c);
			g.drawOval(this.getX(),this.getY(),1,1);

Non, le phénomène est pire encore je trouve.

J'ai l'impression que cela arrive moins souvent avec un seul objet qu'avec plusieurs

Deux,trois choses à préciser sur le 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
36
37
38
39
 
public void run() {
		while(true){
			//Beginning of the process
			//First part of Display Management
                                      //Je récupère les graphics de mon JPanel
			Graphics g=this.getJp().getGraphics();
                                      //Je récupère la couleur de fond de mon JPanel
			Color c = g.getColor();
                                      //J'efface l'ancien point
			g.drawOval(this.getX(),this.getY(),1,1);
 
 
			//Move management
			int dx,dy;
			dx=(int) (Math.round((this.getSpeed()*Math.random()-this.getSpeed()*Math.random())));
			dy=(int) (Math.round((this.getSpeed()*Math.random()-this.getSpeed()*Math.random())));
			this.setX(this.getX()+dx);
			this.setY(this.getY()+dy);
			System.out.println("Id: "+this.getId()+" Ship: "+this.getName()+" X: "+this.getX()+" Y: "+this.getY());
 
			//Second part of Display Management
                                      //Je change la couleur de dessin
			g.setColor(Color.red);
                                      //Je dessine en rouge
			g.drawOval(this.getX(),this.getY(),1,1);
                                      //Je remets la couleur initiale 
			g.setColor(c);
 
			//End of the process
			try {
				//Wait 1 second
				Thread.sleep(1000);
			} catch (InterruptedException e) {
				// TODO Auto-generated catch block
				e.printStackTrace();
			}
		}
	}

01/05/2008, 23h06
herve91

Ah c'est plus clair !
Des pistes :
:arrow: le code de la méthode run() doit être appelé depuis la méthode paintComponent() de ton panel
:arrow: enlever le Thread.sleep(1000)
:arrow: gérer le rafraichissement via un timer javax.swing.Timer qui effectuera un repaint() du panel toutes les secondes
01/05/2008, 23h17
glloq59

Ok, je vais tester les trois demain:
-le run() de mon objet runnable est actuellement appelé lors d'un clic sur un bouton. Avec ma maigre expérience, je vois pas comment goupiller l'appel avec le paintComponent() de mon jPanel?
-si j'enleve le thread.sleep(1000), ma cadence va augmenter sans résoudre le problème et peut être faire ramer le pc non?
-ta solution du timer me parait la plus adapté. Il faut encore que je trouve comment comment synchroniser mes threads à un timer qui affiche tous les points de mon JPanel en même temps. A moins de lister les données de tous mes objets runnable/thread() dans un tableau et de procéder à une boucle de lecture et d'affichage non?

Tout çà pour afficher des points qui bougent...

En gros...

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
interface Paintable {
    void paint(Graphics g);
}
 
class PaintablePoint implements Paintable {
 
   Color fg;
   Color bg;
 
   public PaintablePoint(Color fg, Color bg) {
      this.fg = fg;
      this.bg = bg;
   }
 
   public void paint(Graphics g) {
	Color c = g.getColor();
 
	g.setColor(bg);
	g.drawOval(this.getX(),this.getY(),1,1);
 
	int dx,dy;
	dx=(int) (Math.round((this.getSpeed()*Math.random()-this.getSpeed()*Math.random())));
	dy=(int) (Math.round((this.getSpeed()*Math.random()-this.getSpeed()*Math.random())));
	this.setX(this.getX()+dx);
	this.setY(this.getY()+dy);
 
	g.setColor(fg);
	g.drawOval(this.getX(),this.getY(),1,1);
 
	g.setColor(c);
   }
 
   ...
}
 
 
class MyJPanel extends JPanel {
 
  Collection paintables;
 
  public void setPaintables(Collection paintables) {
      this.paintables = paintables;
  }
 
  public void paintComponent(Graphics g) {
      super.paintComponent(g);
      for (Iterator it = paintables.iterator(); it.hasNext();) {
         ((Paintable) it.next()).paint(g);
      }
  }
}
 
// quelque part...
Timer timer = new Timer(1000, new ActionListener() {
   public void actionPerformed(ActionEvent e) {
      myJPanel.repaint();
   }
});
timer.start();

09/06/2008, 11h40
glloq59

Problème résolu!
Ca m'apprendra à développer endormi...
Donc le code fourni à la base est bon, j'ai plus qu'à recopier cent fois: "Je ne lancerai pas un thread avec un start PUIS un run..."
Forcément, j'ai regardé mon code hier et quand j'ai vu cela...

Du coup, mon thread affichait un point puis un autre avant d'effacer deux fois la même chose en fonction de la vitesse d'éxécution du thread.

Merci pour l'aide
09/06/2008, 12h38
sinok

Quand bien même....

Pour faire du dessin sur un composant swing la seule méthode sécurisée (pour éviter de perdre le dessin et ainsi de suite) est de redéfinir la méthode paintComponent pour qu'elle effectue le dessin. Les threads ne sont là que pour réaliser les calculs sur lesqule se basera la méthode paintComponent pour réaliser le dessin, puis déclencher le reapint du component.