Discussion :

[menuge]

Salut à tous,

Voici ma réfexion du moment:
J'ai repris un projet assez complexe réalisé en Swing, je souhaite simplifier un peu le code notemment au niveau des menus.
En effet, l'ancien "propriétaire" du code repérait ses menuItem simplement par leur chiffre. par exemple, menu.getComponent(2)...
Donc pas très explicite.
je souhaite donc mettre ces index dans des variables static.

A votre avis, meiux vaut-il les mettre dans la classe en question ou dans une classe complètement à part qui comporterait toutes les variables static du projet.

J'espère avoir été clair
Merci d'avance

[@ldehan]

J'aurais tendance à dire que ca depends où ces variables seront utilisées. Si c'est juste dans la classe, ca vaut peut-etre pas trop le coup de les externalisée....

par contre, si elles peuvent être amenées a étre utilisées ailleurs, je pense que tu as tout interet à externaliser.

(dans un premier temps, tu peux aussi les mettre dans la classe et les externaliser au besoin plus tard )

[sinok]

Ou sinon une façon simple de les repérer serait de passer par un setActionCommand(String s) sur chaque menu item (ça affecte une chaine au menu) puis tu peux récupérer cet chaine au moment de l'évènement

[Pill_S]

Ou sinon une façon simple de les repérer serait de passer par un setActionCommand(String s) sur chaque menu item (ça affecte une chaine au menu) puis tu peux récupérer cet chaine au moment de l'évènement

Oui, mais je trouve ça pas propre (obligé d'utiliser la méthode equals pour tester la chaîne ActionCommand)

Le mieux est encore de garder une référence sur chacun des composants, sous forme de champs d'instance, et de tester avec ==

Code :

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

1
2
3
4
5
6
7
8
9
 
class C extends JFrame implements ActionListener {
  private JButton b = new JButton();
  ...
  public void actionPerformed(ActionEvent ae) {
    Object source = ae.getSource();
    if(source==b){/*Click sur b*/}
  }
}

Sinon, le top de la propreté, c'est d'utiliser des AbstractAction, qui peuvent être utilisée avec plusieurs composants différents...

Code :

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

1
2
3
4
5
 
Action copy = new AbstractAction(){public void actionPerformed(ActionEvent ae){/*...*/}}
...
myButtonCopy.setAction(copy);
myMenuCopy.setAction(copy);

[mavina]

Sinon, le top de la propreté, c'est d'utiliser des AbstractAction, qui peuvent être utilisée avec plusieurs composants différents...

Code :

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

1
2
3
4
5
 
Action copy = new AbstractAction(){public void actionPerformed(ActionEvent ae){/*...*/}}
...
myButtonCopy.setAction(copy);
myMenuCopy.setAction(copy);

Bonjour,

Pour moi le top du top de la propreté c'est de faire une classe privée par action de bouton, ca permet une lisibilité excellente du code et une separation des evenement et de la classe principale

mavina

[Xavinou]

Encore plus top (enfin à mon avis) ;-)

C'est effectivement d'utiliser des actions swing (comme cela a été dit), mais de les mettre dans une classe différente de celle de la fenêtre (ou du panneau) pour séparer la partie vue et la partie controleur... Ainsi on respecte mieux le MVC.
Et puis c'est bien utile quand la même action est utilisée dans plusieurs vues.

[menuge]

Merci beaucoup pour toutes ces réponses....