问题
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.
来源：https://stackoverflow.com/questions/27317055/tic-tac-toe-checking-winner