Discussion :

[schine]

Bonjour,

Je suis un cours de Java sur un site Internet. Dans une partie du cours, on doit animer une boule rouge en appelant la méthode repaint. Cependant l'animation ne se déroule pas et la méthode repaint ne semble pas fonctionner

voir le code ci-dessous

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
 
 
import java.awt.Color; import java.awt.Graphics; import javax.swing.JPanel;
 
public class Panneau extends JPanel { private int posX = -50;
private int posY = -50;
 
public void paintComponent(Graphics g){ g.setColor(Color.red);
 
g.fillOval(posX, posY, 50, 50); }
 
public int getPosX() {
 
return posX; }
 
public void setPosX(int posX) {
 
this.posX = posX; }
 
public int getPosY() {
 
return posY; }
 
public void setPosY(int posY) {
 
this.posY = posY; }
 
}

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
import java.awt.Dimension; import javax.swing.JFrame;
 
public class Fenetre extends JFrame{ private Panneau pan = new Panneau();
 
public Fenetre(){
this.setTitle("Animation");
this.setSize(300, 300); this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); this.setLocationRelativeTo(null); this.setContentPane(pan);
 
this.setVisible(true);
 
go(); }
 
private void go(){
for(int i = -50; i < pan.getWidth(); i++){
 
int x = pan.getPosX(), y = pan.getPosY(); x++;
y++;
pan.setPosX(x);
 
pan.setPosY(y); pan.repaint(); try {
 
Thread.sleep(10);
} catch (InterruptedException e) {
 
e.printStackTrace(); }
 
} }
 
}

si quelqu'un a une explication

merci

[wax78]

Le code fonctionne, la boule se déplace. (Elle laisse une grosse trainée certes mais l'animation fonctionne).

(Bon tu bloques l'EDT certainement avec ta boucle et ton sleep, mais ca n'empêche pas le code de tourner et de faire ce qu'il doit).

Et pour agrémenter ton paintComponents() afin qu'il ne laisse "pas de traces" je te suggère quelque chose du genre :

Code :

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

1
2
3
4
5
6
7
8
 
	public void paintComponent(Graphics g)
	{
		g.setColor(getBackground());
		g.fillRect(0, 0, getWidth(), getHeight());
		g.setColor(Color.red);
		g.fillOval(posX, posY, 50, 50);
	}

[joel.drigo]

Salut,

Même si c'est la mauvais façon de réaliser une animation (exécution de l'animation en dehors de l'Event Dispatch Thread, avec une boucle, entre autres), ce programme fonctionne normalement, si tant est qu'il y ait une méthode main quelque part qui le lance (on ne la voit nulle part dans ce que tu nous montres).

Une classe par exemple comme ça :

Code :

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

1
2
3
4
5
6
7
public class Launcher {
 
        public static void main(String[] args) {
		new Fenetre();
	}
 
}

Aussi, ce serait pas mal que quand tu nous montres du code, qu'il soit correctement indenté, que ça soit plus simple pour ceux qui vont le lire.

[joel.drigo]

(Bon tu bloques l'EDT certainement avec ta boucle et ton sleep, mais ca n'empêche pas le code de tourner et de faire ce qu'il doit).

Non, en fait, justement, parce que la boucle n'est pas exécutée dans l'EDT.

[Népomucène]

Le code fonctionne, la boule se déplace. (Elle laisse une grosse trainée certes mais l'animation fonctionne).

Pour supprimer la trainée, tu peux "mettre un coup de peinture blanche" avant de redessiner la boule :
Classe Panneau:

Code :

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

1
2
3
4
5
6
7
8
9
    public void paintComponent(Graphics g) {
 
        g.setColor(Color.white);
        g.fillRect(0, 0, 300, 300);
 
        g.setColor(Color.red);
 
        g.fillOval(posX, posY, 50, 50);
    }

[wax78]

Avec ton exemple de main je pense que tu as raison Joel.

Mais si j'utilise le template à la eclipse (quand on crée une JFrame) ?

Code :

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

1
2
3
4
5
6
7
8
9
10
public static void main(String args[]){
		EventQueue.invokeLater(new Runnable()
		{
			public void run()
			{
				new Fenetre();
			}
		});
 
}

[joel.drigo]

Pour supprimer la trainée, tu peux "mettre un coup de peinture blanche" avant de redessiner la boule :

Oui, ou tout simplement en laissant Swing faire :

Code :

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

1
2
3
4
5
6
7
8
  public void paintComponent(Graphics g) {
 
        super.paintComponent(g);
 
        g.setColor(Color.red);
 
        g.fillOval(posX, posY, 50, 50);
    }

Avec ton exemple de main je pense que tu as raison Joel.

Mais si j'utilise le template à la eclipse (quand on crée une JFrame) ?

Code :

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

...

Oui, mais alors là, si le tutoriel en question est fait dans cet esprit, il vaut mieux carrément supprimer le lien de ses favoris

[schine]

merci pour le retour rapide.

c'est bizzare chez moi la boulle s'affiche mais ne veut pas bouger elle reste immobile.
la je ne comprends pas

[joel.drigo]

Montre-nous alors tout le programme (la méthode main que tu utilises).

[wax78]

Oui, mais alors là, si le tutoriel en question est fait dans cet esprit, il vaut mieux carrément supprimer le lien de ses favoris

C'est le code que génère Eclipse quand tu lui demande de créer une JFrame avec "l'assistant", je ne l'invente pas ni ne le sort d'un tutorial Mais celui qui ne sait pas alors se retrouve avec l'interface qui fige le temps de la boucle. (Enfin bref je risques d'embrouiller mon monde désolé).

[schine]

voici ma methode main

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
import javax.swing.JFrame;
 
 
 
public class Test {
 
  public static void main(String[] args){       
 
    JFrame fenetre = new JFrame();
 
  }       
 
}

[joel.drigo]

Dans ta méthode main, tu instancie un objet de classe JFrame (la classe de base pour faire une fenêtre Swing), alors que la classe qui fait ton animation s'appelle Fenetre. C'est donc un objet de cette classe que tu dois instancier, pour en observer le comportement spécifique (comme dans celle que j'ai indiquée dans mon message #3).

[schine]

ok merci pour le retour