打破听众的递归

Question

我试图打破使此代码递归的特殊情况。

我有一个 Javafx 游戏，其中有人类和计算机玩家在轮到他的时候玩，并且可能有很多轮。

计算机应该自动播放并立即移动到下一个播放器，并且不会直接向 UI 显示任何指示（但可以查看它之后的操作）。

问题是在只有电脑玩家的情况下，我们会在加载currentBoardPane的那一刻来到这里，进入所有玩家都是电脑的条件，设置下一个玩家的棋盘，然后没有完成调用，再次调用相同的函数：

    currentBoardPane.addListener((e) -> {  
        if(gameManager.checkIfCurrentPlayerIsComputer()){

            gameManager.playAutoMovesForCurrentPlayer();
            gameManager.setNextPlayer(); // it does current player property = next player

            //update board on scene
            currentBoardPaneIndex = ((currentBoardPaneIndex + 1) % gameManager.getPlayers().size());
            currentBoardPane.setValue(boardPanes.get((currentBoardPaneIndex))); //this is a recursive call
        }
    });

如果我订阅了GameManager 中的currentPlayer 属性的侦听器，那么我仍然需要从该侦听器调用setNextPlayer()，而这又是递归的。

如果所有玩家都是计算机，我可以做一个特殊情况，然后从 while(true){} 运行游戏而不是侦听器和绑定，但必须有更好的方法来打破这种递归。

有没有办法在仍然有侦听器和绑定的同时不进入递归？

注意事项：

currentBoardPane 表示屏幕上的当前游戏板，它是ObjectProperty。

Answer 1

对您的代码做出以下假设：

1. 目前一切都在 FX 应用程序线程上运行
2. currentBoardPane.setValue(...) 会导致 UI 更新（因此您每次移动都会更新 UI）

那么一个“快速而肮脏”的方法是：

currentBoardPane.addListener((e) -> {  
    if(gameManager.checkIfCurrentPlayerIsComputer()){

        gameManager.playAutoMovesForCurrentPlayer();

        //update board on scene
        Platform.runLater(() -> {
            gameManager.setNextPlayer(); // it does current player property = next player
            currentBoardPaneIndex = ((currentBoardPaneIndex + 1) % gameManager.getPlayers().size());
            currentBoardPane.setValue(boardPanes.get((currentBoardPaneIndex))); //this is a recursive call
        });
    }
});

这会将更新委托给新的Runnable，调度可运行在 FX 应用程序线程上执行，并立即退出处理程序。因此对currentBoardPane.setValue(...) 的调用稍后执行，不再递归。

事实上，如果你多做一点工作：

private final Executor aiExecutor = Executors.newSingleThreadExecutor();

// ...

currentBoardPane.addListener((e) -> {  
    if(gameManager.checkIfCurrentPlayerIsComputer()){

        Task<Void> makeMoveTask = new Task<Void>() {
            @Override
            protected Void call() {
                gameManager.playAutoMovesForCurrentPlayer();
                return null ;
            }
        };

        makeMoveTask.setOnSucceeded(e -> {

            //update board on scene
            gameManager.setNextPlayer(); // it does current player property = next player
            currentBoardPaneIndex = ((currentBoardPaneIndex + 1) % gameManager.getPlayers().size());
            currentBoardPane.setValue(boardPanes.get((currentBoardPaneIndex))); //this is a recursive call
        });

        aiExecutor.execute(makeMoveTask);
    }
});

如果计算移动需要足够的时间，以至于在移动发生时阻止 UI 是不可接受的，那么这正是您将使用的代码。 （如果计算移动需要很少的时间，这仍然可以正常工作。）这假设 playAutoMovesForCurrentPlayer() 不更新 UI。