Question

请帮助我了解我的代码在做什么错。我正在尝试使用BFS来解决问题的最短路径，但它要么给我-1要么给我2。应该给我6作为答案。我究竟做错了什么？这是问题所在：

给出一个国际象棋棋盘，找到一个骑士从给定来源到达给定目的地的最短距离（最小步数）。

例如，N = 8（8 x 8板），源=（7，0）目标=（0，7）

所需的最小步数为6

我的代码如下：

class Point {
    int x, y;
    public Point(int x, int y){
    this.x = x;
    this.y = y;
 }
}

class knightShortestPath {
    int N = 8;
    public static boolean visited[][];

public boolean isPositionValid(int x, int y){
    if( x < 0 || y < 0 || x > this.N || y > this.N){
        return false;
    }
    return true;
}

public void createChessBoard(int N) {
    this.N = N;
    visited = new boolean[this.N][this.N];
    for (int i = 0; i < this.N; i++) {
        for (int j = 0; j < this.N; j++) {
            visited[i][j] = false;
        }
    }
}

public int BFS(Point source, Point destination) {
    int row[] = {2, 2, -2, -2, 1, 1, -1, -1};
    int col[] = {1, -1, 1, -1, 2, -2, 2, -2};
    Queue<Point> queue = new LinkedList<>();
    queue.offer(source);
    visited[source.x][source.y] = true;
    int minimumNumSteps = 0;

    while (!queue.isEmpty()) {
        int size = queue.size();
        for (int i = 0; i < size; i++) {
            Point pt = queue.poll();
            if (pt.x == destination.x && pt.y == destination.y) {
                return minimumNumSteps;
            }
            for (int j = 0; j < size; j++) {
                Point next = new Point(pt.x + row[i], pt.y + col[j]);
                if (isPositionValid(pt.x + row[i], pt.y + col[j]) && !visited[i][j]) {
                    visited[i][j] = true;
                    queue.offer(next);
                }
            }
        }
        minimumNumSteps++;
    }
    return minimumNumSteps;
}


public static void main(String[] args) {
    knightShortestPath position = new knightShortestPath();
    position.createChessBoard(8);
    Point src = new Point(0,7);
    Point dest = new Point(7,0);
    System.out.println("The minimum number of steps are: " + position.BFS(src, dest)); //answer is 6
 }
}

Answer 1

第一件事：我不知道如何以负值结束。将minimumNumSteps初始化为0后，再也不会减少它。可能是溢出吗？对我来说似乎很奇怪..

除此之外，我看到两个问题：

两个for循环不正确。您当前在queue.size()上进行迭代。您要做的是遍历当前节点的所有子节点。
在for循环外轮询当前点。

所以：

while(!queue.isEmpty()) {
    Point pt = queue.poll();
    // check if it's target 
    // ...
    for (int i = 0; i < row.length; i++) {
        // ... 
        for (int j = 0; j < col.length; j++) {
            // ...
        }
    }
}

另一个注意事项：当队列为空并且您尚未达到目标时，将没有解决方案。当前，您正在返回一些可能会错误解释的值。

骑士的最短路径（BFS）

1 个答案: