如何通过避免重复来缩短井字游戏代码

Question

这是我的第一个 JavaFX 项目。

我使用 JavaFx 创建了一个井字游戏。该游戏适用于玩家对玩家模式，非常完美，但我使用的代码太长了。

这里有一些明显的重复，但它们完全不同：

Tic-tac-toe code help improve

Code Golf: Tic Tac Toe

这是我的按钮一的代码，我必须重复八个按钮：

one.setOnAction(new EventHandler<ActionEvent>() {
    @Override
    public void handle(ActionEvent actionEvent) {
        System.out.println(1);
        System.out.println("my state is " + state[0]);
        if (state[0] == 0){
            state[0]  = 1 ;
            for (int a: state ) {
            System.out.print(a);}
            if(i%2==0){
                one.setText("X"); i+= 1;
                System.out.println(i+"recorded");
                turn.setText("O turn");
                result[0] = 'x';
                win() ;
            }
            else{
                one.setText("O"); i+= 1 ;
                System.out.println(i+"recorded");
                turn.setText("X turn");
                result[0] = 'o';
                win() ;
            }
        }
    }
});

截图：

是否可以选择避免为另外 8 个按钮重复代码？

您可以在屏幕截图中看到也有重复的 for 循环。

这是我的第一个 gui 项目，但我不打算成为初学者。

编辑

在我没有在代码中看到任何 for 循环之后。也许您的意思是 if 语句？ – NiVer

是的，我的意思是 if 语句。

Answer 1

尝试这样做：

void makeMove(Button button, JLabel turn, int cell){
    System.out.println(cell);
    System.out.println("my state is " + state[cell-1]);
    if (state[cell-1] == 0){
        state[cell-1]  = 1 ;
        for (int a: state ) {
        System.out.print(a);}
        if(i%2==0){
            button.setText("X"); i+= 1;
            System.out.println(i+"recorded");
            turn.setText("O turn");
            result[cell-1] = 'x';
            win() ;
        }
        else{
            button.setText("O"); i+= 1 ;
            System.out.println(i+"recorded");
            turn.setText("X turn");
            result[cell-1] = 'o';
            win() ;
        }
    }
}

one.setOnAction(new EventHandler<ActionEvent>() {
    @Override
    public void handle(ActionEvent actionEvent) {
        makeMove(one,turn,1);
    }
);

并通过将 1 更改为 2...9 对其他按钮重复此操作

这段代码实际上还可以改进，但是，既然你说你刚刚开始，这是开始考虑模块化代码的好方法。

我不确定turn 是什么，所以我将它设为makeMove 定义中的标签，将其更改为实际的任何内容。