Discussion :

[mimievan]

Salut à tous! Voilà mon problème, j'utilise Timer et le JPanel se déplace mais sort de l'écran. Or je ne veux le déplacer qu'à une position bien définie. Voilà 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
 
public void move(){
		Dimension size = this.getPreferredSize();
		setBounds(position_x, position_y, size.width, size.height);
		Timer tm = new Timer(6,new ActionListener() {
            public void actionPerformed(ActionEvent e) {
				int i=0;
                while( i<5){
				position_x++;
				position_y--;
				i++;
				if(i==5){break;}
            }
        }});
        tm.start();
 
 
 
 
	}

[joel.drigo]

Salut,

Tu peux nous montrer un code plus complet, parce que je ne vois rien ici qui bougerait un JPanel dans un Timer. Tout ce que je vois c'est une méthode qui change la position d'un composant (je suppose qu'il s'agit du JPanel) à un endroit désigné par position_x et position_y, et qui lance une action qui toutes les 6 ms fait une boucle inutile pour incrémenter position_x de 5 et décrémenter position_y de 5.


Par ailleus, le if(i==5){break;} est inutile à la fin d'un bloc while(i<5), parce que redondant.

[mimievan]

Merci pour la réponse.
Donc voilà :
/*/Class Essai où on appelle la class ballon/*/

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
 
public class Essai extends JPanel{
	final MonJSlide puissance = new MonJSlide(100, 0, 0, 10, 20);
	final MonJSlide degre = new MonJSlide(180, 0, 0, 15, 30);
	final Ballon ball = new Ballon();
 
	public Essai(){
		JLabel lab_h = new JLabel();
		String h = "D:/Rugby/h.png";
 
		JLabel instruction = new JLabel("Instructions");
		JLabel instruction0 = new JLabel("- Choisissez la puissance de votre tir.");
		JLabel instruction1 = new JLabel("- Choisissez le degre du tir ");
		JLabel instruction2 = new JLabel("- Cliquez sur le ballon pour faire un essai ");
		JLabel instruction3 = new JLabel("- Attention, choisissez bien vos valeurs!!! ");
 
		setLayout(null);
 
		insertImage(lab_h,h);
		Dimension size = lab_h.getPreferredSize();
		lab_h.setBounds(0,0,
        size.width, size.height);
 
 
		size = ball.getPreferredSize();
		ball.setBounds(343,480,
        size.width, size.height);
		ball.tirerDrop(puissance.getIndice(),degre.getIndice());
		ball.setFocusable(true);
 
 
		size = puissance.getPreferredSize();
		puissance.setBounds(830,360,
        size.width, size.height);
 
		size = degre.getPreferredSize();
		degre.setBounds(830,300,
        size.width, size.height);
 
 
		add(ball);
		add(lab_h);
		add(degre);
		add(puissance);
 
 
 
	}
	public void insertImage(JLabel contenant_image, String path_name){
		ImageIcon icon = new ImageIcon(path_name);
		contenant_image.setIcon(icon);
	}
}

La Class Ballon

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
 
public class Ballon extends JPanel{
	int force = 10;
	int degre;
	int position_x = 343;
	int position_y = 480;
 
    public Ballon(){
		JLabel lab_ball = new JLabel();
		String ball = "D:/Rugby/ballon.png";
		ImageIcon icon = new ImageIcon(ball);
		lab_ball.setIcon(icon);
		add(lab_ball);
	}
 
	public void tirerDrop(int force, int degre){
		this.addMouseListener(new EssaiListener(this));
	}
 
	public void move(){
		Dimension size = this.getPreferredSize();
		setBounds(position_x, position_y, size.width, size.height);
		Timer tm = new Timer(6,new ActionListener() {
            public void actionPerformed(ActionEvent e) {
				int i=0;
                while( i<5){
				position_x++;
				position_y--;
				i++;
            }
        }});
        tm.start();	
	}	
}

Et le Listener de la Class Ballon et Essai

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
 
public class EssaiListener implements MouseListener{
	Ballon balle;
 
    public EssaiListener(Ballon ball){
		balle = ball;
		ball = balle;
	}
 
	public void mouseClicked( MouseEvent e )
    {
		balle.move();
 
    }
	public void mousePressed(MouseEvent e){	
	}
	public void mouseReleased(MouseEvent e){
	}
	public void mouseEntered(MouseEvent e){
	}
	public void mouseExited(MouseEvent e){
	}
}

J'espère que tu vois où je veux en venir. Et merci d'avance.

[joel.drigo]

