java .lang.OutOfMemoryError при попытке реализовать минимаксный алгоритм - PullRequest
0 голосов
/ 03 апреля 2020

Я реализовал минимаксный алгоритм для варианта «дикого времени» Java в *1015* и наткнулся на проблему. У меня есть класс Node, который содержит игровую сетку и объекты ArrayList of Node, которые являются его дочерними элементами, и минимаксный метод, который рекурсивно реализует алгоритм.

Я получаю ошибку:

Exception in thread "main" java.lang.OutOfMemoryError: GC overhead limit exceeded
        at Grid.<init>(Grid.java:35)
        at MiniMax$Node.findChildren(MiniMax.java:27)
        at MiniMax.minimax(MiniMax.java:135)
        at MiniMax.minimax(MiniMax.java:126)
        at MiniMax.minimax(MiniMax.java:139)
        at MiniMax.minimax(MiniMax.java:126)
        at MiniMax.minimax(MiniMax.java:139)
        at MiniMax.minimax(MiniMax.java:126)
        at MiniMax.minimax(MiniMax.java:139)
        at MiniMax.nextMove(MiniMax.java:77)
        at ComputerPlayer.play(ComputerPlayer.java:12)
        at TicTacToe.main(TicTacToe.java:146)

Process finished with exit code 1

Я думаю, что проблема возникает из-за большого количества дочерних элементов (всего узлов: 2 ^ 8 * 8!), Которые каждый раз создаются рекурсивно и сохраняются в ArrayLists.

Вот класс Node:

 private static class Node
    {
        protected Grid grid;
        protected ArrayList<Node> children;

        public Node(Grid grid)
        {
            this.grid = grid;
            children = new ArrayList<>();
        }

        //Find all possible next moves
        public void findChildren()
        {
            char[][] board = grid.getGrid();
            for(int i = 0; i < board.length; i++)
            {
                for(int j = 0; j < board.length; j++)
                {
                    if(board[i][j] == ' ')
                    {
                        board[i][j] = 'X';
                        children.add(new Node(new Grid(board)));
                        board[i][j] = 'O';
                        children.add( new Node(new Grid(board)));
                        board[i][j] = ' ';
                    }
                }
            }
        }
    }

Вот минимаксная реализация:

private int minimax(Node state, int depth, boolean isMaximizer)
    {
        //If the game is in a terminal state or has reached the desired depth
        boolean someoneWon = state.grid.someoneHasWon();
        boolean isDraw =  state.grid.isDraw();
        if(someoneWon || isDraw || depth == 3)
        {
            return evaluateState(someoneWon, isDraw, !isMaximizer);//Evaluate the state
        }
        //MAX player's turn
        if(isMaximizer)
        {
            //Find maximum score of all possible state's scores
            int bestScore = Integer.MIN_VALUE;
            state.findChildren();
            for(int i = 0; i < state.children.size(); i++)
            {
                Node child = state.children.get(i);
                int score = minimax(child, depth + 1, false);
                bestScore = Math.max(bestScore, score);
            }
            return bestScore;
        }
        else//MIN player's turn
        {
            //Find minimum score of all possible move's scores
            int bestScore = Integer.MAX_VALUE;
            state.findChildren();
            for(int i = 0; i < state.children.size(); i++)
            {
                Node child = state.children.get(i);
                int score = minimax(child, depth + 1, true);
                bestScore = Math.min(bestScore, score);
            }
            return bestScore;
        }
    }

1 Ответ

1 голос
/ 03 апреля 2020

Вместо создания списка дочерних узлов переместите итерацию в Node (или эквивалентный. Вы заметите, что вам не нужно каждый раз создавать новую доску - просто замените состояние, которое вы изменили после его завершения с.

...