Question

我目前正在开发一个Java类，它生成一个简单的Tic-Tac-Toe的JFrame / JButton布局。实现ActionListener，我打算让选定的JButton将其标题设置为“X”或“O”（基于是否是X轮流选择JButton的布尔语句）并被禁用（因此无法播放以下转弯的顶部）。我创建的当前应用程序执行此操作，但有时不会更改JButton文本或禁用按钮，直到我单击另一个按钮。当我点击其中一个JButton时，似乎没有任何一种有凝聚力的顺序。我花了好几个小时试图修复这个问题但没有用。我是如何对actionPerformed方法进行编码或者如何将其添加到我的JButtons中的？

以下是我班级的代码：

import javax.swing.*;
import java.awt.event.*;
import javax.swing.*;

public class TTT extends JFrame implements ActionListener{

  // private fields
  private JButton[] buttonArray;
  private JLabel prompt;
  private boolean turnX;
  private String letter;



 public TTT() {
    // Instantiates JFrame window and adds lines to board
    super.setSize(235, 280);
    super.setTitle("Tic-Tac-Toe");
    // Instantiates JButton array
    buttonArray = new JButton[9];
    // Loop that creates the JButton squares
    for(int y = 30; y <= 140; y += 55) {
      for(int x = 30; x <= 140; x += 55) {
        for(int index = 0; index < buttonArray.length; index++) {
        buttonArray[index] = new JButton();
        buttonArray[index].setSize(50, 50);
        buttonArray[index].setLocation(x, y);
        buttonArray[index].addActionListener(this);
        super.add(buttonArray[index]);
      }  
    }
    }
    prompt = new javax.swing.JLabel("X's TURN");
    prompt.setVerticalAlignment(JLabel.BOTTOM);
    super.add(prompt);

    turnX = true;

    super.setVisible(true);
  } 

 public void actionPerformed(java.awt.event.ActionEvent a) {

        // Calculate whose turn it is
        if(turnX){
            letter = "X";
            prompt.setText("O's TURN");
            turnX = false;    
        } else if(!turnX){
            letter = "O";
            prompt.setText("X's TURN");
            turnX = true;
        }
        JButton pressedButton = (JButton)a.getSource(); 
        pressedButton.setText(letter);
        pressedButton.setEnabled(false);
        super.repaint();
 }

  public static void main(String[] args) {
    new TTT();
  } 
}

Answer 1

使用此代码：

for(int y = 30; y <= 140; y += 55) {
  for(int x = 30; x <= 140; x += 55) {
    for(int index = 0; index < buttonArray.length; index++) {

您正在添加82（！）JButtons（一个在9个组中，另一个在另一个上面）。所以，实际发生的是一个被更改（禁用等），但是在调用重绘时，绘制它们的顺序可能会改变，因此触发ActionListener的那个被绘制在仍然启用且空白的8个中的一个之下。

你可以这样纠正：

...
int index = 0;   // <-- Add this line
for(int y = 30; y <= 140; y += 55) {
    for(int x = 30; x <= 140; x += 55) {
           // <-- remove the third for-loop
        buttonArray[index] = new JButton();
        buttonArray[index].setSize(50, 50);
        buttonArray[index].setLocation(x, y);
        buttonArray[index].addActionListener(this);
        super.add(buttonArray[index]);
        index++;   // <-- increment 'index'
    }  
}
...

您也可以删除super.repaint()行，因为它是多余的。

与您的问题没有直接关系，但Swing相关内容应该（出于各种原因）从 Event Dispatching Thread (EDT) 调用。实现此目的的一种方法是使用 SwingUtilities.invokeLater() ，因此用这个替换new TTT();可能是个好主意：

SwingUtilities.invokeLater(new Runnable() {
    public void run() {
        new TTT();
    }
});

ActionListener：禁用按钮

1 个答案: