Boucle infinie et paintComponent

Version imprimable

Bonjour à tous,

Peut-on m'aider sur une boucle infinie?

je suis sûr que la réponse est devant mes yeux mais etant un novice,je ne trouve pas et ca m'enerve :cry:

j'ai fait une boucle infinie pour faire rebondire une balle quand elle touche les bords de la fenetre( enfantin normalement...)
mais voila, le code ci-dessous n'affiche rien et si je désactive la méthode lancer(), ben l'affichage est correct mais ca ca bouge pas; le probleme est donc la méthode lance()...

Encore une chose, pour les animations, est il plus adéquat d'utiliser un Thread.sleep ou un objet ActionListener? et pourquoi?

Merci d'avance.
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 private void lancer(){ int x,y; boolean invX=false; boolean invY=false; while(true) { x=bal.getPosX(); y=bal.getPosY(); if (x>this.getWidth()-bal.getLargBal()) invX=true; if (x<1) invX=false; if (y>this.getHeight()-bal.getHautBal()) invY=true; if (y<1) invY=false; if (invX==true) { bal.setPosX(x--); } else { bal.setPosX(x++); } if (invY==true) { bal.setPosY(y--); } else { bal.setPosY(y++); } this.repaint(); try { Thread.sleep(10); } catch (InterruptedException e) { // TODO Auto-generated catch block e.printStackTrace(); } } }

22/10/2010, 09h20
dinobogan

Comment est appelée ta méthode "lancer" ? Via l'EDT (sur un listener de composant graphique) ?
Si oui, alors il faut faire un thread dédié pour cette méthode car tu figes l'interface graphique.

J'ai essayé les 2 façons ci-dessous, en appelant lancer() simplement et aussi via un actionlistener.

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 Panneau(){
 
		this.setSize(800, 600);
		rac = new Racket();
		rac.setPosX((this.getWidth()/2)-rac.getLargRaquette()/2);
		rac.setPosY(this.getHeight()-rac.getHauteurRaquette()-rac.vide);	
 
		bal = new Balle();
		bal.setPosX(rac.getPosX()+((rac.getLargRaquette()/2)-(bal.getLargBal()/2)));
		bal.setPosY(this.getHeight()-40-rac.getHauteurRaquette()-bal.getHautBal());	
 
		lancer();
		/*t= new Timer(100,action);
		t.start();
		action = new ActionListener(){
 
			@Override
			public void actionPerformed(ActionEvent arg0) {
				// TODO Auto-generated method stub
 
				lancer();
				repaint();
			}
		 };*/
	}

22/10/2010, 10h18
dinobogan

Tu dois créer un Thread à part, soit toi-même, soit via un Timer pour appeler le code de "lancer".

Manifestement, je n'arrive pas a faire ce que je veux.
J'ai lu des infos sur les threads mais j'ai sans doute raté quelque chose.

J'ai ajouté un thread que je lance avec un Timer mais lancer() ne fonctionne pas.
Cependant, grace a mon thread, maintenant la fenetre apparait mais l'animation ne fonctionne tjs pas...

Je commence a croire que c'est une erreur de logique dans la méthode lancer()...

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 Panneau(){
 
		this.setSize(800, 600);
		this.setBackground(Color.black);
		rac = new Racket();
		rac.setPosX((this.getWidth()/2)-rac.getLargRaquette()/2);
		rac.setPosY(this.getHeight()-rac.getHauteurRaquette()-rac.vide);	
		bal = new Balle();
		bal.setPosX(rac.getPosX()+((rac.getLargRaquette()/2)-(bal.getLargBal()/2)));
		bal.setPosY(this.getHeight()-40-rac.getHauteurRaquette()-bal.getHautBal());	
 
		action = new ActionListener(){
 
			public void actionPerformed(ActionEvent arg0) {
				// TODO Auto-generated method stub
				System.out.println("Juste avant le thread");
				th = new Thread() {
 
					public void run() {	
						System.out.println("Dans le thread");
						lancer();
 
					}
				};
				th.start();	
			}
		 };	
 
		 t = new Timer(5000,action);
		 t.start();
	}

Et voilà le code de la méthode lancer()

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
 
	private void lancer(){
		System.out.println("Dans lancer");
		int x,y;
		boolean invX=false;
		boolean invY=false;
		while(true)
		{
			x=bal.getPosX();
			y=bal.getPosY();
			if (x>this.getWidth()-bal.getLargBal())
				invX=true;
			if (x<1)
				invX=false;
			if (y>this.getHeight()-bal.getHautBal())
				invY=true;
			if (y<1)
				invY=false;
			if (invX==true)
			{
				bal.setPosX(x--);
			}
			else
			{
				bal.setPosX(x++);
			}
			if (invY==true)
				{
					bal.setPosY(y--);
				}
			else 
				{
					bal.setPosY(y++);	
				}
			this.repaint();
		/*	try {
					Thread.sleep(5000);
				} catch (InterruptedException e) {
				// TODO Auto-generated catch block
					e.printStackTrace();
				}*/
		}
 
	}

22/10/2010, 11h22
dinobogan

Tu n'as pas saisie le fonctionnement de Timer.
Tu ne dois pas faire de boucle ni construire un Thread, c'est le Timer qui va s'occuper d'exécuter périodiquement le code.
22/10/2010, 12h21
bigram

Ca marche, merci beaucoup.