三连棋游戏重置太快

Question

我有一个井字棋游戏，用户(x)与电脑(o)对战。游戏开始时，电脑会在中心放置(o)，并在用户落子后随机移动到另一个位置。游戏设置为循环进行，但一旦有胜者就会重置，并不显示“你赢/输”的横幅。有时电脑会在我的位置上放置它的字母，我还在努力解决这个问题。

 public class example2

 {
   static boolean checkWinner(String string, JButton buttons []){
    if (buttons[0].getText() .equals("X") && buttons[1].getText()       .equals("X") && buttons[2].getText() .equals("X"))
        //label.setText("You win");
        return true;        
    else if (buttons[0].getText() .equals("O") && buttons[1].getText() .equals("O") && buttons[2].getText() .equals("O"))
        //label.setText("You lose");
        return true;

    if (buttons[3].getText() .equals("X") && buttons[4].getText() .equals("X") && buttons[5].getText() .equals("X"))
        //label.setText("You win");
        return true;        
    else if (buttons[3].getText() .equals("O") && buttons[4].getText() .equals("O") && buttons[5].getText() .equals("O"))
        //label.setText("You lose");
        return true;            

    if (buttons[6].getText() .equals("X") && buttons[7].getText() .equals("X") && buttons[8].getText() .equals("X"))
        //label.setText("You win");
        return true;        
    else if (buttons[6].getText() .equals("O") && buttons[7].getText() .equals("O") && buttons[8].getText() .equals("O"))
        //label.setText("You lose");
        return true;

    if (buttons[0].getText() .equals("X") && buttons[3].getText() .equals("X") && buttons[6].getText() .equals("X"))
        //label.setText("You win");
        return true;        
    else if (buttons[0].getText() .equals("O") && buttons[3].getText() .equals("O") && buttons[6].getText() .equals("O"))
        //label.setText("You lose");
        return true;        

    if (buttons[1].getText() .equals("X") && buttons[4].getText() .equals("X") && buttons[7].getText() .equals("X"))
        //label.setText("You win");
        return true;        
    else if (buttons[1].getText() .equals("O") && buttons[4].getText() .equals("O") && buttons[7].getText() .equals("O"))
        //label.setText("You lose");
        return true;

    if (buttons[2].getText() .equals("X") && buttons[5].getText() .equals("X") && buttons[8].getText() .equals("X"))
        //label.setText("You win");
        return true;        
    else if (buttons[2].getText() .equals("O") && buttons[5].getText() .equals("O") && buttons[8].getText() .equals("O"))
        //label.setText("You lose");
        return true;

    if (buttons[0].getText() .equals("X") && buttons[4].getText() .equals("X") && buttons[8].getText() .equals("X"))
        //label.setText("You win");
        return true;        
    else if (buttons[0].getText() .equals("O") && buttons[4].getText() .equals("O") && buttons[8].getText() .equals("O"))
        //label.setText("You lose");
        return true;

    if (buttons[2].getText() .equals("X") && buttons[4].getText() .equals("X") && buttons[6].getText() .equals("X"))
        //label.setText("You win");
        return true;        
    else if (buttons[2].getText() .equals("O") && buttons[4].getText() .equals("O") && buttons[6].getText() .equals("O"))
        //label.setText("You lose");
        return true;        

    return false;
}

主方法：

public static void main(String args[])
{   //creating an array of buttons, and adding ActionListeners to them using a for loop.
    JLabel label = new JLabel("Welcome to the tic tac toe game");
    JButton[] buttons = new JButton[9];
    Random rn = new Random();


    for( int i=0; i<9; i++)
    {
         final int temp = i;
         buttons[i] = new JButton("--");
         buttons[i].addActionListener(new ActionListener()
             {


                public void actionPerformed(ActionEvent e)
                {
                    if (buttons[temp].getText() .equals("--"))
                            {
                        String userx = "X";
                        buttons[temp].setText(userx);
                        if(checkWinner("X", buttons))
                        {
                            label.setText("You Win");
                        }
                        buttons[rn.nextInt(9)].setText("O");
                        if (checkWinner("O", buttons)){
                            label.setText("You Lose");
                        }

                        if (label.getText() .equals("You Lose") || label.getText().equals("You Win"))
                        {
                            for(int a =0; a<9; a++)
                            { buttons[a].setText("--");

                            }
                        label.setText("Welcome to the tic tac toe game");
                        }           
                };{
                    }}}         
             );};

    buttons[4].setText("O");

    Box left = createVerticalBox(); 
    for( int j=0; j<3; j++)
    {
        left.add(buttons[j]);

    }
    Box middle = createVerticalBox();
    middle.add(buttons[3]);

    Box center = createHorizontalBox();
    center.add(createGlue());
    center.add(buttons[4]);
    center.add(createGlue());
    middle.add(center);
    middle.add(buttons[5]);

    Box right = createVerticalBox();
    for(int k=6; k<9; k++)
    {
        right.add(buttons[k]);
    }

    Box panel = createHorizontalBox();
    panel.add(left);
    panel.add(middle);
    panel.add(right);

    Box top = createVerticalBox();
    top.add(label);
    top.add(panel);

    JFrame frame = new JFrame();
    frame.add(top);
    frame.pack();
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setVisible(true);
}
}

- Jerum

---

他放在同一个位置，因为你只是随机放置它，而没有事先检查那里是否已经写有其他内容。 - Alex Weitz

解决这两个 bug 后，你应该考虑重构/优化你的代码。甚至可以将它发布到“代码审查”中以获取一些灵感。 - slartidan

或者只需学习现有的井字棋实现及其评论。 - 200_success

Answer 1

为了确保您不会覆盖现有的标签，您可以在设置新标签之前检查其字符串：

    boolean notSet = true;
    while (notSet)
    {
        int buttonToSet = rn.nextInt(9);
        if (buttons[buttonToSet].getText() == "--" ){
            buttons[buttonToSet].setText("0");
            notSet=false;
        }

    }

关于游戏重置： 存在逻辑漏洞。你在游戏结束后立即重新开始游戏，并将标签设置回“欢迎来到井字棋游戏”。 相反，我建议使用JOptionPane.showMessageDialog(null,"You Lose/Win");来告知用户结果。 您可以通过在监听器内声明布尔变量并相应地更改其状态来监视游戏状态。

buttons[i].addActionListener(new ActionListener() {
Boolean endGame = false;
public void actionPerformed(ActionEvent e) {
    if (buttons[temp].getText().equals("--")) {
        String userx = "X";
        buttons[temp].setText(userx);
        if (checkWinner("X", buttons)) {
            endGame = true;
            JOptionPane.showMessageDialog(null,"You Win");

        }
        buttons[rn.nextInt(9)].setText("O");
        if (checkWinner("O", buttons)) {
            endGame = true;
            JOptionPane.showMessageDialog(null,"You Lose");

        }

        if (endGame) {
            endGame = false;
            for (int a = 0; a < 9; a++)
                buttons[a].setText("--");

            label.setText("Welcome to the tic tac toe game");
        }
    }
    ;
    {
    }
    }
}

- Alex Weitz

---

这似乎只回答了问题的一半。仍未解决的问题是：一旦有赢家，它就会重置并且不显示“你赢/输”的横幅。 - slartidan

空的代码块语句在结尾是否有意义？因为这样很凌乱... - AxelH