java - 不好的做法？省略循环条件

Question

我正在研究一个国际象棋游戏，现在我正在制定一种方法来检查国王的方向，如果发现威胁则返回 true，如果没有发现威胁则返回 false。 此方法采用两个整数来增加 x 和 y。并且此方法将从起始位置开始，沿给定的方向((1,0)向上，(1,1)向上和向右，(0,1)向下)。

我的问题是，这个方法用for循环检查路径，最终会碰到棋局的边界，找到一个障碍，或者找到一个棋子。无论哪种方式，这就是 for 循环应该如何结束。但是什么条件会进入 for 循环呢？ 我觉得把它排除在外可能是不好的做法，但唯一合乎逻辑的条件是确保 x 和 y 都在 1-8 的范围内。但这会使代码看起来像: for(int i = begx + xdiff, j = begy + ydiff; (i < 8 && i > 1) && (j < 8 && j > 1); i += xdiff, j += ydiff) { .. }

这将是我看到的唯一其他选择，而且这本身对我来说也是不好的做法。因为，它看起来过于复杂和拥挤。而且这个条件永远不应该被打破，那么如果它永远不会成为循环中断的原因，我为什么要把它放在那里呢？我不希望其他程序员阅读我的代码并认为这可能是他们必须注意的事情，实际上，我只是不需要那里的条件并将其放入进行双重检查。

这是整个方法，供引用:

public boolean incheckPath(Location l1, int xdiff, int ydiff) {
    int begx = l1.getX();
    int begy = l1.getY();

    String team = board[begx][begy].getTeam();

    for(int i = begx + xdiff, j = begy + ydiff; ; i += xdiff, j += ydiff) {
        if(board[i][j].getType() != ' ') {

            if(board[i][j].getType() == '#') {
                return false;
            }
            if(board[i][j].getTeam().equals(team)) {
                return false;
            }
            if(board[i][j].getType() == 'Q' || board[i][j].getType() == 'R') {
                return true;
            }

        }
    }

}

编辑 实现 while 循环已得到改进，但现在如何改进呢？有人建议使用 break and continue，所以我在我看到它的地方实现了它。但是，有人说我不应该循环返回。不做一个变量来保存返回值，这怎么可能？

public boolean incheckPath(Location l1, int xdiff, int ydiff) {
    int x = l1.getX();
    int y = l1.getY();

    String team = board[x][y].getTeam();

    while(true) {
        x += xdiff;
        y += ydiff;

        if(board[x][y].getType() == '#' || board[x][y].getTeam().equals(team)) {
            break;
        }

        if(board[x][y].getType() == 'Q' || board[x][y].getType() == 'R') {
            return true;
        }

    }

    return false;

}

Answer 1

如果您根据自己编写的其他代码自行确定 for 循环何时应该结束，那么您应该声明一个 boolean 值

boolean keepLo​​oping = true

当您希望循环停止时将其设置为 false。请注意，您必须在 for 循环内执行此操作。

现在将循环的条件设为您刚刚创建的变量。换句话说，它会说 ...;保持循环； ...

此外，我不确定您对 getType 的实现，它可能会返回一个字符。如果它返回一个字符串，您可能并不打算使用 == 来比较字符串。您想要使用 oneString.equals(anotherString) 来检查它们是否等价。使用 == 实际上是检查它们在内存中是否确实是同一个变量。可能这不是您想要的。

所以在那种情况下你的代码看起来像

if(board[i][j].getType().equals("Q")   ......

但如果 getType 返回一个字符，则本段不适用。