Почему происходит ошибка "unreachable statement" в блоке if?

Question

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

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 &lt; SIZE; i++) {
        for (int j = 0; j &lt; SIZE; j++) {
            map[i][j] = DOT_EMPTY;
        }
    }
}

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

public static void human() {
    int x, y;
    do {
        System.out.println(&quot;Введите координаты точки (х у)&quot;);
        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(&quot;ход компьютера: x-&quot; + (x + 1) + &quot; у-&quot; + (y + 1));
    map[x][y] = DOT_O;
}

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

public static boolean fullmap() {
    for (int i = 0; i &lt; SIZE; i++) {
        for (int j = 0; j &lt; 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 &lt; 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 &lt; 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(&quot;Победа человека&quot;);
            break;
        }
        if (fullmap()){
            System.out.println(&quot;Ничья&quot;);
            break;
        }
        comp();
        printMap();
        if (proverka(DOT_O));{
            System.out.println(&quot;Победа компьютера&quot;);
           // break;
        }
        if (fullmap()){
            System.out.println(&quot;Ничья&quot;);
            break;
        }
    }
    System.out.println(&quot;Game over&quot;);
}

В методе 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.

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

так а с чего второй if должен достигаться? break - напрямую говорит сломать цикл и выйти к едрене фене — Алексей Шиманский, Dec 17 '17 at 19:16
а еще вот это if (proverka(DOT_X)); и if (proverka(DOT_O)); - вас ничего не смущает в конце? — Алексей Шиманский, Dec 17 '17 at 19:18
break должен возвращаться только тогда, когда этот метод возвращает true, а он работает всегда и программа просто не может запуститься, если его не закомментить — Леша Белый, Dec 17 '17 at 19:18
емое, сколько сидел так и не увидел бревна в глазу — Леша Белый, Dec 17 '17 at 19:19

score 3 · Accepted Answer · answered Dec 17 '17 at 19:19

3

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

if (proverka(DOT_X)); {         if (proverka(DOT_O)); {
                    ^---         и                  ^----

answered Dec 17 '17 at 19:19

Алексей Шиманский

73,186

если не сложно, просмотрите, пожалуйста код. пытаюсь поставить точку в координату 3-1, ставит ее в координату то 5-1, то 2-4 – Леша Белый Dec 17 '17 at 19:22
@ЛешаБелый я б предложил задать новый вопрос с именно этой проблемой. А данный вопрос - закрыть. А еще больше я бы рекомендовал поотлаживать немного код. Как и какими средствами находить ошибки в коде Java? ....... Ведь на данный момент с текущим кодом другому человеку придется именно это и сделать – Алексей Шиманский Dec 17 '17 at 19:40

score 1 · Answer 2 · answered Dec 17 '17 at 22:56

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

if (proverka(DOT_X));

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

Почему происходит ошибка "unreachable statement" в блоке if?

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

2 Answers2