Je comprends mieux maintenant ce que tu cherches à faire. Et ce n'est pas du tout ce que fait ton code.

Premièrement, il semble que tu veuilles lorsqu'on clique sur le ballon, que celui-ci se déplace progressivement (animation). C'est bien avec un Timer qu'il faut faire ça, mais l'action du Timer doit :

1. modifier les coordonnées pour un déplacement : le timer va appeler l'action plusieurs fois, toutes les 6 ms (le paramètre delay). Donc, à chaque appel, on déplace un tout petit peu le ballon, et cumulé, le ballon sera déplacé d'autant de fois que l'action aura été appelée multiplié par la valeur de l'incrément ou décrément des coordonnées. A chaque fois que les variables qui te servent de coordonnées change, il faut les réappliquer, sinon il n'y a aucun raison pour que le composant change de position. Avec ton code, il change de position à chaque clic, mais en plus ça relance un timer qui ne fait que modifier les coordonnées. Les timers s'accumulent (5 clics = 5 timers), qui tous modifient les coordonnées, qui très vite vont sortir de la frame.
2. ensuite, il faut que le ballon s'arrête à un moment donné, donc que le timer s'arrête à un moment donné (ce qui se fait par tm.stop()). Il te faut donc avoir une condition qui arrête le timer : soit par exemple en comparant la nouvelle position obtenue à chaque action, avec des limites (celles du panel). Il faudrait également empêcher de relancer un timer lorsque l'animation est en cours, pour éviter de cumuler les timers (on peut par exemple faire setEnable(false).
3. de la même manière, il ne faut pas ajouter le mouseListener dans une méthode qu'on pourrait appeler plusieurs fois : une seule fois suffit, sinon ça va cumuler les appels (si tu as 2 mouseListners, chacun va appeler move(), donc ça va lancer 2 timers (le ballon se déplacera en théorie 2 fois plus vite : en pratique, ça va forcément pas forcément être le cas pour des raisons que je n'expliquerais pas en détail (voir les notions de synchronisation, volatile, si ça t’intéresse)). En bref, ça va faire n'importe quoi : autant ne pas le faire.

