Ecouteur sur JButton qui ne fonctionne pas

**Morgan7469** · 26/12/2011, 13h37

Bonjour à tous,
Je viens solliciter votre aide car je me casse la tête sur un problème d'écouteur depuis plusieurs jours.

Mon application est une sorte de jeu de cartes. Elle contient un JPanel (inclus dans un JScrollPane, pour avoir une ScrollBar), qui peut contenir différents boutons représentant des cartes du jeu (qui dérivent de JButton).
En fait, de base, le JPanel n'est pas visible et ne contient aucun bouton. Si la touche F1 est appuyée, mon application ajoute dans le JPanel les cartes du joueur 1, puis l'affiche de la manière suivante :

Code :

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

1
2
3
4
5
6
7
if (e.getKeyCode() == e.VK_F1) {
	this.panelHand.SetCards(this.players.get(0).getHand()); // On définit les cartes du panel comme celles du joueur 1
	this.panelHand.setVisible(true);
	this.panelHand.paintImmediately(getBounds()); // Redessiner le panel (car c'est la méthode paintComponent du panel qui va ajouter les cartes, ce qui est nécessaire avant d'appeler GetVarButtons pour ajouter les écouteurs)
	this.panelHand.validate();
	this.AddActionListener(this.panelHand.GetVarButtons());
}

J'ajoute également des écouteurs sur chaque carte :

Code :

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

1
2
3
4
5
public void AddActionListener(ArrayList<JButton> _buttons) {
	for (JButton button : _buttons) {
		button.addActionListener(this);
	}
}

Mon problème avec ce code est que lorsque je clique sur un bouton représentant une carte, rien ne se passe, comme ci je n'avais pas ajouté d'écouteur à mes boutons...
Par exemple, lorsque je clique sur un bouton, le programme n'affiche rien, malgré ce code :

Code :

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

1
2
3
public void actionPerformed(ActionEvent e) {
	System.out.println("Action");
}

Mon problème peut-il venir du fait que pour mes boutons j'ai créé une classe qui hérite de JButton, et non pas JButton directement ?

Merci d'avance !

**deathness** · 26/12/2011, 15h18

Soit tu ne passes pas dans ton code pour ajouter les écouteurs, soit ta liste de bouton est vide quand tu y passes, soit dans button.addActionListener(this); this représente une classe qui a une méthode actionPerformed vide.

Éventuellement tu réinitialises ou efface peut être les écouteurs après les avoir ajoutés.

**oneagaindoguys** · 26/12/2011, 15h25

Envoyé par Morgan7469

Mon problème peut-il venir du fait que pour mes boutons j'ai créé une classe qui hérite de JButton, et non pas JButton directement ?

Merci d'avance !

Non, cela doit fonctionner sans problème.

Par contre, es-tu déjà certain que

Envoyé par Morgan7469

Code :

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

1
2
3
4
5
public void AddActionListener(ArrayList<JButton> _buttons) {
	for (JButton button : _buttons) {
		button.addActionListener(this);
	}
}

est réellement éxécuté ?
Vérification par une trace

**Morgan7469** · 26/12/2011, 15h27

J'ai ajouté des affichages un peu partout :
- Je passe bien dans ma méthode AddActionListener(ArrayList<JButton> _buttons)
- Dans cette méthode, ma liste n'est pas vide, je boucle bien autant de fois que le joueur a de cartes
- Ma classe représentée par this implémente bien la méthode actionPerformed(ActionEvent e)

Je vais voir si mes écouteurs ne sont pas réinitialisés quelque part...

**oneagaindoguys** · 26/12/2011, 15h27

D'accord avec deathness

Une autre chose (je pense inutile mais on ne sait jamais), la classe en question (this) implémente bien ActionListener ?

**Morgan7469** · 26/12/2011, 15h32

Oui bien sûr !

Code :

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

public class Game extends JFrame implements ActionListener, KeyListener, MouseListener

J'ai d'autres panels dans ma fenêtre, et je n'ai aucun souci avec mes écouteurs concernant les boutons de ces panels.

**oneagaindoguys** · 26/12/2011, 15h36

Est-ce que tu affiches le bouton sur un ojbjet autre qu'un JPanel ou scrollPane ? Il arrive parfois que le bouton soit positionné sur un autre objet que ce soit cet autre objet qui reçoit l'action du clic et non le bouton

**Morgan7469** · 26/12/2011, 15h44

