问题尚未解决，但必须解决。
【问题讨论】：
for 循环。这太重复了。
1 代表播放器，使用4 代表计算机。然后，您可以将行/列/对角线相加并从中得出含义。 1 和 4 很有用，因为当您将不同的可能排列相加时，含义上不会有任何重叠。
标签： java arrays tic-tac-toe
问题在于 if 语句中的这个表达式：
GameBoard[0][0] == player && GameBoard[0][1] == player && GameBoard[0][2] == player &&
GameBoard[1][0] == player && GameBoard[1][1] == player && GameBoard[1][2] == player &&
GameBoard[2][0] == player && GameBoard[2][1] == player && GameBoard[2][2] == player
您正在检查玩家是否在每个棋盘位置。
您想确定没有获胜者。这意味着检查行、列和对角线中没有相同的玩家。
【讨论】：
您没有检查“计算机”是否赢了，如果计算机没有赢，玩家也没有，那么您可以将 tieGame 标志设置为 true
【讨论】：
这可能不是您想要的，但我强烈建议您使用不同的移动/棋盘表示：
如何
首先，我建议您将板单元实现为枚举（因为只有 9 个）。
然后，将板子实现为一组 9 个枚举，如下所示：
public enum Move {
A0, A1, A2, B0, B1, B2, C0, C1, C2; }
此外，将获胜模式（例如整条线、整条对角线）实现为 3 个枚举的集合（即棋盘集的子集）。
进行这些更改后，检查玩家（无论是您还是计算机）是否获胜变得容易得多。
我建议您在 Move 类中添加子集和检查玩家是否获胜的方法。
可以这样创建子集： 例如
static private final EnumSet<Move> lineA = EnumSet.of(A0,A1,A2);
从您的获胜方法中进行的检查（注意您需要检查所有子集）可能是：
moves.containsAll(lineA)
其中moves是玩家做出的一组动作（当玩家做出动作时，您将枚举添加到它的动作集合中）。
为什么
这种设计更加简洁、直接、优雅。如果您使用这种方法，您应该不会再遇到您描述的问题。
【讨论】：
将所有 9 个条件的 GameBoard[0][0] == player 更改为 GameBoard[0][0] != 0。
【讨论】：
首先，在您的主要方法中，您说finalwinner=CheckForWinner(currentplayer);，因此您不必每次检查行、列和对角线后都设置finalwinner=true;。 return true; 就够了。
您的问题是，您检查平局游戏，如果是这样，您也返回 true，但这是错误的，因为玩家和计算机都没有获胜！所以你的 checkForWinner 方法 always 返回 true。
您实际上不必为 Tie Game 进行额外检查。只需检查玩家的胜利，检查计算机的胜利，如果两者都是错误的并且最后一步完成 - 这是一场平局。
【讨论】：
错误出现在函数的主要代码/结构中。您的函数需要返回两个变量，finalwinner 和 tie game。现在，您的函数在第一个 return 语句处停止，并将其设置为 main 中的 finalwinner。
试试这样的：
public static boolean[] CheckForWinner(int player)
{
//Row and Column Check
if (GameBoard[0][0] == player && GameBoard[0][1] == player && GameBoard[0][2] == player ||
GameBoard[1][0] == player && GameBoard[1][1] == player && GameBoard[1][2] == player ||
GameBoard[2][0] == player && GameBoard[2][1] == player && GameBoard[2][2] == player ||
GameBoard[0][0] == player && GameBoard[1][0] == player && GameBoard[2][0] == player ||
GameBoard[0][1] == player && GameBoard[1][1] == player && GameBoard[2][1] == player ||
GameBoard[0][2] == player && GameBoard[1][2] == player && GameBoard[2][2] == player)
{
boolean finalwinner=true;
}
//diagonal checks
if (GameBoard[0][0]==player && GameBoard[1][1]==player && GameBoard[2][2]==player)
{
boolean finalwinner=true;
}
if (GameBoard[2][0]==player && GameBoard[1][1]==player && GameBoard[0][2]==player)
{
boolean finalwinner=true;
}
//Tie Game Check
if (GameBoard[0][0] == player && GameBoard[0][1] == player && GameBoard[0][2] == player &&
GameBoard[1][0] == player && GameBoard[1][1] == player && GameBoard[1][2] == player &&
GameBoard[2][0] == player && GameBoard[2][1] == player && GameBoard[2][2] == player)
{
boolean tieGame=true;
}
return new boolean [] {finalwinner, tiegame};
}
public static void main (String[] args)
{
System.out.println("Tic Tac Toe Game");
System.out.println("Player 1 is 1, Computer is 2");
Draw();
currentplayer=1;
while (winner!= true)
{
if(currentplayer==1)
{
System.out.println("Your Turn");
Player1Turn(currentplayer);
}
else
ComputerTurn(currentplayer);
boolean result[] = CheckForWinner(currentplayer);
if(result[0]==true && result[1]==false)
{
System.out.println("Winner is Player" +currentplayer+ "Congrats Champion!");
System.out.println("Beginning New Game! Press Cancel then Enter to Exit");
Reset();
}
if(result[1]==true)
{
System.out.println("It's a Tie! Play Again.");
System.out.println("Beginning New Game! Press Cancel then Enter to Exit");
Reset();
}
if(currentplayer==1)
currentplayer=2;
else if(currentplayer==2)
currentplayer=1;
}
}
【讨论】：