Java anonyme de la classe qui implémente ActionListener?

J'ai récemment fait une programmation de la mission qui nous a obligé à mettre en œuvre dans le code d'un programme spécifié par un diagramme UML. À un moment donné, le schéma spécifié que je devais créer un anonyme JButton qui affiche un nombre (à partir de) et décrémenté à chaque fois qu'il a cliqué. Le JButton et ses ActionListener a dû être anonyme.

Je suis venu avec la solution suivante:

public static void main(String[] args) {
  JFrame f = new JFrame("frame");
  f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
  f.setSize(400, 400);

  f.getContentPane().add(new JButton() {

    public int counter;

    {
      this.counter = 1;
      this.setBackground(Color.ORANGE);
      this.setText(this.counter + "");

      this.addActionListener(new ActionListener() {
        public void actionPerformed(ActionEvent arg0) {
          counter --;
          setText(counter + "");
        }
      });

    }
  });

  f.setVisible(true);

}

Cela ajoute un anonyme, un JButton, puis ajoute un autre (intérieure) anonyme ActionListener pour gérer des événements et de mettre à jour le texte du bouton, tant que nécessaire. Est-il une meilleure solution? Je suis assez sûr que je ne peux pas déclarer un anonyme JButton implements ActionListener (), mais est-il un autre moyen plus élégant pour atteindre le même résultat?

Regardez où vous êtes maintenant @Tim

InformationsquelleAutor Tim | 2009-05-21

anonymous-class java

13

J'ai l'habitude d'aller quelque chose comme ceci:
```
JPanel panel = new JPanel();
panel.add(new JButton(new AbstractAction("name of button") {
    public void actionPerformed(ActionEvent e) {
        //do stuff here
    }
}));
```
AbstractAction implémente ActionListener ce qui devrait satisfaire à la tâche.

Il peut être une mauvaise pratique de l'écraser si le nombre de lignes de code, mais si vous êtes habitué à la lecture, alors il peut être très élégant.
- Ce n'est pas réellement vous permettre de vous changer les boutons de texte à partir de l'intérieur de l'action de l'auditeur... C'est un assez maladroit exigence et je ne peux pas penser à une façon d'améliorer ce que vous avez déjà.
- Donc, la déclaration publique de compteur à l'intérieur de l'anonyme JButton et sans réserve l'accès de l'intérieur de l'ActionListener sont OK?
- Yup, c'est la seule façon vous serez en mesure d'obtenir une "poignée" sur ces champs. Rappelez-vous juste que les champs dans l'intérieur de la classe permet de masquer les champs du même nom, a déclaré à l'extérieur de la portée. Nous espérons que votre compilateur/IDE va vous avertir explicitement.
InformationsquelleAutor Cogsy
4

C'est assez moche, mais vous pouvez effectuer les opérations suivantes à l'aide de l'ActionListener méthode et une classe anonyme:
```
  f.getContentPane().add(new JButton(new AbstractAction("name of button") {
      private int counter = 0;

      public void actionPerformed(ActionEvent e) {
          ((JButton) e.getSource()).setText(Integer.toString(counter--));
      }
  }) {
      {
          setText("1");
      }
  });
```
Pour le rendre plus facile l'accès à la contre vous pouvez déplacer vers le haut niveau de votre classe et de l'accès à partir de deux endroits où setText est appelé.
- +1. Il pourrait être "moche" (votre mot, pas le mien), mais je trouve ça plus facile à comprendre que le code fourni dans la question. Je suppose que la seule question serait de savoir s'il répond aux critères que " Le JButton et ses ActionListener a dû être anonyme.' AbstractAction implémente ActionListener, mais l'instructeur peut-être que les critères n'étaient pas respectés.
InformationsquelleAutor Mike Houston
2

La mise en œuvre de plusieurs types est généralement une mauvaise idée.

