Tic Tac Toe Checking winner

问题:

Im having problems with the win code. Ive been trying for a really long time and i have no idea what the problem is. Ive tried debugging but i got nothing from that.(Sorry for the Swedish comments)

import java.util.Scanner;
public class Tictactoe {

static char[][] MakeMove (char[][] spelplan, char spelare, int rad, int kolumn){
spelplan[rad][kolumn]=spelare;
System.out.println(spelplan[rad][kolumn]);
return spelplan;
}
static boolean CheckMove (char[][] spelplan, int x, int y){
if (spelplan[x][y] != ' ')
return false;
else return true;
}
static void SkrivUtSpelplan(char[][] spelplan){

System.out.println("-------");
System.out.println("|"+spelplan[1][1] + "|" + spelplan[1][2] + "|" +spelplan[1][3] + "|");
System.out.println("|-+-+-|");
System.out.println("|"+spelplan[2][1] + "|" + spelplan[2][2] + "|" +spelplan[2][3] + "|");
System.out.println("|-+-+-|");
System.out.println("|"+spelplan[3][1] + "|" + spelplan[3][2] + "|" +spelplan[3][3] + "|");
System.out.println("-------");

}

Here is the code part that checks for the winner

public static boolean KollaVinst(char[][] spelplan) {

return isHorizontalSolved(spelplan) || isVerticalSolved(spelplan) || isDiagonalSolved(spelplan);
}

Horizontal

 //Kollar om horisontella är löst
 public static boolean isHorizontalSolved(char[][] spelplan) {
 for (int y = 0; y < spelplan.length; ++y) {
    //För varje rad kolla om varje kolumn är fylld
    boolean solved = true;
    char first = spelplan[0][y];
    for (int x = 0; x < spelplan[y].length; ++x) {
        if (spelplan[x][y] == ' ' || first != spelplan[x][y]) {
            // Om en kolumn inte är fylld så är raden inte klar
            // Om en kolumn i raden är fylld med olika tecken så är den inte klar
            solved = false;
        }

    if (solved == true) {
        return true;
    }
}
}
return false;
}

horizontal ends

vertical

//Kollar om vertikala är löst
public static boolean isVerticalSolved(char[][] spelplan) {
for (int x = 0; x < spelplan.length; ++x) {

    boolean solved = true;
char first = spelplan[x][0];
for (int y = 0; y < spelplan[x].length; ++y){
    if (spelplan[x][y] == ' ' || first != spelplan[x][y]){
        solved = false;
    }
}
if (solved == true){
    return true;
}
}
return false;
}

vertical ends.

diagonal left to right

// Kollar om digonalen är löst
public static boolean isDiagonalSolved(char[][] spelplan) {
// Kollar vänster till höger
char first = spelplan[0][0];
boolean solved = true;
for (int y = 0, x = 0; y < spelplan.length && x < spelplan[y].length; ++y, ++x) {
    if (spelplan[x][y] == ' ' || first != spelplan[x][y]) {
        //Om en plats är tom eller om det är olika tecken så är den inte klar
        solved = false;
    }
}
if (solved) {
    return true;
}

diagonal left to right ends

diagonal right to left

//Kollar höger till vänster
int topRightX = spelplan[0].length - 1;
solved = true;
first = spelplan[0][topRightX];
for (int y = 0, x = topRightX; y < spelplan.length && x >= 0; ++y, --x) {
    if (spelplan[x][y] == ' ' || first != spelplan[x][y]) {
        //Om en plats är tom eller om det är olika tecken så är den inte klar
        solved = false;
    }
}
return solved;
}

Here the check winner code ends.

public static void main(String[] args) {
char spelplan[][] = new char [4][4];
char spelare;
int rad=3, kolumn=3, i=0;
for(int x=1; x<4; x++){
    for (int y=1; y<4; y++){
    spelplan[x][y]=' ';
    }
}


System.out.println("-------");
System.out.println("| | | |");
System.out.println("|-+-+-|");
System.out.println("| | | |");
System.out.println("|-+-+-|");
System.out.println("| | | |");
System.out.println("-------");

    for (i=0; i<=9; i++){
    if (KollaVinst(spelplan) == false){
    break;
}
    else

    CheckMove(spelplan, rad, kolumn);

    for (i=0; i<9; i++){
    if (i%2==0){
    spelare='X';
  }
    else
    spelare='O';

    System.out.println("Spelare 1 skriv vilken rad: 1-3");
    int x = new Scanner(System.in).nextInt();

    System.out.println("Spelare 1 skriv vilken kolumn: 1-3");
    int y = new Scanner(System.in).nextInt();

    if (CheckMove(spelplan, x, y) == true){
    MakeMove(spelplan, spelare, x, y);


}
System.out.println(" ");
SkrivUtSpelplan(spelplan);
}
}
}
}

回答1:

Two things to do (generally speaking):

1) change your main class - you are checking winner before first move and after last move... So game for should looks like this:

    for (i = 0; i < 9; i++) {
        if (KollaVinst(spelplan)) {
            break;
        } else {
            CheckMove(spelplan, rad, kolumn);
        }

        if (i % 2 == 0) {
            spelare = 'X';
        } else {
            spelare = 'O';
        }

        System.out.println("Spelare 1 skriv vilken rad: 1-3");
        int x = new Scanner(System.in).nextInt();

        System.out.println("Spelare 1 skriv vilken kolumn: 1-3");
        int y = new Scanner(System.in).nextInt();

        if (CheckMove(spelplan, x, y) == true) {
            MakeMove(spelplan, spelare, x, y);

        }
        System.out.println(" ");
        SkrivUtSpelplan(spelplan);
    }

2) Checking winner - I upgraded your functions (see below). You don't need to iterate over everything (especially fileds[0][x]), because beyond field you aer checking, there only two other fields ;) So in horizontal and vertical checking one for is enough (moreover, you can check both this possibilities in one for). And for checking diagonals, for is not needed - its only two possibilities to win game this way.

    public static boolean isHorizontalSolved(char[][] spelplan) {
        boolean solved = false;
        for (int y = 1; y < spelplan.length; y++) {
            if ((spelplan[y][1] == spelplan[y][2]) && (spelplan[y][1] == spelplan[y][3]) && (spelplan[y][1] != ' ')) {
                solved = true;
            }
        }
        return solved;
    }

    public static boolean isVerticalSolved(char[][] spelplan) {
        boolean solved = false;
        for (int y = 1; y < spelplan.length; y++) {
            if ((spelplan[1][y] == spelplan[2][y]) && (spelplan[1][y] == spelplan[3][y]) && (spelplan[1][y] != ' ')) {
                solved = true;
            }
        }
        return solved;
    }

    public static boolean isDiagonalSolved(char[][] spelplan) {
        boolean solved = false;
        if ((spelplan[1][1] == spelplan[2][2]) && (spelplan[1][1] == spelplan[3][3]) && (spelplan[1][1] != ' ')) {
                solved = true;
            }

        if ((spelplan[1][3] == spelplan[2][2]) && (spelplan[1][3] == spelplan[3][1]) && (spelplan[1][3] != ' ')) {
                solved = true;
            }

        return solved;
    }

回答2:

In your isHorizontalSolved you have a mistake. The for loop closing bracket should be placed before the following if:

//Kollar om horisontella är löst
public static boolean isHorizontalSolved(char[][] spelplan) {
    for (int y = 0; y < spelplan.length; ++y) {
        //För varje rad kolla om varje kolumn är fylld
        boolean solved = true;
        char first = spelplan[0][y];
        for (int x = 0; x < spelplan[y].length; ++x) {
           if (spelplan[x][y] == ' ' || first != spelplan[x][y]) {
              // Om en kolumn inte är fylld så är raden inte klar
              // Om en kolumn i raden är fylld med olika tecken så är den inte klar
              solved = false;
           }
        }

        if (solved == true) {
           return true;
        }
    }
    return false;
}

The way you did it, the loop would exit with true in the first iteration.