Java: Connect 4 Winning 对角线

Question

我最近做了一个 Connect4 游戏，当我的 Connect4 向右对角线连接时，我的 Connect4 没有赢得游戏。并且它只适用于某些组合，当它连接到左边的对角线时。坐标:- 左上角:(0,0)，左下角:(5,0)，右上角:(0,6)，右下角:(5,6)。 Connect4 板是 6 x 7。

问题:向左对角线连接仅对某些组合有效。但是，没有任何连接是对角连接到正确的工作。

/** A method of winning diagonally towards the left side when playing connect 4 two player*/
/** Giving the new method with all the possible possibilities for a user to win diagonally-left */
public static void diagWinningLeft() {
    for (int x = 5; x > 2; x--) { // Checks to see if same colored pegs are lining diagonally to the left
        for (int y = 6; y > 3; y--) {
            if (board[x][y] == 1 && board[x - 1][y - 1] == 1 && board[x - 2][y - 2] == 1 && board[x - 3][y - 3] == 1) {
                JOptionPane.showMessageDialog(null, playerNames[0]+" has connected four diagonally-left in a row in " +(countForRed)+ " turns!");
                b.drawLine(x,y,x-3,y-3);
            }
            if (board[x - 1][y - 1] == 1 && board[x - 2][y - 2] == 1 && board[x - 3][y - 3] == 1 && board[x - 4][y - 4] == 1) {
                JOptionPane.showMessageDialog(null, playerNames[0]+" has connected four diagonally-left in a row in " +(countForRed)+ " turns!");
                b.drawLine(x,y,x-3,y-3);
            }

            if (board[x][y] == 2 && board[x - 1][y - 1] == 2 && board[x - 2][y - 2] == 2 && board[x - 3][y - 3] == 2) {
                JOptionPane.showMessageDialog(null, playerNames[1]+ " has connected four diagonally-left in a row in " +(countForYellow)+ " turns!");
                b.drawLine(x,y,x-3,y-3);
            }
        }
    }
}

/** Another method of winning diagonally towards the right side when playing connect 4 two player*/
/** Giving the new method with all the possible possibilities for a user to win diagonally-right*/
public static void diagWinningRight()  {
    for (int x = 0; x < 2; x++) { // Check to see if same colored pegs are lining diagonally to the right
        for (int y = 0; y < 3; y++) {
            if (board[x][y] == 1 && board[x + 1][y + 1] == 1 && board[x + 2][y + 2] == 1 && board[x + 3][y + 3] == 1) {
                JOptionPane.showMessageDialog(null, playerNames[0]+" has connected four diagonally-right in a row in " +(countForRed)+ " turns!");
            }
            if (board[x][y] == 2 && board[x + 1][y + 1] == 2 && board[x + 2][y + 2] == 2 && board[x + 3][y + 3] == 2) {
                JOptionPane.showMessageDialog(null, playerNames[1]+" has connected four diagonally-right in a row in " +(countForYellow)+ " turns!");
            }

        }
    }
}

Answer 1

请原谅我没有直接回答这个问题，但这些内容可以帮助您解决问题并最终获得更好的代码和更好的将来编写代码的能力。

将“if”条件的逻辑提取到一个单独的方法中，可以更轻松地单独考虑该逻辑，并让您独立于程序的其余部分对其进行测试。

所以代替:

  if (board[x][y] == 1 && board[x - 1][y - 1] == 1 && board[x - 2][y - 2] == 1 && board[x - 3][y - 3] == 1) {
       JOptionPane.showMessageDialog(...)
  }

...使用:

 if(isDiagonalLeft(x,y,1) { ... }

...和...

 boolean isDiagonalLeft(int x, int y, int player) {
      return board[x][y] == player && 
             board[x - 1][y - 1] == player  && 
             board[x - 2][y - 2] == player  && 
             board[x - 3][y - 3] == player 
 }

现在您可以对 isDiagonalLeft() 运行单元测试以确保其正常工作。也就是说，一个设置棋盘的小程序 只是 运行 isDiagonalLeft() 以确保它在各种情况下给出正确的答案。这感觉像是额外的工作，但大多数尝试过它的人都知道它可以通过及早发现错误来节省精力。

您所做的是将游戏逻辑与表示代码 (JOptionPane) 稍微分开，这样当您只想运行游戏逻辑时，表示代码就不会妨碍您。稍后在您的编程研究中，您将遇到将它们进一步分离的方法，例如模型- View - Controller 模型。

如果您需要在 Stack Overflow 上提问，像这样提取逻辑也很有帮助——通过将游戏逻辑与 Swing 分开，您可以向对 Swing 一无所知的潜在回答者开放问题。

而且，您可以重复使用此方法，每个玩家一次，而不是像您那样将逻辑复制到两个地方。

如果它不起作用，请使用您的 IDE 中的调试器逐步完成它。

现在您已经完成了这一步，您可以改进方法，让计算机代替程序员进行递减...

  boolean isDiagonalLeft(int x, int y, int player) {
       for(int i = 0; i<4; i++) {
           if(board[x-i][y-i] != player) {
               return false;
           }
       }
       return true;
  }

...您可以概括它，使其涵盖对角线的两个方向:

  boolean isDiagonal(int x, int y, int player, boolean direction) {
       int dirUnit = direction ? -1 : 1;
       for(int i = 0; i<4; i++) {
           if(board[x-i][y + dirUnit] != player) {
               return false;
           }
       }
       return true;
  }

...所以现在您可以在 4 个地方重复使用该方法；对于每个玩家和每个方向。

当你遇到它在你的 GUI 中不起作用的情况时，进行单元测试，按照它在 GUI 中的方式设置电路板，并在其上运行 isDiagonal() .如果测试通过，您就知道问题出在其他地方。如果测试失败，您可以使用调试器和方法代码，使其通过。