JButton Array with actionListener...trouble printing out result

Question

Basically I've made a array for each letter of the alphabet and then added it to a array of JButtons. That works fine however I've now attempted to add on a action listener which I managed to succesfully get working.

BUT, it works as I have 26 if statements to check if each button is pressed hence why I've tried adding a for loop.

Now whenether I press the button it prints out a load of garbage regarding the JbUTTON properties. Where could I be going wrong?

String[] letters = { "Q", "W", "E", "R", "T", "Y", "U", "I", "O", "P", "A", "S", "D", "F", "G", "H", "J", "K", "L", "Z", "X", "C", "V", "B", "N", "M" };
    layout.add(scrollBar);
     for (int i=0; i < 26; i++)
     {

        if (i==25)
        {
            layout.add(spacebar);
            spacebar.setPreferredSize(new Dimension(310,50));
            spacebar.setBackground(Color.black);
            spacebar.setForeground(Color.white);
            spacebar.addActionListener(new action());
        }

         AlphaButton[i] = new JButton(letters[i]);
         AlphaButton[i].setPreferredSize(new Dimension(50,50));
         AlphaButton[i].setBackground(Color.black);
         AlphaButton[i].setForeground(Color.white);
         layout.add(AlphaButton[i]);
         AlphaButton[i].addActionListener(new action());
     }
    class action implements ActionListener
        {
            public void actionPerformed(ActionEvent e)
            {
                String V = screenArea.getText();

                for (int i=0; i < 26; i++)
                {
                    if( e.getSource() == AlphaButton[i] )
                    {
                        screenArea.setText(V + AlphaButton[i]);
                    }
                }

            }
        }

PM 77-1 · Accepted Answer · 2013-11-11 01:25:47Z

2

If you need to return the letter that was pressed:

screenArea.setText(V + AlphaButton[i].getText);

answered Nov 11, 2013 at 1:25

PM 77-1

13.4k21 gold badges73 silver badges118 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Paul Samsotha Over a year ago

+1 now I think about it, this seems like the more appropriate simplest answer OP is looking for.

tbodt · Accepted Answer · 2013-11-11 01:24:14Z

0

You need to override the AlphaButton toString method

// your class
public class AlphaButton {

    @Override
    public String toString() {
        return "This is an AlphaButton";  // or whatever output you want it to say
    }
}

edited Nov 11, 2013 at 1:24

tbodt

17.1k7 gold badges62 silver badges87 bronze badges

answered Nov 11, 2013 at 1:21

Paul Samsotha

210k39 gold badges503 silver badges756 bronze badges

Comments

Hovercraft Full Of Eels · Accepted Answer · 2013-11-11 02:03:51Z

0

Why not simply do:

public void actionPerformed(ActionEvent e) {
   String V = screenArea.getText();
   screenArea.setText(V + e.getActionCommand());
}

answered Nov 11, 2013 at 2:03

Hovercraft Full Of Eels

286k25 gold badges268 silver badges391 bronze badges

Collectives™ on Stack Overflow

JButton Array with actionListener...trouble printing out result

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related