Problem setting ImageIcon onto JLabel

Question

Hi guys I am currently writing a Pente board game.
Now I have two classes called Stone and GameStatus.
In the GameStatus constructor it basically generates a JPanel and I try to fill it with 19 x 19 JLabel and add a mouselistener to each of them. This way when the label gets clicked, I can change the Image of it to represents empty spot, with white stone or with black stone.
Here is the code of Stone class
package pente;
import javax.swing.*;
public class Stone extends JLabel
  // the color of the Stone object
  private String color;
  private int x, y; // the location of current Stone
  boolean occupied; // see if the spot is occupied
  // the location of each image file
  private final String EMPTY = "c:/empty.jpg";
  private final String BLACK = "c:/black.jpg";
  private final String WHITE = "c:/white.jpg";
  // private JLabel label = new JLabel();
  * The default constructor that initializes a
  * Stone object with a given color.
  public Stone(int x, int y, boolean o, String c)
      this.x = x;
      this.y = y;
      this.occupied = o;
      this.color = "empty"; // the color has not yet been set
      this.setIcon(new ImageIcon(EMPTY));
    // assign the given color to the Stone
   * This method sets the image of the current object depends on
   * the given variable c.
   * @return the Stone object with an appropriate image on it
  public Stone setColor(String c)
      // swtich the image of the Panel depends on the c given
      if ((c.equals("w")) || (c.equals("white")))
        this.color = "white";
        this.setIcon(new ImageIcon(WHITE));
      else if ((c.equals("b")) || (c.equals("black")))
          this.color = "black";
          this.setIcon(new ImageIcon(BLACK));
      return this;
   * This method replaces the current image with the empty image
   * @return the Stone object with empty image on it
  public Stone setEmpty()
      this.color = "empty";
      this.setIcon(new ImageIcon(EMPTY));
      return this;
   * This method gets the x coordinate of Stone
   * @return the x coordinate
  public int getX()
      return this.x;
   * This method gets the y coordinate of Stone
   * @return the y coordinate
  public int getY()
      return this.y;
   * This method tests if the current Stone has been set to a color
   * @return if the Stone has been set to a color
  public boolean isOccupied()
      return (!(this.color).equals("empty"));
}Here is the part where GameStatus calls Stone class
public GameStatus(int size)
        this.setTitle("PENTE");
        this.setLocation(100,30);
        this.setSize(747,695);
        this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        JPanel p = new JPanel();
        p.setLayout(new GridLayout(0, size, 0, 0));
        // initializes the array and starts inserting labels
        for (int i = 0; i < size ; i++)
            for (int j = 0; j < size ; j++)
                array[i][j] = new infoArray(i, j, false, EMPTY);
                Stone s = new Stone(i, j, false, EMPTY);
                s.addMouseListener(new MousePressListener());
                p.add(s);
        this.setResizable(false);
        this.getContentPane().add(p);
        this.setVisible(true);
        System.out.print("hello");
    }The problem is, there is only one Stone object can be shown on the JPanel.
This is weird since it works fine when I tried to call JLabel object instead of Stone object and fill them into JPanel.
Since Stone class extends JLabel, I just dont see why JLabel works while Stone doesn't.
I need to know the reason badly, so I appreciate any suggestions.
Thanks.

Answer 1

Code looks reasonable. Only comment I have is that icons can be shared by multiple components. So you only need to create 3 icons, one for each state of the stone. Then you just use setIcon to change the state. No need to recreate the icon every time.