Игра Крестики-Нолики. как оптимизировать?

@Yegorh8 · Регистрация: 30.12.2018

Студворк — интернет-сервис помощи студентам

Всем доброго времени суток.
Задали задачку для поступления на курс.

Создать игру Крестики-нолики (класс XOBoard)
и в ней три процедуры "функции"

1 Put распологает Х и О на доске. Нельзя распаложить уже в занятой ячейке.
2 Status возврощает состояние доски Х - игрок Х выиграл, О - игрок О выиграл,
None игра неокончина и Draw игра окончина и нет победителя.
3 Display печатает игровую доску в Console.

Я как то попытался соблюсти все требования
Пожалусто подскажите что можно улучшить в моём кривом коде, сделать его "понятнее"
Заранее огромное спасибо

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
import java.util.Scanner;
 
public class XOBoard {
 
 
    public static void put(char[][] board, char player) {
 
        System.out.println("Player " + player + " Make your Move and press enter ");
 
        Scanner scanner = new Scanner(System.in);
        String userInput = scanner.nextLine();
        int userInputInt = Integer.parseInt(userInput);
 
        if (userInputInt < 0 || userInputInt > 9) {
            System.out.println("Please enter only one numbers between 1-9");
            put(board, player);
        } else {
 
            switch (userInput) {
                case "1" -> {
                    if (board[0][0] == '1') {
                        board[0][0] = player;
                    } else {
                        System.out.println("This place is not empty");
                        put(board, player);
                    }
                }
                case "2" -> {
                    if (board[0][1] == '2') {
                        board[0][1] = player;
                    } else {
                        System.out.println("This place is not empty");
                        put(board, player);
                    }
                }
                case "3" -> {
                    if (board[0][2] == '3') {
                        board[0][2] = player;
                    } else {
                        System.out.println("This place is not empty");
                        put(board, player);
                    }
                }
                case "4" -> {
                    if (board[1][0] == '4') {
                        board[1][0] = player;
                    } else {
                        System.out.println("This place is not empty");
                        put(board, player);
                    }
                }
                case "5" -> {
                    if (board[1][1] == '5') {
                        board[1][1] = player;
                    } else {
                        System.out.println("This place is not empty");
                        put(board, player);
                    }
                }
                case "6" -> {
                    if (board[1][2] == '6') {
                        board[1][2] = player;
                    } else {
                        System.out.println("This place is not empty");
                        put(board, player);
                    }
                }
                case "7" -> {
                    if (board[2][0] == '7') {
                        board[2][0] = player;
                    } else {
                        System.out.println("This place is not empty");
                        put(board, player);
                    }
                }
                case "8" -> {
                    if (board[2][1] == '8') {
                        board[2][1] = player;
                    } else {
                        System.out.println("This place is not empty");
                        put(board, player);
                    }
                }
                case "9" -> {
                    if (board[2][2] == '9') {
                        board[2][2] = player;
                    } else {
                        System.out.println("This place is not empty");
                        put(board, player);
                    }
                }
                default -> System.out.println("something wrong =(");
            }
        }
    }
 
    public static char status(char[][] board, char player, int round){
 
        if((board[0][0] == player && board[0][1] == player && board[0][2] == player) ||
           (board[1][0] == player && board[1][1] == player && board[1][2] == player) ||
           (board[2][0] == player && board[2][1] == player && board[2][2] == player) ||
 
           (board[0][0] == player && board[1][0] == player && board[2][0] == player) ||
           (board[0][1] == player && board[1][1] == player && board[2][1] == player) ||
           (board[0][2] == player && board[1][2] == player && board[2][2] == player) ||
 
           (board[0][0] == player && board[1][1] == player && board[2][2] == player) ||
           (board[0][2] == player && board[1][1] == player && board[2][0] == player) ){
            System.out.println("The Player " + player + " is a WINNER!!!");
            return player;
 
        } else if (round == 8) {
            System.out.println("The Game is over and NO WINNER");
            return 'D';
        } else {
            System.out.println("This is not the end of the Game");
            return 'N';
        }
    }
 
    public static void display(char[][] board) {
        for (int i = 0; i < 3; i++) {
            for (int j = 0; j < 3; j++) {
                System.out.print("|" + board[i][j] + "|");
            }
            System.out.print("\n");
        }
    }
 
