Как закончить игру крестики нолики?

Question

[Janna1603]

Привет! Я пишу программу игру крестики-нолики. Я пытаюсь сделать проверку на выигрыш, и так и сяк пыталась. Ниже моя программа и проверка "check win". НО у меня почему то теперь подчеркиваются мои переменные 'player' и 'computer'. Я уже окончательно запуталась. Помогите. И если знаете, как по другому проверку можно сделать...

import java.util.Scanner;
import java.util.Random;


public class Main {

    public static void main(String[] args) {
        System.out.println("Приветствую Вас!");
        System.out.println("Сыграем в крестики-нолики?");
        System.out.println("===============");
        initMap();
        printMap();
        while (true) {
            humanTurn();
            printMap();
            if (checkWin(player)) {
                System.out.println("Победил человек");
                break;
            }
            if (isMapFull()) {
                System.out.println("Ничья");
                break;
            }
            compTurn();
            printMap();
            if (checkWin(computer)) {
                System.out.println("Победил Искуственный Интеллект");
                break;
            }
            if (isMapFull()) {
                System.out.println("Ничья");
                break;
            }
        }
        System.out.println("Игра закончена");
    }



    public static char[][] map;
    public static int size = 3;
    public static int dots_for_win = 4;

    public static char space = '-';
    public static char player = 'O';
    public static char computer = 'X';
    public static Scanner sc = new Scanner(System.in);
    public static Random rand = new Random();



    static void initMap() {
            map = new char[size][size];
            for (int i = 0; i < size; i++) {
                for (int j = 0; j < size; j++) {
                    map[i][j] = space;
                }
            }
        }

        public static void printMap() {
            for (int i = 0; i <= size; i++) {
                System.out.print(i + " ");
            }
            System.out.println();
            for (int i = 0; i < size; i++) {
                System.out.print((i + 1) + " ");
                for (int j = 0; j < size; j++) {
                    System.out.print(map[i][j] + " ");
                }
                System.out.println();
            }
            System.out.println();
        }
        public static void humanTurn() {
            int x, y;
        do {
            System.out.println("Введите координаты в формате X Y");
            x = sc.nextInt() - 1;
            y = sc.nextInt() - 1;
        } while (!ValidCell(x, y));
        map[y][x] = computer;
    }


    public static void compTurn() {
        int x, y;
        do {
            x = rand.nextInt(size);
            y = rand.nextInt(size);
        } while (!ValidCell(x, y));
        System.out.println("Компьютер походил в точку " + (x + 1) + " " + (y + 1));
        map[y][x] = player;
    }





    public static boolean ValidCell(int x, int y) {
        if (x < 0 || x >= size || y < 0 || y >= size) return false;
        if (map[y][x] == space) return true;
        return false;
}







    private static boolean checkWin(String[][] desc) {
        return checkWinnerHorizontal(desc) ||
                checkWinnerVertical(desc) ||
                checkWinnerDiagonals(desc);
    }

    private static boolean checkWinnerHorizontal(String[][] desc) {
        return desc[0][0].equals(desc[1][0]) && desc[0][0].equals(desc[2][0]) ||
                desc[0][1].equals(desc[1][1]) && desc[0][1].equals(desc[2][1]) ||
                desc[0][2].equals(desc[1][2]) && desc[0][2].equals(desc[2][2]);
    }

    private static boolean checkWinnerVertical(String[][] desc) {
        return desc[0][0].equals(desc[0][1]) && desc[0][0].equals(desc[0][2]) ||
                desc[1][0].equals(desc[1][1]) && desc[1][0].equals(desc[1][2]) ||
                desc[2][0].equals(desc[2][1]) && desc[2][0].equals(desc[2][2]);
    }

    private static boolean checkWinnerDiagonals(String[][] desc) {
        return desc[0][0].equals(desc[1][1]) && desc[0][0].equals(desc[2][2]) ||
                desc[2][0].equals(desc[1][1]) && desc[2][0].equals(desc[0][2]);
    }


    public static boolean isMapFull() {
        for (int i = 0; i < size; i++) {
            for (int j = 0; j < size; j++) {
                if (map[i][j] == space) return false;
            }
        }
        return true;
    }

}

Comments

[prrrrrrr]

System.out.println("Приветствую Вас!");
System.out.println("Сыграем в крестики-нолики?");
System.out.println("===============");

это жесть

[GavriKos]

prrrrrrr, да ну, нормально для новичка.
Janna1603 оберните код в соответствующий тег - читать невозможно

[Janna1603]

GavriKos, это как? у меня же тег Java

[GavriKos]

Janna1603, кнопка есть - вставить исходный код

[Денис Загаевский]

prrrrrrr, что не так в этом месте?

[Денис Загаевский]

GavriKos, взял бы да поправил

[prrrrrrr]

Денис Загаевский,

что не так в этом месте?

то, что придумали символ перехода на новую строку.

[Денис Загаевский]

prrrrrrr, да ну? И что?

Answer 1

[Dmitry Roo]

1. Метод checkWin принимает String[][], а вы туда пытаетесь char засунуть.
2. Методы checkWinnerHorizontal, checkWinnerVertical, checkWinnerDiagonals, похоже проверяют равенство ячеек массива, но не его содержание. Т.е. нужно еще добавить проверку на равенство крестику или нолику.

Comments

[Сергей Водаков]

Равенство крестику или нолику проверять не обязательно. Ведь если небыло заполненных линий, а после хода такая линия появилась, то логично, что игрок этого хода и победил.

[Dmitry Roo]

Похоже на правду.
Но все равно надо будет предусмотреть проверку на не пустые ячейки. На первом ходу игра закорнчится.

Приветствую Вас!
Сыграем в крестики-нолики?
===============
0 1 2 3 
1 - - - 
2 - - - 
3 - - - 

Введите координаты в формате X Y
1 1
0 1 2 3 
1 X - - 
2 - - - 
3 - - - 

Победил человек
Игра закончена

[Сергей Водаков]

Dmitry Roo, а вот с этим согласен, такая проблема похоже есть.