几年前，我制作了一个GUI TicTacToe游戏，由于我现在拥有更多的编程技能，所以想重做它。我能够将代码从600行缩减到150行左右。
当我使用相同的方案时，遇到了一些我无法解决的问题，所以请帮助我。
该程序包括两个类，主要类TTTMain：
public class TTTMain {
public static void main(String[] args) {
TTTFrame tttf = new TTTFrame(0,0);
/*Tic Tac Toe Field:
* 0 1 2
* 3 4 5
* 6 7 8
*/
}}
和
TTTFrame：import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
public class TTTFrame extends JFrame implements ActionListener {
private Button[] btnPlayButton;
private Button btnRestart;
private int buttonCounter;
private int xScore;
private int oScore;
private Label Olabel, Xlabel;
TTTFrame(int xScore, int oScore) {
this.xScore = xScore;
this.oScore = oScore;
btnPlayButton = new Button[9];
for (int i = 0; i < 9; i++) {
btnPlayButton[i] = new Button("" + i);
btnPlayButton[i].setBackground(Color.white);
btnPlayButton[i].setForeground(Color.white);
btnPlayButton[i].addActionListener(this);
this.add(btnPlayButton[i]);
}
Xlabel = new Label("X: " + this.xScore);
Xlabel.setFont(new Font("Arial", Font.BOLD, 24));
Xlabel.setForeground(Color.white);
Xlabel.setBackground(Color.black);
this.add(Xlabel);
btnRestart = new Button("Restart");
btnRestart.setActionCommand("Restart");
btnRestart.setFont(new Font("Arial", Font.PLAIN, 18));
btnRestart.addActionListener(this);
this.add(btnRestart);
Olabel = new Label("O: " + this.oScore);
Olabel.setFont(new Font("Arial", Font.BOLD, 24));
Olabel.setForeground(Color.white);
Olabel.setBackground(Color.black);
this.add(Olabel);
this.setLayout(new GridLayout(4, 3));
this.pack();
this.setResizable(true);
this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
this.setTitle("Tic Tac Toe");
this.setSize(300, 400);
this.getContentPane().setBackground(Color.black);
this.setVisible(true);
}
@Override
public void actionPerformed(ActionEvent e) {
if (e.getActionCommand().equals("Restart")) {
System.out.println("Restarted");
for (int i = 0; i < 9; i++) {
btnPlayButton[i].setLabel("" + i);
btnPlayButton[i].setForeground(Color.white);
btnPlayButton[i].setBackground(Color.white);
btnPlayButton[i].addActionListener(this);
this.buttonCounter = 0;
}
} else {
((Button) e.getSource()).setFont(new Font("Arial", Font.BOLD, 48));
((Button) e.getSource()).setForeground(Color.black);
System.out.println(buttonCounter);
if (buttonCounter % 2 == 0) {
((Button) e.getSource()).setLabel("X");
((Button) e.getSource()).removeActionListener(this);
} else {
((Button) e.getSource()).setLabel("O");
((Button) e.getSource()).removeActionListener(this);
}
buttonCounter++;
CheckField();
}
}
private void CheckField() {
if (ButtonsWithIdenticalLabels(0, 1, 2)) {
Deactivatebuttons();
}
if (ButtonsWithIdenticalLabels(3, 4, 5)) {
Deactivatebuttons();
}
if (ButtonsWithIdenticalLabels(6, 7, 8)) {
Deactivatebuttons();
}
if (ButtonsWithIdenticalLabels(0, 3, 6)) {
Deactivatebuttons();
}
if (ButtonsWithIdenticalLabels(1, 4, 7)) {
Deactivatebuttons();
}
if (ButtonsWithIdenticalLabels(2, 5, 8)) {
Deactivatebuttons();
}
if (ButtonsWithIdenticalLabels(0, 4, 8)) {
Deactivatebuttons();
}
if (ButtonsWithIdenticalLabels(2, 4, 6)) {
Deactivatebuttons();
}
}
private boolean ButtonsWithIdenticalLabels(int i, int j, int k) {
if (btnPlayButton[i].getLabel() == btnPlayButton[j].getLabel()
&& btnPlayButton[j].getLabel() == btnPlayButton[k].getLabel()) {
btnPlayButton[i].setBackground(Color.red);
btnPlayButton[j].setBackground(Color.red);
btnPlayButton[k].setBackground(Color.red);
if (btnPlayButton[i].getLabel().equals("X")) {
xScore++;
Xlabel.setText("X: " + xScore);
} else {
oScore++;
Olabel.setText("O: " + oScore);
}
return true;
} else {
return false;
}
}
private void Deactivatebuttons() {
for (int i = 0; i < 9; i++) {
btnPlayButton[i].removeActionListener(this);
}
}
}
现在让我解释一下程序是如何工作的。 3x3比赛场地由ButtonArray
btnPlayButton组成。这些按钮将通过其标签进行比较，因此在游戏开始时没有匹配的标签，因此在创建按钮时，它们的标记从1到9。这里：for (int i = 0; i < 9; i++) {
btnPlayButton[i] = new Button("" + i); // Right here
btnPlayButton[i].setBackground(Color.white);
btnPlayButton[i].setForeground(Color.white);
btnPlayButton[i].addActionListener(this);
this.add(btnPlayButton[i]);
}
只要您单击
btnPlayButton，程序就会跳入actionPerformed方法。由于btnPlayButtons没有ActionCommand，它直接跳到方法的else部分。在这里，int buttonCounter增大1。如果buttonCounter是偶数还是奇数，则单击的btnPlayButton将重新标记为“ X”或“ O”。由于每次单击buttonCounter都会获得+1，因此X和O交替出现。这是说的部分：
else {
((Button) e.getSource()).setFont(new Font("Arial", Font.BOLD, 48));
((Button) e.getSource()).setForeground(Color.black);
System.out.println(buttonCounter);
if (buttonCounter % 2 == 0) {
((Button) e.getSource()).setLabel("X");
((Button) e.getSource()).removeActionListener(this);
} else {
((Button) e.getSource()).setLabel("O");
((Button) e.getSource()).removeActionListener(this);
}
buttonCounter++;
CheckField();
}
单击的按钮的
ActionListener被删除以防止作弊。每按一次按钮，就会检查比赛场地是否有获胜组合。这发生在CheckField()中。在
CheckField()中，或更准确地说，在ButtonsWithIdenticalLabels(x, y, z)中，将对btnPlayButtons[x]，btnPlayButtons[y]，btnPlayButtons[z]的标签进行比较，如果它们相同，则返回true。由于btnPlayButton的排序如下：
0 1 2
3 4 5
6 7 8
获胜组合为：012,345,678,036,147,258,045和246
因此，例如，当
btnPlayButton[0]，btnPlayButton[1]和btnPlayButton[2]都具有相同的标签时。 ButtonsWithIdenticalLabels为true，程序跳入Deactivatebuttons()，所有btnPlayButton均被禁用，这意味着找到了获胜组合，游戏结束了。如果btnPlayButton[1]的标签为“ X”，则int xScore将添加1。此外，出于美观目的，btnPlayButton[0]，btnPlayButton[1]和btnPlayButton[2]也涂成红色。使用“重新启动”按钮，您将进入一个for循环，该循环再次重新标记btnPlayButton并将其添加到在TTTFrame中实现的
ActionListener。 buttonCounter也将重置为0。重新标记与课程开始时的相同：if (e.getActionCommand().equals("Restart")) {
System.out.println("Restarted");
for (int i = 0; i < 9; i++) {
btnPlayButton[i].setLabel("" + i);
btnPlayButton[i].setForeground(Color.white);
btnPlayButton[i].setBackground(Color.white);
btnPlayButton[i].addActionListener(this);
this.buttonCounter = 0;
}
现在的问题是，在重新启动几次后，X和O的标签不再交替了。有时连续3个O，有时甚至像这样的Field也被认为是胜利
Picture
如果有人知道如何解决此错误，我将非常高兴。
提前致谢，
菲迪
最佳答案
这里的问题是：当您重新启动游戏时，新的ActionListener添加到每个按钮。但是，只有当您单击它或某人赢得游戏时，它才会被删除。这意味着当您在任何人赢得比赛之前重新启动游戏时，每个未单击的按钮都会获得第二个ActionListener，因此该点击将被注册两次，并且会出现此错误。重置板前，请尝试致电DeactivateButtons()。