    public static void main(String[] args) {
 
        char whoWin = ' ';
        char[][] board = {{'1', '2', '3'},
                          {'4', '5', '6'},
                          {'7', '8', '9'}};
 
 
        display(board);
 
        for(int i = 0; i < 9; i++){
            if(i % 2 == 0){
                put(board,'X');
                if(i >= 4){
                    whoWin = status(board, 'X', i);
                    if(whoWin == 'X'){
                        i = 10;
                    }
                }
                display(board);
            } else {
                put(board,'O');
                if(i >= 4){
                    whoWin = status(board, 'O', i);
                    if(whoWin == 'O'){
                        i = 10;
                    }
                }
                display(board);
            }
        }
    }
}

@xoraxax · 28.04.2021, 02:11

Что если доска будет 5х5?

@Yegorh8 · 28.04.2021, 02:20 **[ТС]**

В задаче написано "Доска может быть любого размера"
Без уточнений и т.п
Я не уверен как это понимать, так что пошел от простого 3х3

@Aviz__ · 28.04.2021, 13:54

Yegorh8, вот, держи начало, где доска может быть примерно 20х20

код

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
import java.util.Arrays;
 
public class Helper {
 
    public static void main(String[] args) {
        XOBoard xoBoard = new XOBoard(5,5);
        if (xoBoard.put(1,1, 1))
            xoBoard.printBoard();
        if (xoBoard.put(1,2, 2))
            xoBoard.printBoard();
 
    }
}
//###
class XOBoard {
    private int[][] board;
    private boolean isEmpty = true;
    private int lastMoveRow;
    private int lastMoveCol;
 
    public XOBoard(int[][] board) {
        this.board = board;
    }
 
    public XOBoard(int row, int column) {
        this.board = new int[row][column];
        for (int[] aRow : board)
            Arrays.fill(aRow, 0);
    }
 
    public boolean put(int row, int column, int aplayerMarker) {
        isEmpty = false;
        boolean retVal = false;
        if (isCellValid(row, column)) {
            board[row][column] = aplayerMarker;
            lastMoveRow = row;
            lastMoveCol = column;
            retVal = true;
        }
        return retVal;
    }
 
    private boolean isCellValid(int row, int column) {
        boolean isRowValid = false;
        boolean isColValid = false;
        if (row >= 0 && row < board.length) isRowValid = true;
        if (column >= 0 && row < board[0].length) isColValid = true;
        if (isRowValid && isColValid) {
            if (isEmpty)   // на пустую доску делается ход?
                return  true;
            else if (board[row][column] == 0) { // как далеко ли от последнего хода
                int rowDist = Math.abs(row - lastMoveRow);
                int colDist = Math.abs(column - lastMoveCol);
                if (rowDist + colDist <= 2) return true;
            }
        }
         return false;
    }
 
    public void printBoard() {
        for (int i = 0; i < board.length; i++) {
            for (int j = 0; j < board[0].length; j++) {
                if (board[i][j] == 0)
                    System.out.print("|   ");
                else if (board[i][j] == 1)
                    System.out.print("| X ");
                else if (board[i][j] == 2)
                    System.out.print("| O ");
            }
            System.out.print("|\n" + getLineSeparator('-',(board[0].length)  * 4) + '\n');
        }
        System.out.println(getLineSeparator('=',(board[0].length)  * 4));
    }
    private String getLineSeparator(char chr,int length) {
        StringBuilder builder = new StringBuilder();
        while (length-- > -1)
            builder.append(chr);
        return builder.toString();
    }
}

@Tavashi · 28.04.2021, 19:17

Yegorh8, можете еще свериться еще с этим источником.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11680&d=1772460536 Одним из. . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .	SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .

@xoraxax 3639 / 2971 / 918 Регистрация: 05.07.2013 Сообщений: 14,220
	28.04.2021, 02:11
	Что если доска будет 5х5? 1

@Yegorh8 0 / 0 / 0 Регистрация: 30.12.2018 Сообщений: 12
	28.04.2021, 02:20 [ТС]
	В задаче написано "Доска может быть любого размера" Без уточнений и т.п Я не уверен как это понимать, так что пошел от простого 3х3 0

@Tavashi 1172 / 762 / 194 Регистрация: 21.05.2016 Сообщений: 1,858
	28.04.2021, 19:17
	Yegorh8, можете еще свериться еще с этим источником. 0