Discussion :

[afrikha]

voilà, j'ai un JPanel avec plein de composants dessus, j'ai crée une classe à part pour gérer les evenements des composants de ce JPanel.Seulement si la methode actionPerformed agit sur d'autres composants je suis obligé de définir ces composants en public static.
je trouve que c'est pas trés propre tout ça, j'imagine qu'il doit y avoir mieux mais quoi?

Merci de votre aide

[bouye]

Pas besoin de declarer tes composants en static : tu passe juste un reference au panel principal (qui si j'ai bien compris contients les autres en membre d'instance protected ou package protected) dans le constructeur de ta classe gerant les evenements.

[afrikha]

non ils étaient en private, donc je mets protected à la place, c'est ça?
sinon c'est une bonne manière de faire? mieux que les classes anonymes?

P.S: grand grand grand merci à toi bouye,tu m'a sorti d'affaire un nombre incalculable de fois

si je suis president un jour,je fais une statue à ta gloire

[bouye]

Wahouu, a part ca je n'ai pas le tete qui enfle ... 8)
Sans compter que j'ai moi-meme ete aide par d'autres membres du forum. Prevoir beaucoup de statues donc.

Oui c'est clair quand le code du controlleur (la gestion des evenement) est trop gros ca peut etre une bonne idee de le mettre dans une classe a part ne serait-ce que pour rendre la classe principale un peu plus lisible. J'imagine que ton controller ne sera pas visible depuis l'exterieur donc c'est une bonne idee de le mettre en package protected (pas de modificateur donc devant le mot-cle class). Penser cependant a rajouter suffisement de doc pour indiquer le lien entre le controlleur et la classe principale.

Quand aux membres de ta classe principale et bien cela depend de si tu comptes l'etendre ou pas (et ces modificateurs-la sont eux-meme majores par le modificateur de la classe bien sur) donc protected ou package-protected celon l'usage.

Attention cependant , au niveau conception, si ta classe est public et extensible (en general dans une API et oriente reutilisabilite donc) ce n'est pas toujours une bonne idee de fixer ce genre de comportements en dur et de rendre les champs accessibles. Mais bon j'imagine que dans ton interface, ce n'est pas le cas present : il s'agit juste du composant primaire de l'application.

[afrikha]

ce que je ne comprends pas c'est qu'au départ j'ai mis mes champs en public , et je faisais appel à eux avec: monChamp.methode() et là le compilateur m'indique :non-static monChamp cannot be referenced from a static context

[bouye]

Probablement car tu avais mis ton listener dans une classe interne/temporaire static.

Normalement il n'y a aucune difficulte a faire ca :

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
 
class Toto extends JPanel {
  JButton trucButton = new JButton("Truc");
  TotoListener totoListener = new TotoListener();
  ...
 
  trucButton.addActionListener(totoListener);
 
  ...
 
  private class TotoListener implements ActionListener {
    /** @inheritDoc
    */
    public void actionPerformed(ActionEvent event) {
      if (event.getSource() == trucButton) { 
        ...
      } 
    }
  }
}

Ce qui apres decoupage deviendra :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
class Toto extends JPanel {
  JButton trucButton = new JButton("Truc");
  TotoListener totoListener = new TotoListener(this);
  ...
 
  trucButton.addActionListener(totoListener);
 
  ...
 
}

Et :

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
 
class TotoListener implements ActionListener {
  private Toto toto;
 
  /** Creates a new instance.
  * @param toto The target toto.
  */
  public TotoListener(Toto toto) {
    this.toto = toto;
  }
 
  /** @inheritDoc
  */
  public void actionPerformed(ActionEvent event) {
    if (event.getSource() == toto.trucButton) { 
      ...
    } 
  }
}

Idem pour des appels de methodes d'instance de la classe Toto ou des methodes de ses membres.