Question

对于基于控制台的Tic Tac Toe程序，我无法在以下代码中获得所需的输出。任何人都可以帮我识别错误：

import java.util.Scanner;

以下是TicTacToe课程：

public class TicTacToe {
TicTacToe(){
    System.out.println("---------------------");
    System.out.println("    Tic Tac Toe     ");
    System.out.println("---------------------");
    System.out.println();
}
public static void main(String[] args) {
TicTacToe t = new TicTacToe();    

    Scanner sc = new Scanner(System.in);
    int a[][] = new int [3][3];

    for(int i=0; i<3; i++){
        for(int j=0; j<3; j++){
            a[i][j] = 2;
            System.out.print(" "+a[i][j]+" ");
        }
        System.out.println();
    }
    int count = 0;

我已经使用了do-while循环来执行break语句。虽然这样效率很低，但那时没有其他任何事情让我感到震惊。

    do{
    int[][] turn1 = t.player1(a);
        t.display(turn1);
    int[][] turn2 = t.player2(turn1);
        t.display(turn2);
    int[][] turn3 = t.player1(turn2);
        t.display(turn3);
    int[][] turn4 = t.player2(turn3);
        t.display(turn4);
    int[][] turn5 = t.player1(turn4);
        t.display(turn5);
        if(t.checkwin(turn5))
            break;
    int[][] turn6 = t.player2(turn5);
        t.display(turn6);
        if(t.checkwin(turn6))
            break;
    int[][] turn7 = t.player1(turn6);
        t.display(turn7);
        if(t.checkwin(turn5))
            break;
    int[][] turn8 = t.player2(turn7);
        t.display(turn8);
        if(t.checkwin(turn5))
            break;
    int[][] turn9 = t.player1(turn8);
        t.display(turn9);
        if(t.checkwin(turn5))
            break;
    System.out.println("Game Draw");
    count++;
    }while(count ==1);
}

以下是checkwin（）方法，该方法采用2D数组并返回boolean，用于中断或继续前一个do-while循环。

public boolean checkwin(int m[][]){
   for(int i=0; i<3; i++){
       if(m[i][0] == m[i][1] && m[i][1] == m[i][2]){
            if(m[i][0] == 1){
               player1wins();
               return true;
            }
            else if(m[i][0] == 0){
                player2wins();
                return true;
            }
       }

以下if语句的条件显示错误..

       if(m[0][i] == [1][i] && m[1][i] == m[2][i]){
            if(m[0][i] == 1){
               player1wins();
               return true;
            }
            else if(m[0][i] == 0){
                player2wins();
                return true;
            }
        }
   }

以下if语句的条件显示错误..

    if(m[0][0] == [1][1] && m[1][1] == m[2][2]){ 
            if(m[0][0] == 1){
               player1wins();
               return true;
            }
            else if(m[0][0] == 0){
                player2wins();
                return true;
            }
    }
    return false;
}

在player1wins（）之后，如果其中任何一个获胜，则显示player2wins（）方法。

public static void player1wins(){
    System.out.println("1st player wins");
}

public static void player2wins(){
    System.out.println("2nd player wins");
}

按照每次转弯后打印矩阵的显示方法..

public void display(int matrix[][]){
    for(int i=0; i<3; i++){
        for(int j=0; j<3; j++){
            System.out.print(" "+matrix[i][j]+" ");
        }
        System.out.println();
    }
}

跟随player1（）和player2（）方法获取用户的输入坐标..它们将输入矩阵作为参数并返回2D矩阵..

public int[][] player1(int matrix[][]){

    Scanner sc = new Scanner(System.in);
    System.out.println("1st player's turn...");
    System.out.print("Enter x coordinate: ");
    int x = sc.nextInt();
    System.out.print("Enter y coordinate: ");
    int y = sc.nextInt();

    matrix[x][y] = 1;
    return matrix;         
}

public int[][] player2(int matrix[][]){

    Scanner sc = new Scanner(System.in);
    System.out.println("2nd player's turn...");
    System.out.print("Enter x coordinate: ");
    int x = sc.nextInt();
    System.out.print("Enter y coordinate: ");
    int y = sc.nextInt();

    matrix[x][y] = 0;
    return matrix;         
}    
}

此外，如果任何人都可以帮助我优化这个基于tic-tac-toe的java代码或发布任何链接，那么它将会有很大的帮助。谢谢，提前......

Answer 1

if(m[0][0] == [1][1]...

应该是：

if(m[0][0] == m[1][1]...

Answer 2

你的if语句在语法上看起来不正确：

if(m[0][i] == [1][i] && m[1][i] == m[2][i]){

[1][i]之前缺少m。它应该是这样的：

if(m[0][i] == m[1][i] && m[1][i] == m[2][i]){

这同样适用于其他if语句：

更改

if(m[0][0] == [1][1] && m[1][1] == m[2][2]){

到

if(m[0][0] == m[1][1] && m[1][1] == m[2][2]){

这些都在你的checkwin（）方法中。

编辑：另一个错误：你的代码每回合检查一次转弯5。（对于第7,8和9轮，情况就是这样.6很好）。

此外，您可以使用return代替break，而不是使用do循环。

但我建议完全取消该位的重复：将其改为使用for和3d数组。

int[][][] turns = new int[10][][];
turns[0] = a;

// We start at 1 rather than 0 because the first value is a.
for (int i = 1; i <= 9; i++) {
    if (i % 2 == 1) { // Odd-numbered turn
        turns[i] = t.player1(turns[i - 1]);
    } else { // Even-numbered turn
        turns[i] = t.player2(turns[i - 1]);
    }

    t.display(turns[i]);

    if (t.checkwin(turns[i])) {
        return;
    }
}

System.out.println("Game Draw");

这看起来更清洁，应该运作良好。

Tic Tac Toe控制台程序 - Java

2 个答案: