Question

I’m creating a keyboard using buttons with java, when a user clicks on a button labelled from A to Z it will set a JTextField text to A or whatever button they pressed.

I have a seperate class for each button so for A its public class listenser1 implements ActionListener, B its public class listenser2 implements ActionListener is this a good way of doing it?

Also I tried to do do it under one class and used if and if else statements buy using

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

And this doesn’t work, it prints out ABCDEFGHIJKLMNOPQRSTUVWXYZ instead of just the one letter.

Answer 1

No, that is not the most efficient way. That takes way too long to write. Instead, try this:

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();
    }
}