实现 Dijkstra 算法的问题

时间:2021-01-16 08:42:06

标签: java dijkstra

我正在尝试解决一个问题,我需要找到图中两点之间的最大距离。我编写了这段代码来实现 Dijkstra 算法(显然是通过将最小化更改为最大化),但在某些情况下它似乎不起作用,而且我无法弄清楚未处理的情况。任何人都可以帮助我指出我在实施中遗漏了什么,或者某些东西本质上是不正确的吗?

public double maxDistance(int n, int start, int end) {
    
    Map<Integer, List<Integer>> adjacencyList = getAdjacencyList(); // Adj. list in form of HashMap 
    Map<String, Double> distanceMap = getDistanceMap(); // <Edge, Double> (value is distance)
    Queue<Integer> bfsQueue = new LinkedList<>(); // queue to do a BFS
    
    boolean[] visited = new boolean[n];
    double[] distance = new double[n];
    
    bfsQueue.add(start);
    
    while (!bfsQueue.isEmpty()) {
        int node = bfsQueue.poll();
        
        List<Integer> neighbors = adjacencyList.getOrDefault(node, new ArrayList<>());
        for (int neighbor: neighbors) {
            if (visited[neighbor]) continue;
            bfsQueue.add(neighbor);
            double edgeLength = distanceMap.get(new Edge(node, neighbor));
            double newLength = distance[node] + edgeLength;
            if (newLength > distance[neighbor]) {
                distance[neighbor] = newLength;
            }
        }
        visited[node] = true;
    }

    return distance[end];
}

1 个答案:

答案 0 :(得分:1)

首先,Dijkstra 算法找到最短路径,因此它应该是 minDistance,而不是 maxDistance

接下来,要实现广度优先搜索,您需要一个已排序的数据结构。您的 bfsQueue 目前只是一个 LinkedList。使用链表,您可以按插入顺序遍历项目。但是在 Dijkstra 的算法中,始终处理下一个最近的邻居很重要。因此通常使用优先级队列来实现。

举个例子,为什么这会有所不同:图片你想从 A 到 B。有两条可用的路线,一条由两条边组成的长路线,一条由 4 条边组成的较短路线。在您的情况下,您基本上通过自开始以来的边数而不是距离来扩展图形。因此,您将首先找到由两条边组成的路线,即使这是较慢的一条。

如果您使用 PriorityQueue,请注意减少邻居的距离/优先级(if (newLength > distance[neighbor]) { distance[neighbor] = newLength; })将不起作用,因为优先级队列不会自动重新排序.您必须首先从优先级队列中删除邻居,更新距离,然后将其重新插入优先级队列中,以便将其分类到正确的位置。

相关问题