我多年前制作了一个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变得更大.Willher buttonCounter是偶数或奇数，被点击的btnPlayButton被重新标记为“X”或“O”。由于buttonCounter每次点击都会获得+1，因此X和Os是交替的。
以下是该部分：
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为真，程序会跳转到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个Os，有时甚至像这样的字段被认为是胜利
如果有人知道如何修复这个错误，我会非常高兴。
提前致谢，
Fihdi
答案 0 :(得分：1)
这里的问题是：当你重新开始游戏时，每个按钮都会添加一个新的ActionListener。但是，只有在您点击它或有人赢得游戏时才会将其删除。这意味着当你在任何人赢了之前重新开始游戏时，每个未点击的按钮都会获得第二个ActionListener，因此点击将被注册两次并出现此错误。在重置电路板之前，请尝试拨打DeactivateButtons()。
