9x9 array of buttons and not a single button worked

Question

im trying to build a 9 x 9 block composed of buttons. I used array of buttons.
here's my code with my explanation and poor logic thinking details.
public class ButtonArrayDemo extends JFrame implements ActionListener
private JButton[] sButton;
private int[] x;//manipulation purposes only
public ButtonArrayDemo()
     JPanel buttonPanel = new JPanel();
     buttonPanel.setLayout( new GridLayout(9, 9) );
     sButton = new JButton[81];
     x = new int[81];
// creates the buttons,action listeners, syntax is correct but
// i doubt the logic
     for (int i = 0; i < sButton.length ; i++)
          JButton button = new JButton();
          button.setText(" ");
          button.addActionListener(this);
          sButton[i] = button;
          buttonPanel.add(sButton);
          x[i] = 0;
     getContentPane().add(buttonPanel,BorderLayout.SOUTH);
     setResizable(false);
// what i want for this is that, when a button is clicked, it displays the
//number 1 then increments by 1 if clicked again. Like,when i click a
//blank button, it displays the number "1", if i clicked it again, it displays
//the number 2. This goes on until number 9. When the button is clicked
//on the 10th time, it leaves the button's text blank. Syntax is fine but
//i doubt the logic.
public void actionPerformed(ActionEvent e)
     int y;
     for(y = 0; y < sButton.length; y++)
          if(e.getActionCommand().equals(sButton[y]))
               sButton[y].setText(Integer.toString(++x[y]));
               if(x[y] == 10)
               sButton[y].setText(" ");
               x[y] = 0;
Now, i created the main and compiled the program. The frames appeared complete with 81 buttons. But, when I click on any button,
it does nothing. The button's text which is supposedly to change is not displaying anything. this is a vital thinking process for me and any help from someone would be very much appreciated.
thanks for reading and i hope someone will respond to my problem.
lots of thanks.

Answer 1

I believe the problem is that you are using the actionCommand but you did not set the actioncommand for the buttons. I suggest you either set the action command for each button to a unique item or use getSource instead.
Also, you might want to break out of your for loop once you find the correct button.
Also, if the future use the code formatting tags and this probably should have been in the Swing forum.