Discussion :

[nouknouk]

Bonjour,

Voilà j'ai dérivé un JPanel pour afficher plusieurs images sur lesquelles je veux catcher les clics (cf. le code simplifié en dessous).

Le problème: il arrive plus ou moins régulièrement que certains clics dans mon JPanel ne soient pas 'catchés', ie. processClick() n'est pas appelé.

C'est (très) ennuyeux. Aurais-je fait un truc de travers ?

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
 
public class ImagePanel extends JPanel {
    public ImagePanel () {
        super.addMouseListener(new MouseAdapter() {
            public void mouseClicked(MouseEvent e) {
                processClick(e);
            }
        });
    }
 
    protected void processClick(MouseEvent e) {
        // pour chaque image de la liste, j'appelle getImageRectangle()
        // je compare avec la coordonnée contenue dans 'e' et ainsi 
        // je sais quelle image a été cliquée ; je préviens des listeners.
    }
    public void paintComponent(Graphics g) {
        // ici j'affiche ma liste d'images
        // en calculant leur pos + taille avec getCardRectangle(...)
        for (int pos = 0 ; pos < m_images.size() ; ++pos) {
            Rectangle rect = getImageRectanle(i);
            g.drawImage(img, rect.x, rect.y, rect.width, rect.height, this);
        }
    }
    public Rectangle getImageRectangle(int pos) {
        // je retourne ici la pos + taille de mon image à afficher.
    }
 
    public static void main(String[] argv) {
        CardsPanel cards = new CardsPanel(CardsPanel.Direction.HORIZONTAL, true);
        //etc...
   }
}

[Janitrix]

Pour ce genre de problème, le mieux (je trouve) c'est de mettre des System.out.println("..."); ) chaque étape importante. Donc ça te donne ça :

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
public ImagePanel () {
        super.addMouseListener(new MouseAdapter() {
            public void mouseClicked(MouseEvent e) {
                processClick(e);
System.out.println("state1");
            }
        });
    }
 
    protected void processClick(MouseEvent e) {
System.out.println("state2");
        // pour chaque image de la liste, j'appelle getImageRectangle()
        // je compare avec la coordonnée contenue dans 'e' et ainsi 
        // je sais quelle image a été cliquée ; je préviens des listeners.
    }

Comme ça tu sais où ton évènement est bloqué. Si à chaque clique, il y a state1 écrit dans la console, ton évènement est normal. De plus, essai de voir si le fait d'implémenter toutes les méthodes de MouseAdapter corrige ton problème. Bonne chance.

[nouknouk]

Salut, et merci pour ta réponse.

En fait, le code présenté ci dessus est raccourci pour montrer le principe ; je n'ai pas copié/collé le code complet pour ne pas surcharger mon post.
J'ai effectivement déjà ajouté des println un peu partout, et ça donne que les clics non détectés ne passent ni par mouseClicked() ni par processClick()
Les clics détectés, eux, passent normalement par les deux fonctions.

essaie de voir si le fait d'implémenter toutes les méthodes de MouseAdapter corrige ton problème.

J'ai du mal à bien voir l'intérêt de ce genre d'essai : soit mon adapter est bon et j'arrive au moins une fois à avoir une méthode d'appelée (c mon cas), soit l'adapteur n'est jamais appelé (pas mon cas). Je comprends pas à quoi servirait de surcharger des méthodes vides par d'autres méthodes vides. Je cite la JavaDoc:

The methods in this class are empty. This class exists as convenience for creating listener objects.

[Janitrix]

Tu surchages ces méthodes afin de voir si l'évènement n'est pas "attrapé" dans ces méthodes, ce qui me semble peu probable, mais on sait jamais. Essai d'enlever le super quand tu fais super.addMouseListener(...). Bonne chance.

[nouknouk]

Alors,

- J'ai bien enlevé le 'super' (même si logiquement, ça ne change rien puisque ma classe dérive de JPanel)

- J'ai implémenté les autres méthodes, notamment mousePressed() et MouseReleased().

Cas d'un clic qui 'marche':

Code :

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

1
2
3
mousePressed()
mouseReleased()
mouseClicked()

Cas d'un clic qui 'déconne'

Code :

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

1
2
mousePressed()
mouseReleased()

Donc le problème persiste sachant que quand les clics ne sont pas 'catchés' par mouseClicked(), on catche quand même bien mousePressed() et mouseReleased().

La solution de facilité serait de se brancher sur MousePressed() en lieu et place de mouseClicked(). Certes, dans mon cas c'est à peu près équivalent (quoique), mais je trouve quand même cela anormal. La JavaDoc précise bien pour mouseClicked() :

Invoked when the mouse button has been clicked (pressed and released) on a component.

Donc en toute logique, mouseClicked() devrait être appelé. Bug ou erreur de ma part ?

EDIT: à noter qu'il semble que les clics non 'catchés' se produisent plus fréquemment pendant ou après une salve de clics rapides.

[Janitrix]

Je vais peut être dire une connerie, mais l'invocation de mouseClicked() est peut être réalisée en fonction du temps écoulé entre la pression et le release du bouton : genre si il y eu 3 secondes, entre les deux, il fait rien. Il y a rien dans la javadoc à ce sujet ?