0

Вот есть сам код: (это игра крестики нолики):


public class Main {

    public static char[][] map;
    public static final int SIZE = 5;
    public static final int DOTS_TO_WIN = 4;
    public static final char DOT_EMPTY = '•';
    public static final char DOT_X = 'X';
    public static final char DOT_O = 'O';
    public static Scanner vvod = new Scanner(System.in);

    public static void newMap() {
        map = new char[SIZE][SIZE];
        for (int i = 0; i < SIZE; i++) {
            for (int j = 0; j < SIZE; j++) {
                map[i][j] = DOT_EMPTY;
            }
        }
    }

    public static void printMap() {
        for (int i = 0; i < SIZE; i++) {
            for (int j = 0; j < SIZE; j++) {
                System.out.print(map[i][j] + " ");
            }
            System.out.println();
        }
        System.out.println();
    }

    public static void human() {
        int x, y;
        do {
            System.out.println("Введите координаты точки (х у)");
            x = vvod.nextShort() - 1;
            y = vvod.nextShort() - 1;
        } while (!freefield(x, y));
        map[x][y] = DOT_X;
    }

    public static void comp() {
        int x, y;
        do {
            x = (int) (Math.random() * 6) - 1;
            y = (int) (Math.random() * 6) - 1;
        } while (!freefield(x, y));
        System.out.println("ход компьютера: x-" + (x + 1) + " у-" + (y + 1));
        map[x][y] = DOT_O;
    }

    public static boolean freefield(int x, int y) {
        if (x < 0 || x >= SIZE || y < 0 || y >= SIZE) {
            return false;
        }
        return map[y][x] == DOT_EMPTY;
    }

    public static boolean fullmap() {
        for (int i = 0; i < SIZE; i++) {
            for (int j = 0; j < SIZE; j++) {
                if (map[i][j] == DOT_EMPTY) return false;
            }
        }
        return true;
    }

    public static boolean proverka (char symb) {
        int x_win = 0, y_win = 0, col=0, row = 0, stop_1, stop_2;
        do {
            for (col = 0; col < 4; col++) {
                if (map[col][row] == symb) x_win++;
                else x_win = 0;
            }
            row++;
            if (x_win == 4) break; else x_win = 0;
        } while (row!=5) ;
        col = 0;
        do {
            for (row = 0; row < 4; row++) {
                if (map[row][col] == symb) y_win++;
                else y_win = 0;
            }
            col++;
            if (y_win == 4) break; else y_win = 0;
        } while (col!=5) ;
        if (x_win == 4 || y_win == 4) return true; else return false;
    }

    public static void main(String[] args) {
        newMap();
        printMap();
        while (true) {
            human();
            printMap();
            if (proverka(DOT_X));{
                System.out.println("Победа человека");
                break;
            }
            if (fullmap()){
                System.out.println("Ничья");
                break;
            }
            comp();
            printMap();
            if (proverka(DOT_O));{
                System.out.println("Победа компьютера");
               // break;
            }
            if (fullmap()){
                System.out.println("Ничья");
                break;
            }
        }
        System.out.println("Game over");
    }

В методе Main в цикле While есть вызывается класс proverka. В этом классе есть break, который выводит нас из цикла, если возвращается true. Если false, цикл продолжается дальше. Однако:

if (proverka(DOT_X));{
            System.out.println("Победа человека");
            break;
        }
        if (fullmap()){
            System.out.println("Ничья");
            break;
        }

Второй if уже не работает после break и программа говорит, что этот if-

unreachable statement.

Уже битый час не могу разобраться почему так, может тут помогут ☺☻

4
  • 1
    так а с чего второй if должен достигаться? break - напрямую говорит сломать цикл и выйти к едрене фене 17 дек 2017 в 19:16
  • а еще вот это if (proverka(DOT_X)); и if (proverka(DOT_O)); - вас ничего не смущает в конце? 17 дек 2017 в 19:18
  • break должен возвращаться только тогда, когда этот метод возвращает true, а он работает всегда и программа просто не может запуститься, если его не закомментить 17 дек 2017 в 19:18
  • емое, сколько сидел так и не увидел бревна в глазу 17 дек 2017 в 19:19

2 ответа 2

3

Проблема очевидно тут:

if (proverka(DOT_X)); {         if (proverka(DOT_O)); {
                    ^---         и                  ^----
2
  • если не сложно, просмотрите, пожалуйста код. пытаюсь поставить точку в координату 3-1, ставит ее в координату то 5-1, то 2-4 17 дек 2017 в 19:22
  • @ЛешаБелый я б предложил задать новый вопрос с именно этой проблемой. А данный вопрос - закрыть. А еще больше я бы рекомендовал поотлаживать немного код. Как и какими средствами находить ошибки в коде Java? ....... Ведь на данный момент с текущим кодом другому человеку придется именно это и сделать 17 дек 2017 в 19:40
1

Во-первых, нужно убрать точку с запятой после условия в if - если условие будет выполняться, ничего не будет происходить. А тот блок, который в логике твоего кода должен выполняться при этом условии - будет выполняться всегда , думай об этом так - если бы в твоем коде вообще не было бы этой строки:

if (proverka(DOT_X));

Таким образом, у тебя программа всегда вылетает из цикла (потому что break в этом блоке находится) и даже не доходит до второго if

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.