Mes boutons sont bien positionnés sur un JPanel, positionné dans un JScrollPane.
En plus, quand je clique sur le bouton, je le vois s'enfoncer donc j'imagine que c'est bien lui qui reçoit l'action du clic...

**oneagaindoguys** · 26/12/2011, 15h49

S'il s'enfonce, je pense que c'est bon.

Pourrais-tu mettre tout le code de la classe pour voir ?
Il doit y avoir quelque chose qui traine qui entrave l'action

**oneagaindoguys** · 26/12/2011, 15h52

Une fois que tu appuyé sur F1, tu peux afficher :

Code :

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

System.out.printl(_buttons.get(0).getActionListerners()

J'ai pris le 0 au hasard, tu peux aussi faire une boucle

**Morgan7469** · 26/12/2011, 15h58

Le code de PanelHand, mon JScrollPane :

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
static class PanelHand extends JScrollPane {
 
	private Hand currentHand; // La classe Hand contient des objets de type _Card (les cartes de la main du joueur)
 
	public PanelHand() {
 
		super(new PanelCards(), JScrollPane.VERTICAL_SCROLLBAR_NEVER, JScrollPane.HORIZONTAL_SCROLLBAR_AS_NEEDED);
		this.setLayout(new ScrollPaneLayout());
		this.setBorder(BorderFactory.createTitledBorder("Main"));
		this.setVisible(false);
		this.currentHand = null;
	}
 
	public ArrayList<JButton> GetVarButtons() { // Retourne la liste des cartes
 
		ArrayList<JButton> buttons = new ArrayList<JButton>();
		ArrayList<CardComponent> cards = ((PanelCards)this.getViewport().getView()).getCards();
		for (CardComponent card : cards) {
			buttons.add((JButton)card);
		}
		return buttons;
	}
 
	public void AddCardComponent(_Card _card) { // Ajoute une carte
 
                ((PanelCards)this.getViewport().getView()).AddCardComponent(_card);
	}
 
	public void SetCards(Hand _hand) { // Définit la main du joueur
 
		this.currentHand = _hand;
	}
 
	public void paintComponent(Graphics g) { // Dessine les boutons à partir de la main du joueur
 
		super.paintComponent(g);
		((PanelCards)this.getViewport().getView()).RemoveCards();
		if (this.currentHand != null && this.currentHand.getCards() != null) {
			for (_Card card : this.currentHand.getCards()) {
				this.AddCardComponent(card);
			}
		}
	}

Le code de PanelCards, mon JPanel :

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
static class PanelCards extends JPanel {
 
	private ArrayList<CardComponent> cards;
 
	public PanelCards() {
 
		this.setLayout(new FlowLayout());
		this.setVisible(true);
		this.setPreferredSize(new Dimension(2000, 256));
		this.cards = new ArrayList<CardComponent>();
	}
 
	public void AddCardComponent(_Card _card) { // Ajoute une carte au panel
 
		this.cards.add(new CardComponent(this.cards.size() * CARD_SIZE_X, _card));
		this.add(this.cards.get(this.cards.size() - 1), this.cards.size() - 1);
	}
 
	public ArrayList<CardComponent> getCards() { // Retourne les cartes du panel
 
		return cards;
	}
 
	public void setCards(ArrayList<CardComponent> cards) { // Définit les cartes du panel
 
		this.cards = cards;
	}
 
	public void RemoveCards() { // Supprime les cartes du panel
 
		this.cards = new ArrayList<CardComponent>(0);
		this.removeAll();
	}
}

Le code de CardComponent, mon JButton :

Code :

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

1
2
3
4
5
6
7
8
9
10
static class CardComponent extends JButton {
 
	public CardComponent(int _posX, _Card _card) {
 
		this.setLayout(null);
		this.setBounds(_posX, 0, CARD_SIZE_X, CARD_SIZE_Y);
		this.setIcon(new ImageIcon(PATH_IMAGES + _card.getId() + EXTANSION_IMAGES));
		this.setVisible(true);
	}
}

**Morgan7469** · 26/12/2011, 16h02

Envoyé par oneagaindoguys

Une fois que tu appuyé sur F1, tu peux afficher :

Code :

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

System.out.printl(_buttons.get(0).getActionListerners()

J'ai pris le 0 au hasard, tu peux aussi faire une boucle

J'avais déjà effectué cette vérification :

Code :

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

1
2
3
System.out.println(this.panelHand.GetVarButtons().get(0).getActionListeners().toString());
System.out.println(this.panelHand.GetVarButtons().get(0).getActionListeners().length);
System.out.println(this.panelHand.GetVarButtons().get(0).getActionListeners()[0].toString());

Affichage :

[Ljava.awt.event.ActionListener;@2465a484
1
P_Game.Game[frame0,0,0,1024x768,invalid,layout=java.awt.BorderLayout,title=Munchkin,normal,defaultCloseOperation=EXIT_ON_CLOSE,rootPane=javax.swing.JRootPane[,3,25,1018x740,invalid,layout=javax.swing.JRootPane$RootLayout,alignmentX=0.0,alignmentY=0.0,border=,flags=16777673,maximumSize=,minimumSize=,preferredSize=],rootPaneCheckingEnabled=true]

**oneagaindoguys** · 26/12/2011, 16h11

Je ne vois rien d'annormal dans ce code. Du coup, Game ressemble à quoi ?

**Morgan7469** · 26/12/2011, 16h15

C'est une fenêtre avec différents panels...

Lorsque je fais F1, le panel de la main s'affiche, avec les cartes du joueur 1.
Si je fais barre espace les cartes sont supprimées du panel et le panel caché.
Puis je fais F2, le panel de la main s'affiche à nouveau, mais cette fois avec les cartes du joueur 2.

Le seul problème est que rien ne se passe lorsque je clique sur les cartes...

**oneagaindoguys** · 26/12/2011, 16h20

Pour moi il ne reste que le problème d'un listener retiré. Dans Game, tu dois avoir un endroit ou est retiré listener (je pense que tu le fais lorsque tu caches les cartes). Essaye d'enlever toute les instructions remove. Ensuite est ce que les boutons sont accessibles via d'autres classes ?
Autrement si tu places le bouton directement sur la scrollpane ça fait toujours pareil.
Il n'y a pas de mystère, il faut trouver l'instruction qui empèche de faire ce que l'on veut. actionPerformed est correcte

**Morgan7469** · 26/12/2011, 16h31

Oui, j'ai l'impression que les écouteurs disparaissent, mais cela persiste même après avoir enlevé toutes les instructions de remove.

J'ai remarqué que lorsque j'exécute le code suivant :

Code :

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

System.out.println(this.panelHand.GetVarButtons().get(0).getActionListeners().length);

juste après avoir ajouté les écouteurs, toujours dans ma méthode keyPressed, l'écouteur est présent (affichage : 1).

Par contre, si j'exécute le même code en dehors de la méthode keyPressed, l'écouteur n'est pas présent (affichage : 0).

Serait-il possible que cela vienne du fait que le panel où sont positionnées les cartes est redessiné après avoir ajouté les écouteurs ?

**oneagaindoguys** · 26/12/2011, 16h51

Envoyé par Morgan7469

Serait-il possible que cela vienne du fait que le panel où sont positionnées les cartes est redessiné après avoir ajouté les écouteurs ?

Non, et heureusement j'ai envie de dire...

Quelles seraient les raisons pour lesquelles

Code :

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

RemoveCards()

serait utilisé ?

Pour vérifier s'il s'agit bien des mêmes boutons qui sont utilisés, compare les adresses des boutons avant et après. Si l'adresse du bouton 0 dans l'arraylist change, c'est qu'il y a effectivement un reconstruction de l'objet

**Morgan7469** · 26/12/2011, 16h59

Si en fait l'erreur vient de ma méthode paintComponent !

Je force le repaint() lorsqu'on appuye sur la touche F1 pour qu'il dessine les cartes dans le panel, en prenant soin de supprimer les cartes déjà présentes avec RemoveCards() (qui peuvent appartenir à un autre joueur).

Le problème c'est qu'en sortant de la méthode keyPressed, le programme fait un repaint() et supprime/affiche les cartes du panel. Les écouteurs sont donc supprimés !

Je pense que je vais résoudre mon problème en ajoutant les écouteurs dans la méthode paintComponent().

**oneagaindoguys** · 26/12/2011, 17h05

Oui ça pourrait suffire. Mais ne serait-ce pas plus propre de ne pas tout supprimer ?

**Morgan7469** · 26/12/2011, 17h12

J'ai essayé et ça marche !
La méthode paintComponent, qui affiche les cartes, ajoute aussi les écouteurs.

Je ne vois pas ce que tu veux dire par "tout supprimer" ?

En tout cas, je marque le problème comme résolu, et merci à tous pour votre aide !

Ecouteur sur JButton qui ne fonctionne pas

Interfaces Graphiques en Java

Discussions similaires

Partager

Partager