1

我创建了一个简单的游戏，玩家可以与电脑对战。我在转弯的时间上遇到了问题，计算机应该是第一个移动的，而不是真正的玩家应该通过点击LeftChoice或RightChoice按钮来移动。

这是我的代码问题：

public class GameForm extends javax.swing.JFrame {

/**
 * Creates new form GameForm
 */
final int SIZE = 10;
int CurrentSize = 10;
int PC_SUM=0;
int HUMAN_SUM=0;
boolean PC_TURN = true;
int[] RandArr = new int[SIZE];
public GameForm() {
    initComponents();
}
public void init(){
    for(int i = 0 ; i<SIZE;i++){
        RandArr[i] = (int)(Math.random()*100)+1;
    }
    jTextField3.setText("");
    jTextField4.setText(Integer.toString(PC_SUM));
    jTextField5.setText(Integer.toString(HUMAN_SUM));
}
public void HUMAN_updateLeft(){
    HUMAN_SUM+=RandArr[0];
    jTextField5.setText(Integer.toString(HUMAN_SUM));
    jTextField1.setText(Arrays.toString(RandArr));
    CurrentSize--;
    int [] NewRand = new int[CurrentSize];
     for(int i = 1 ; i<=CurrentSize;i++){
        NewRand[i-1] = RandArr[i];
    }
     RandArr = NewRand;
     jTextField2.setText(Arrays.toString(RandArr));
     PC_TURN = true;
}
public void HUMAN_updateRight(){
    HUMAN_SUM+=RandArr[CurrentSize-1];
    jTextField5.setText(Integer.toString(HUMAN_SUM));
    jTextField1.setText(Arrays.toString(RandArr));
    CurrentSize--;
    int [] NewRand = new int[CurrentSize];
     for(int i = CurrentSize-1 ; i>=0;i--){
        NewRand[i] = RandArr[i];
    }
     RandArr = NewRand;
     jTextField2.setText(Arrays.toString(RandArr));
     PC_TURN = true;
}
public static boolean WhoIsBigger(int[] arr){
int even=0,odd=0;

for(int i=0;i<arr.length;i+=2){
    if(i%2==0){
    even+=arr[i];
    odd+=arr[i+1];
    }
    else{
        odd+=arr[i];
        even+=arr[i+1];
    }
  }
return even>odd;
}
public void PC_updateLeft(){
    PC_SUM+=RandArr[0];
    jTextField4.setText(Integer.toString(PC_SUM));
    jTextField1.setText(Arrays.toString(RandArr));
    CurrentSize--;
    int [] NewRand = new int[CurrentSize];
     for(int i = 1 ; i<=CurrentSize;i++){
        NewRand[i-1] = RandArr[i];
    }
     RandArr = NewRand;
     jTextField2.setText(Arrays.toString(RandArr));
}
public void PC_updateRight(){
    PC_SUM+=RandArr[CurrentSize-1];
    jTextField4.setText(Integer.toString(PC_SUM));
    jTextField1.setText(Arrays.toString(RandArr));
    CurrentSize--;
    int [] NewRand = new int[CurrentSize];
     for(int i = CurrentSize-1 ; i>=0;i--){
        NewRand[i] = RandArr[i];
    }
     RandArr = NewRand;
     jTextField2.setText(Arrays.toString(RandArr));
}
public void PC_TURN(){
    if(WhoIsBigger(RandArr))
        PC_updateLeft();
    PC_updateRight();
}
public void FullGame(){
    while(RandArr.length>0){
        if(PC_TURN){
            PC_TURN();
            PC_TURN = false;
        }
    }
}
  //start button listener
private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) {                                         
    init();
    jTextField2.setText(Arrays.toString(RandArr));
    jTextField1.setText("-");
    jButton1.setEnabled(false);
    FullGame();
}                                        
   //left button listener
private void jButton2ActionPerformed(java.awt.event.ActionEvent evt) {                                         
HUMAN_updateLeft();
}                                        
//right button listener
private void jButton3ActionPerformed(java.awt.event.ActionEvent evt) {                                         
HUMAN_updateRight();
}

