Discussion :

[michaeljeru]

Voila, j'ai un MouseMotionListener qui est branché sur un panel, et donc je me sers pour "récupérer" les coordonnées dans une grille.
D'un point de vue de propreté du code et de lisibilité, je me demandais ce qui est le mieux :

Méthode 1 : (ce que j'ai actuellement)
Ma classe implémente MouseMotionListener, et je fais :

panelGrille.addMouseMotionListener(this);

et donc je suis obligé d'ajouter les méthodes MouseMoved et MouseDragged, sachant que cette dernière ne me sers pas.
Problème : il faut défiler 2500 lignes de ma classe avant de voir le code de MouseMoved

Méthode 2 :
J'ai bien envie de faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
panelGrille.addMouseMotionListener( 
	new MouseMotionListener(){
		public void mouseMoved(MouseEvent e) {
			// le contenu de ma méthode
		}
 
		public void mouseDragged(MouseEvent arg0) {
			// TODO Auto-generated method stub
		}
	}
 
);

Qu'est ce qui est le plus propre? le plus efficace? perso, je pencherai vers la 2è solution, parce que ça m'éviterai d'avoir ces 2 méthodes qui se baladent tt en bas de mon code.

[®om]

Ta première méthode n'est pas correcte, car tu pourrais appeler directement mouseMoved directement sur ton composant, ce que tu n'as pas le droit.
Ta deuxième méthode est correcte, simplement si tu veux ajouter le même listener à plusieurs composants, tu es embêté.

Voici donc une solution:

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
public class UnComposant extends JComponent {
 
    private final Controller controller = new Controller();
    private JPanel panelGrille;
 
    //...
 
    public void uneMethod() {
        panelGrille.addMouseListener(controller);
    }
 
    private class Controller extends MouseAdapter {
        @Override
        public void mouseMoved(MouseEvent e) {
            //TODO
        }
    }
}

[remika]

tu peux aussi faire

Code :

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

1
2
3
4
5
6
7
8
9
10
 
panelGrille.addMouseMotionListener
(
    new MouseMotionAdapter()
    {
    	public void mouseMoved(MouseEvent e) {
			// le contenu de ma méthode
	}
    }
);

Et faire ton MouseMotionAdapter dans une autre classe (une classe qui en dérive), ça t'évite d'avoir une méthode vide.

(grillé )