Discussion :

[Morgan7469]

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]

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.

[Morgan7469]

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]

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

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

[oneagaindoguys]

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]

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]

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]

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...