Должен ли я использовать отдельный ActionListener для каждого подобного или общего действия?

Question

Я создаю клавиатуру, используя кнопки с Java, когда пользователь нажимает кнопку с меткой от A до Z, он устанавливает для текста JTextField значение A или любую другую нажатую кнопку.

У меня есть отдельный класс для каждой кнопки, поэтому для A это public class listenser1 implements ActionListener, B для public class listenser2 implements ActionListener, это хороший способ сделать это?

Также я пытался сделать это в одном классе и использовал операторы if и if else, покупая с использованием

        if(a.getText().equals("A"))
        {
           input1.setText(input.getText() + "A");       
        }
        .
        .
        .

И это не работает, он печатает ABCDEFGHIJKLMNOPQRSTUVWXYZ вместо одной буквы.

Answer 1

Нет, это не самый эффективный способ. Это занимает слишком много времени, чтобы написать. Вместо этого попробуйте это:

import javax.swing.*;
import java.awt.*;
import java.awt.event.ActionListener;

public class Example extends JFrame implements ActionListener {
    private final String[] letters = {"A", "B", "C", "D", "E", "F", "G", "H", "I", "J", "K", "L", "M", "N", "O", "P", "Q", "R", "S", "T", "U", "V", "W", "X", "Y", "Z"};
    private JButton[] buttons = new JButton[26];
    private JTextArea text = new JTextArea();

    public Example() {
        super("Example");
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        for (int i = 0; i < letters.length; i++) {
            buttons[i] = new JButton(letters[i]);
            buttons[i].addActionListener(this);
            add(buttons[i]);
        }
        add(text);
        pack();
        setVisible(true);
    }

    public void actionPerformed(ActionEvent event) {
        text.append(event.getActionCommand());
    }

    public static void main(String[] args) {
        Example ex = new Example();
    }
}

Answer 2

Почему бы вам не использовать один класс, скажем так:

ActionListener commonListener = new ActionListener() {

    public void actionPerformed(ActionEvent e) {
        Object o = e.getSource();
        if (o instanceof JButton) {
            JButton button = (JButton)o;
            String command = (String) button.getActionCommand()();
            input1.setText(command);
        }
    }
};

И вы можете добавить этот слушатель к своим кнопкам как:

buttonA = new JButton();
buttonA.addActionListener(commonListener);
buttonA.setActionCommand("A"); // same for others till Z.

Надеюсь, это может помочь вам в некоторыхспособ.

Answer 3

Я бы предложил создать один экземпляр слушателя для каждого действия.Анонимные внутренние классы можно легко параметризовать, прочитав final local.

void addButton(final char letter) {
    JButton button = new JButton(Character.toString(letter));
    button.addActionListener(new ActionListener() {
        public void actionPerformed(ActionEvent event) {
            input.setText(input.getText() + letter);
        }
    });
    panel.add(button);
}

(я бы посоветовал перейти прямо к Document вместо JtextComponent.getText / setText.)

Answer 4

Создайте один экземпляр ActionListener и добавьте его к каждой кнопке в цикле:

    final JTextField input1;

    ...

    final ActionListener l = new ActionListener() {

        @Override
        public void actionPerformed(ActionEvent e) {
            input1.setText(input1.getText() + ((JButton) e.getSource()).getText());
        }
    };
    for (char c = 'A'; c <= 'Z'; c++) {
        final JButton jButton = new JButton(String.valueOf(c));
        panel.add(jButton);
        jButton.addActionListener(l);
    }

Answer 5

Вы можете написать один параметризованный слушатель:

    private class LetterActionListener implements ActionListener {

        private final char letter;

        public LetterActionListener(char letter) {
            this.letter = letter;
        }

        @Override
        public void actionPerformed(ActionEvent e) {
             input1.setText(input1.getText() + letter);
        }

    }