Il est rarement nécessaire d'étendre JComponent classes, bien que beaucoup de mauvais logiciels et tutoriels de le faire. Un idiome/hack qui a été gagne du terrain récemment est Double Croisillon - une classe est uniquement sous-classes afin de donner un exemple initialiser qui agit comme un with déclaration à partir d'autres langues.

Dans ce cas, le code peut être écrit comme:
```
JButton button = new JButton();
button.addActionListener(new ActionListener() {
    int counter = 1;
    {
        updateText();
    }
    public void actionPerformed(ActionEvent arg0) {
        --counter;
        updateText();
    }
    private void updateText()
        setText(Integer.toString(counter));
    }
});
f.getContentPane(button);
```
Si elle devient de plus en plus complexe, alors vous aurez probablement envie de faire un extérieur de la classe (ce qui ne veut pas mettre en œuvre ActionListener ou de prolonger JButton) pour traiter les données.

Noter également, vous devriez être en utilisant la EventQueue.invokeLater passe-partout pour s'assurer que les composants Swing ne sont jamais utilisés sur l'AWT (HAE).
- updateText ne fonctionne pas avec juste setText - je pense que vous avez besoin de bouton.setText, ou de nid de l'auditeur à l'intérieur d'un nouveau anonyme sous-classe JButton. Aussi, m.f.getContentPane ne prend aucun argument, je pense que tu veux dire.f.getContentPane.add(bouton).
- +1 pour le fait de ne pas étendre JButton pour config-raisons
InformationsquelleAutor Tom Hawtin - tackline
1

Je ne voudrais pas faire quelque chose comme ça dans le monde réel programme, mais étant donné les exigences de votre affectation, vous pouvez difficilement faire mieux.

InformationsquelleAutor Erich Kitzmueller
1

Bien il existe une solution beaucoup plus élégante façon de le faire.

Malheureusement, c'est pas un Core Java/Swing approche.

Vous pouvez utiliser SwingBuilder en Groovy, pour obtenir le même résultat, en utilisant un peu plus laconique de la syntaxe, par exemple, pseudo code:
```
button(text: '' + counter,
         actionPerformed: {counter--; text = '' + counter + ''},
         constraints:BL.SOUTH)
```
[http://groovy.codehaus.org/Swing+Builder%5D%5B1%5D

Je ne voudrais pas l'utiliser dans vos devoirs bien, j'ai vu des étudiants vraiment s'écartent de la norme et obtenir marquée vers le bas pour elle, mais au moins, vous pouvez l'inclure comme une avenue possible de poursuivre l'enquête.

Je pense que ce que vous avez à l'heure actuelle est tout à fait bien si.

InformationsquelleAutor Jon
0

C'est l'une des mauvaises pratiques tâches forcé de le faire dans les devoirs seulement 😉 Mauvaises choses:
- utilisation de ActionListener au lieu de l'Action qui est mauvais en lui-même
- en conséquence, la portée des problèmes de la bulle jusqu'
  - du compteur champ d'application plus large que nécessaire
  - besoin d'accéder à bouton à l'intérieur de la actionPerformed (par type de fonte ou de l'accès entourant l'objet de l'api)
- illisible (aka: désuète) code
Mais, alors .. on ne peut pas résister, pouvons-nous 😉 Voici une version à l'aide de l'Action qui leur est propre (ou du moins je le pense) que pour les deux premières question, lisible comme tous les autres exemples (et j'ai triché, bien sûr: d'abord mis en œuvre anonyme, les classes, puis laissez-les IDE ne le inline
```
    f.add(new JButton(new AbstractAction() {

        int counter = 1;
        { //constructor block of action
            updateName();
        }

        @Override
        public void actionPerformed(ActionEvent e) {
            counter--;
            updateName();
        }

        private void updateName() {
            putValue(Action.NAME, "" + counter);
        }

    })  { //subclass button 
          {  //constructor block button
            setBackground(Color.PINK);
        }}
    );
```
InformationsquelleAutor kleopatra

Vous devez vous connecter pour publier un commentaire.