我怎么知道真正的玩家已经做出了他的动作，所以它会改变PC_TURN并且True游戏会继续？

这是我的 GUI 外观的图片：

4

3 回答 3

4

首先，在命名变量时，您应该遵循 Java 命名约定。就目前而言，它PC_TURN看起来好像是一个常数，尽管它不是一个常数，因为您正在更改它的值。所以一个更合适的名字应该是pcTurn。

您似乎也有一个名为的方法PC_TURN()，我假设它是导致计算机轮流执行某些操作的方法。因此，如果将其命名为类似的描述性名称会更好takePCTurn()。注意始终使用 camelCase。名称开头的大写字母应保留给类和接口，而不是变量或方法。

您的方法名称也是如此

public void FullGame()

应该写成

public void fullGame()

遵守这样的编码约定可以让其他人更容易阅读和理解你的代码，并保持一切整洁。养成一个好习惯。:)

我不知道你的左右按钮在哪里被声明或者你给它们起什么名字，但是你需要为每个按钮添加一个事件监听器，当它们被点击时会导致一些事情发生。我也不确定RandArr.length > 0. 你真的不需要在这里循环，因为这是一个带有 GUI 的应用程序，它永远不会关闭，除非你明确告诉它（例如通过单击关闭按钮）。所以我只会给你一个通用的解决方案。

您基本上希望玩家轮流触发计算机轮流，直到满足某些游戏结束条件。

例子：

//Assuming this is called when the Start button is clicked
public void fullGame() {
    takePCTurn();
}

public void takePCTurn() {
    //Do PC's turn logic
    //You might want to check if the Computer Won the game here        
}


class PlayerTurnListener implements ActionListener() {
    public void actionPerformed(ActionEvent e) {
        //Do Player's turn logic            
        //You might want to check if the Player Won the game here
        takePCTurn();
    }
}

//We can create an instance of our listener and add it to both buttons
PlayerTurnListener playerTurnListener = new PlayerTurnListener();

leftButton.addActionListener(playerTurnListener);
rightButton.addActionListener(playerTurnListener);

所以发生的第一件事fullGame()是由您的开始按钮调用，然后调用takePCTurn();导致计算机轮到它。现在在玩家点击左或右按钮之前什么都不会发生。当他们这样做时，会调用PlayerTurnListener'sactionPerformed()方法，您可以在其中执行一些逻辑，然后takePCTurn();再次调用。

冲洗并重复直到游戏结束。

希望有帮助:)

于 2015-12-01T17:31:02.087 回答
2

您应该将 a 附加clickListener到左右按钮上。然后，当用户单击一个该事件将被触发。您应该在此处将 PC_TURN 更改为 true 并在需要时重新运行循环。

例子：

JButton yourButton = new JButton("Your Button");
yourButton.addActionListener(new ActionListener()
{
  public void actionPerformed(ActionEvent e)
  {
    PC_TURN = true;
    // extra code
  }
});
于 2015-12-01T16:36:36.167 回答
1

如果这是 Swing 或其他 GUI，那么您将需要摆脱 while 循环。

请记住，GUI 程序是非线性的，并且是事件驱动的，因此，与其使用可能完全影响您的 GUI 的限制性 while 循环，不如根据轮到谁来更改程序的状态——有一个变量来指示轮到谁是——然后根据状态改变程序的行为。

如果您需要更具体的帮助，那么您将需要提出一个更完整的问题，包括一个包含足够代码和解释的问题，以便我们更好地理解您的问题（但不要太多代码）。

关于您发布的代码：

  • 是的，绝对摆脱那个while循环。
  • 你的代码很难理解，主要是因为你的变量命名很差。与其使用变量名称，不如使用具有jTextField1逻辑意义的名称，使您的代码“自调试”，例如humanScoreFieldandcomputerScoreField等。
  • 还要学习并遵循标准的 Java 命名实践，包括让所有变量以小写字母开头，以大写字母开头的类，使用 camelCase，除了所有的常量都是大写的。
  • 告诉更多关于这段代码应该做什么，按钮和 JTextfields 代表什么，你想要什么行为等等......
于 2015-12-01T17:07:34.817 回答