我该如何使用“或”在 Java 的 if 语句中而不需要重新输入整个表达式？

Question

我正在为类的扫雷项目编写代码，一种方法是 numAdjMines，它计算数组中某个单元格周围的地雷，每种类型的单元格都有不同的值，例如地雷为 -2，而地雷上带有标志是-4。我只想编写一个 if 语句，但最终不得不编写相同的代码两次，最后使用不同的值。

if (row >= 1 && col >= 1 && boardArray[row - 1][col - 1] == MINE)
    {
        adjMines = adjMines + 1;
    }
if (row >= 1 && col >= 1 &&
            boardArray[row - 1][col - 1] == FLAGGED_MINE)
    {
        adjMines = adjMines + 1;
    }

我尝试使用||或 和 写作 || boardArray[row-1][col-1] == FLAGGED_MINE 在第一个的末尾，但随后忽略了检查行和列的开头。有没有一种简短紧凑的方法可以让我编写这段代码？

Answer 1

您的上述代码实际上可以压缩为单个 IF 语句，但是我认为您的实际代码包含更多语句，否则您已经这样做了。

简化此类代码的最简单方法是将其分成两层 IF 语句。外层是一般情况，内层是特殊情况。

if (row >= 1 && col >= 1 ){

    int cell = boardArray[row - 1][col - 1];

    if( cell == MINE ){
        // Code here
    }

    else if( cell == FLAGGED_MINE )
    {
        // Code here
    }

}

Answer 2

为了避免重复，您可以使用嵌套的 if 语句，即两个条件都依赖于 row & col >= 1，所以将其放入它自己的语句中。

然后我猜你想避免多次从数组中取出值，所以最好的办法是将其分配给一个变量。这在运行时可能不会更有效，但是看起来更好。

if (row >= 1 && col >= 1)
{
    int value = boardArray[row - 1][col - 1];
    if (value == MINE || value == FLAGGED_MINE)
    {
        adjMines = adjMines + 1;
    }
}

Answer 3

您可以使用括号对条件进行分组

if(row >= 1 && col >= 1 && 
  (boardArray[row - 1][col - 1] == MINE
    || boardArray[row - 1][col - 1] == FLAGGED_MINE))
{
    adjMines = adjMines + 1;
}

Answer 4

我不确定您要避免重复哪一部分，但您可以通过嵌套 if 语句来避免第一部分的重复。

if( row >= 1 && col >= 1 ) {
    if( boardArray[row - 1][col - 1] == MINE || 
        boardArray[row - 1][col - 1] == FLAGGED_MINE ) {

        adjMines = adjMines + 1;
    }
}

我会更进一步，将内部 if 语句设置为方法调用。

Answer 5

我不确定我是否完全理解这个问题，但以下代码更紧凑并实现相同的目标：

if (row >= 1 && col >= 1 && 
    (boardArray[row - 1][col - 1] == MINE || boardArray[row - 1][col - 1] == FLAGGED_MINE)) {
    adjMines++;
}

我不确定写入主数组对于标记的地雷来说是否是一个好主意。我想玩家可能会错误地标记一个空闲单元格，这会将单元格标记为已标记的地雷，而您将不知道地雷是否确实存在。也许您有处理此问题的逻辑，但最好保留原始板（数据结构）以供将来参考。

Answer 6

for(    int r : array( row-1, row, row+1 ))
    for(int c : array( col-1, col, col+1 ))
    {
        if(r==row && c==col) // center
            continue; 

        if(0<=r&&r<ROWS && 0<=c&&c<COLS) // within bounds
        {
            int state = boardArray[r][c];
            if(state==MINE||state==FLAGGED_MINE)
                adjMines++;
        }
     }

int[] array(int... ints){ return ints; }

所以你不必编写 8 个单独的案例。这没什么大不了的，但如果你有一个 3D 地雷......