A noter par ailleurs, que tu ne devrais pas charger le fichier image avec un chemin en dur (afin de pouvoir par la suite distribuer ton programme sans forcer ceux à qui tu le donnes d'avoir un lecteur d: avec un dossier Rugby (ce qui sera de toute manière impossible pour qui utilise linux ou un mac).

Exemple :

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
public class Ballpanel extends JPanel {
 
	private static final ImageIcon IMAGE_ICON = new ImageIcon(Ballpanel.class.getResource("ballon.png")); // mettre le fichier dans le même dossier/package que la classe
	private Timer timer;
	private final JLabel ball;
	private final int startPos; 
 
	public Ballpanel(final int posx) { 
		super(true);
		this.startPos = posx;
		setLayout(null);
		ball = new JLabel(IMAGE_ICON);
		ball.setSize(IMAGE_ICON.getIconWidth(), IMAGE_ICON.getIconHeight());
		add(ball);
		// quand on clique sur le ballon, il part
		ball.addMouseListener(new MouseAdapter() {
			@Override
			public void mouseClicked(MouseEvent e) { 
				startMoveBall();
			}
		});
		// lorsqu'on clic dans le panel, on remet le ballon à sa place de départ
		addMouseListener(new MouseAdapter() {
 
			@Override
			public void mouseClicked(MouseEvent e) {
				resetBall();
			}
		});
		// place la ballon en bas du panel
		addComponentListener(new ComponentAdapter() {
			@Override
			public void componentResized(ComponentEvent e) {
				resetBall();
			}
		});
	}
 
	/**
         * Remet la balle à sa place de départ
         */
	public void resetBall() {
		if ( timer==null ) {
			ball.setLocation(startPos, getHeight()-IMAGE_ICON.getIconHeight());
		}
	} 
 
	public void stopBall() {
		if ( timer!=null ) {
			if ( timer!=null ) {
				timer.stop();
				timer=null;
			}
		}
	}
 
	private void startMoveBall() { 
		if ( timer==null ) {
			timer = new Timer(15, e-> moveBall());
			timer.start();
		}
	}
 
	private void moveBall() {
		if ( timer==null ) return; // évite que si une action est demandée alors que le timer a été mis à null, ça plante
		Point location = ball.getLocation(); // position actuelle
		// on fait le déplacement
		location.x+=5;
		if ( (getWidth()-location.x)>(getWidth()-startPos)/2 ) {
			location.y-=3; // bidon (il faudrait un mouvement parabolique)
		}
		else {
			location.y+=3; // bidon (il faudrait un mouvement parabolique)
		}
		ball.setLocation(location); // on applique le déplacement
		if ( location.x>getWidth() ) {
			// lorsque la balle dépasse à droite, on arrête le déplacement
			timer.stop();
			timer=null;
		}
	}
 
	public static void main(String[] args) {
 
		JFrame frame = new JFrame("Exemple");
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
		frame.add(new Ballpanel(30));
 
		frame.setSize(600, 400);
		frame.setLocationRelativeTo(null);
		frame.setVisible(true);
	}
 
}

[mimievan]

Donc je n'ai plus besoin de la class EssaiListener si je veux l'utiliser dans mon projet?

[joel.drigo]

Donc je n'ai plus besoin de la class EssaiListener?

Ça, c'est à toi de voir : moi j'ai utilisé une classe anonyme, parce qu'il n'y avait pas grand chose dans la redéfinition, et pas besoin de réusabilité, mais on peut faire une classe interne, ou externe, si on veut (pour peu qu'on veuille gérer plusieurs ballons pourquoi pas).

[mimievan]

J'ai essayé d'utiliser la class que tu as fait avec quelques modifications et quand j'execute mon programme, il y a une erreur du genre nullpointerexception

[joel.drigo]

Une NullPointerException veut dire que tu accèdes à une variable qui contient null : un problème d'initialisation.

Sinon sans savoir quelles modifications et où se passe l'erreur, que veux-tu que je fasses ?

[mimievan]

En fait je n'ai modifié que le nom de la class et c'est tout. Et aussi je veux afficher ton code sur l'image de mon terrain (qui est un JPanel) juste pour voir. Car ça marche quand j'execute seulement la Class que tu as donnée.

[joel.drigo]

Sur quelle ligne se passe la NullPointerException ?

[mimievan]

La ligne 16 de la class Ballon.

Mais j'ai déjà trouvé ce qui n'allait pas. Merci

Et j'aimerais revenir sur comment utiliser la nouvelle class Ballon dans ma class Essai.

[joel.drigo]

La classe que je t'ai donnée en exemple ne remplace pas ta classe Ballon, ni ta classe Essai. C'est juste un exemple que tu dois adapter. Mais en gros, la classe Ballon remplace ta classe Essai, mais tout ce qu'il y a dans essai et qui ne concerne pas le ballon, c'est à dire les textes pour demander les paramètres, les sliders tout ça, doit être mis à part dans un autre JPanel. Tu ne pourras pas mélanger les deux facilement. Il faut que tu prévois a partie animation d'un côté, et ta partie interaction à côté ou autour, un peu comme si la classe Ballon c'était un écran de télé à l'intérieur de l'autre interface.

Tu peux regarder cet exemple (le 2) : c'est presque ce que tu fais (sauf que c'est un ballon de foot, et qu'on clique pas sur le ballon pour le lancer, mais on le dirige avec des boutons ou le clavier). Tu peux voir que les boutons justement sont autour de la partie où se déroule l'animation. Tu verras également une alternative à ce que tu fais, qui n'utilise pas de JLabel pour l'image. Personnellement je préfère, d'autant plus que ça permet d'enrichir le composant avec d'autres effets graphiques (image dans le fond, textes dessinés ou on veut, autres mobiles, etc...)

[mimievan]

ça y est, je l'ai adapté suivant mes besoins et j'ai obtenu cela

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
 
public class Ballon extends JPanel {
 
	private static final ImageIcon IMAGE_ICON = new ImageIcon(("ballon.png")); // mettre le fichier dans le même dossier/package que la classe
	private Timer timer;
	private final JLabel ball;
	private final int startPosX = 343; 
 
	public Ballon() { 
		super(true);
		setLayout(null);
		ball = new JLabel(IMAGE_ICON);
		ball.setSize(IMAGE_ICON.getIconWidth(), IMAGE_ICON.getIconHeight());
		add(ball);
		// quand on clique sur le ballon, il part
		ball.addMouseListener(new MouseAdapter() {
			@Override
			public void mouseClicked(MouseEvent e) { 
				startMoveBall();
			}
		});
		// place la ballon en bas du panel
		addComponentListener(new ComponentAdapter() {
			@Override
			public void componentResized(ComponentEvent e) {
				resetBall();
			}
		});
	}
 
	/**
         * Remet la balle à sa place de départ
         */
	public void resetBall() {
		if ( timer==null ) {
			Dimension size = ball.getPreferredSize();
			ball.setBounds(343,480,size.width, size.height);
		}
	} 
 
	public void stopBall() {
		if ( timer!=null ) {
			if ( timer!=null ) {
				timer.stop();
				timer=null;
			}
		}
	}
 
	private void startMoveBall() { 
		if ( timer==null ) {
			timer = new Timer(15, e-> moveBall());
			timer.start();
		}
	}
 
	private void moveBall() {
		if ( timer==null ) return; // évite que si une action est demandée alors que le timer a été mis à null, ça plante
		Point location = ball.getLocation(); // position actuelle
		// on fait le déplacement
		location.x+=1;
		if ( (getWidth()-location.x)>(getWidth()-startPosX)/2 ) {
			location.y-=1; // bidon (il faudrait un mouvement parabolique)
		}
		else {
			location.y+=1; // bidon (il faudrait un mouvement parabolique)
		}
		ball.setLocation(location); // on applique le déplacement
		if ( location.x>getWidth() ) {
			// lorsque la balle dépasse à droite, on arrête le déplacement
			timer.stop();
			timer=null;
		}
	}
 
	public static void main(String[] args) {
 
		JFrame frame = new JFrame("Exemple");
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
		frame.add(new Ballon());
 
		frame.setSize(1200, 580);
		frame.setLocationRelativeTo(null);
		frame.setVisible(true);
	}
}

et je l'ai utilisé ainsi mais ça n'affiche rien sur le panneau

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
 
public class Essai extends JPanel{
	final MonJSlide puissance = new MonJSlide(100, 0, 0, 10, 20);
	final MonJSlide degre = new MonJSlide(180, 0, 0, 15, 30);
	//final 
 
	public Essai(){
		Ballon ball = new Ballon(343);
		JLabel lab_h = new JLabel();
		String h = "D:/Rugby/h.png";
 
		JLabel instruction = new JLabel("Instructions");
		JLabel instruction0 = new JLabel("- Choisissez la puissance de votre tir.");
		JLabel instruction1 = new JLabel("- Choisissez le degre du tir ");
		JLabel instruction2 = new JLabel("- Cliquez sur le ballon pour faire un essai ");
		JLabel instruction3 = new JLabel("- Attention, choisissez bien vos valeurs!!! ");
 
		setLayout(null);
 
		insertImage(lab_h,h);
		Dimension size = lab_h.getPreferredSize();
		lab_h.setBounds(0,0,
        size.width, size.height);
 
 
		size = ball.getPreferredSize();
		ball.setBounds(343,480,size.width, size.height);
	//	ball.tirerDrop(puissance.getIndice(),degre.getIndice());
	//	ball.setFocusable(true);
 
 
		size = puissance.getPreferredSize();
		puissance.setBounds(830,360,
        size.width, size.height);
 
		size = degre.getPreferredSize();
		degre.setBounds(830,300,
        size.width, size.height);
 
 
		add(ball);
		add(lab_h);
		add(degre);
		add(puissance);
	}
}

Tu as une idée?

[joel.drigo]

la classe Ballon ce n'est le ballon, c'est la zone dans laquelle le ballon se balade : le ballon c'est le JLabel qui est dedans. Donc il est absurde de faire dans la classe Essai :

Code :

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

1
2
size = ball.getPreferredSize();
		ball.setBounds(343,480,size.width, size.height);

Déjà la taille préferentille du panel ball est 0x0 (puisqu'il n'a pas de layout). Et il est absurde de donner la taillet et la position du ballon à la zone dans laquelle le ballon se trouve. En plus, pourquoi tu t'évertues à donner une taille à un ballon qui ne change jamais de taille. Si tu regardes mon exemple, je ne donne la taille du ballon qu'une fois au début (constructeur de Ballon) par rapport à la taille de l'ImageIcon. Ensuite, je ne change jamais que sa position avec setLocation(). Le panel Ballon doit être mis dans un panel : soit celui-ci est layouté (dans mon exemple, c'est le layout par défaut), soit tu lui donnes une taille qui correspond à la taille de la zone à afficher, qui dépend de la taille de la fenêtre et des différents boutons, textes... (bien sûr la première solution est bien mieux parce que non seulement plus simple (tu n'as pas à faire de calculs) et auto-adaptative. A ce propos, arrête d'utiliser des coordonnées en dur, surtout répétées dans tout le code : tu ne peux pas savoir d'avance la taille de l'écran, et si tes coordonnées sont dans la zone visible ou pas.

[mimievan]

Merci pour ton aide!!! ça a marché et je peux maintenant avancer.