按下记忆游戏时JButton不会更改图标

Question

我正在尝试制作自己的简单记忆游戏，并且我有一个 JPanel，它实现了 ActionListener 接口并包含一些从 JButton 继承的 Tiles。几乎所有东西都可以正常工作，除了作为第二个按钮被点击的事实之外，它并没有像它应该做的那样改变它的图标。但是，当我单击两个匹配的图块时，它们仍然翻转并且没问题。当我单击第一个按钮时，它会翻转，并且第二个按下的图块如果它不匹配，则其行为类似于未添加 ActionListener 的按钮。我想过用 PropertyChangeListener 解决这个问题，但不知道如何解决。

这是我的 JPanel 中的方法：

@Override
    public void actionPerformed(ActionEvent e){

        Tile tile = (Tile)e.getSource();

        if(tilesFlipped < 2 ){
            tile.flip();
            flippedTiles[tilesFlipped++] = tile;

            if(tilesFlipped == 1){
                return;
            }

            if(flippedTiles[0].equals(flippedTiles[1])){
                tilesFlipped = 0;
                return;

            } else {
                try {
                    Thread.sleep(1000);
                    for(int i = 0; i < 2; i++){
                        flippedTiles[i].flipBack();
                    }
                } catch (InterruptedException e1) {
                }   
            }
            tilesFlipped = 0;
        } 

    }

这是我的 Tile 课程：

private class Tile extends JButton {
    String photoPath;
    ImageIcon photo;

    Tile(int i){
        photoPath = String.format("/home/stn/Desktop/p/9-%d.jpg", i);
        setPreferredSize(new Dimension(150, 150));
        this.setBackground(Color.cyan);
        this.photo = new ImageIcon(photoPath);
    }

    public void flip(){
        this.setIcon(photo);
        this.setBackground(Color.white);
    }

    public void flipBack(){
        this.setBackground(Color.cyan);
        this.setIcon(null);

    }

    public String getPhotoPath(){
        return photoPath;
    }

    @Override
    public boolean equals(Object o){
        return o instanceof Tile && ((Tile)o).getPhotoPath().equals(photoPath);
    }
    @Override
    public int hashCode(){
        return photoPath.hashCode();
    }

}

Answer 1

使用Swing Timer 代替Thread.sleep(...)。了解Thread.sleep(...) 使当前线程进入睡眠状态，这里是 Swing Event Dispatch Thread 或 EDT，这将使您的整个 GUI 进入睡眠状态，冻结您的应用程序。 Swing Timer 将执行一个或重复的操作，而不会占用 EDT。例如：

 public void actionPerformed(ActionEvent e) {

    Tile tile = (Tile) e.getSource();

    if (tilesFlipped < 2) {
       tile.flip();
       flippedTiles[tilesFlipped++] = tile;

       if (tilesFlipped == 1) {
          return;
       }

       if (flippedTiles[0].equals(flippedTiles[1])) {
          tilesFlipped = 0;
          return;

       } else {
          Timer timer = new Timer(1000, new ActionListener() {

             @Override
             public void actionPerformed(ActionEvent e) {
                for (int i = 0; i < 2; i++) {
                   flippedTiles[i].flipBack();
                }
                tilesFlipped = 0;
             }
          });
          timer.setRepeats(false);
          timer.start();
       }
    }

 }