Question

我似乎在下面的A * pathfinding实现中有一个错误，我实现的是基于找到的伪造的代码here。

function NodeList() {
    this.nodes = [];

    this.add = function(givenNode) {
        for(var i = 0; i<this.nodes.length; i++) {
            if(this.nodes[i].f <= givenNode.f) {
                this.nodes.splice(i, 0, givenNode);
                return;
            }
        }

        this.nodes.push(givenNode);
    }

    this.pop = function() {
        return this.nodes.splice(this.nodes.length-1, 1)[0];
    }

    this.getNode = function(givenNode) {
        for (var i = 0; i < this.nodes.length; i++) {
            if (this.nodes[i].pos.x == givenNode.pos.x && this.nodes[i].pos.y == givenNode.pos.y) {
                return this.nodes.splice(i, 1)[0];
            }
        }

        return -1;
    }

    this.hasNode = function(givenNode) {
        for (var i = 0; i < this.nodes.length; i++) {
            if (this.nodes[i].pos.x == givenNode.pos.x && this.nodes[i].pos.y == givenNode.pos.y) {
                return true;
            }
        }

        return false;
    }

    this.length = function() {
        return this.nodes.length;
    }
}

function PathNode(pos, f, g, h) {
    this.pos = pos;
    this.f = f;
    this.g = g;
    this.h = h;
}

function FindPath(start, goal) {
    var x_array = [0, -1, -1, -1, 0, 1, 1, 1];
    var y_array = [1, 1, 0, -1, -1, -1, 0, 1];

    var open_list = new NodeList();
    open_list.add(new PathNode(start, start.Manhattan(goal) * 10, 0, start.Manhattan(goal) * 10));
    var closed_list = new NodeList();

    while(open_list.length() > 0) {     
        var currentNode = open_list.pop();

        if(currentNode.pos.x == goal.x && currentNode.pos.y == goal.y) {
            var path = [];
            var curNode = currentNode;

            while(true) {
                path.push(curNode);
                curNode = curNode.parent;
                if(curNode == undefined) break;
            }

            return(path);
        }

        closed_list.add(currentNode);

        for(var i=0; i<8; i++) {
            var neighbor = new PathNode(new Vector2(currentNode.pos.x + x_array[i], currentNode.pos.y + y_array[i]), 0, 0, 0);

            if(map.tiles[neighbor.pos.x][neighbor.pos.y].blocked == true) {
            canContinue = false;
        }

        for(var j=0; j<objects.length; j++) {
            if(objects[j].blocks == true && objects[j].position.x == neighbor.pos.x && objects[j].position.y == neighbor.pos.y) canContinue = false;
        }

        if(closed_list.hasNode(neighbor)) continue;
        if(!canContinue) continue;

            if(open_list.hasNode(neighbor)) { // if open_list contains neighbor, do this:
                neighbor = open_list.getNode(neighbor);
                neighbor.parent == currentNode;
                neighbor.g = currentNode.g + 10;
                neighbor.h = neighbor.pos.Manhattan(goal) * 10;
                neighbor.f = neighbor.g + neighbor.h;
                open_list.add(neighbor);
            } else { // otherwise it's not on the open list, do this:
                if(neighbor.g < currentNode.g) {
                    neighbor.parent = currentNode;
                    neighbor.g = currentNode.g + 10;
                    neighbor.f = neighbor.g + neighbor.h;
                }

                open_list.add(neighbor);
            }
        }   
    }
}

我一定做错了，因为代码会循环进入无限循环，并且每当我运行它时都会崩溃浏览器。有人可以指出我的错误吗？

Answer 1

我会在找到它们时用点更新这个答案。

首先，我认为无法逃脱你的外环环境。你有一个console.log而不是return语句，你在这里发布的例子中有。 console.log(path);代替return path;
您没有检查已关闭的节点的已关闭列表。因此，一旦您评估了打开列表中节点的状态，就会将其推送到关闭列表中，但您不会对该列表执行任何操作。没有什么可以阻止您再次将已关闭列表上的节点添加到打开列表中。您只是检查打开列表以防止多次添加同一节点。（虽然您在此处发布的示例代码显示您是）

这些东西的组合看起来会无限次地产生相同的路径。

另外要指出的是，您的示例代码不正确地缩进，因此看起来很多代码不在8个邻居检查循环中。

Answer 2

您似乎忘记从搜索中排除不可访问和已关闭的节点。

A * Pathfinding实现错误

2 个答案: