[英]Dijkstra Alogrithm gives wrong shortest path
I converted my C++
implementation of Dijkstra's Algorithm
to Java
. 我将Dijkstra's Algorithm
C++
实现转换为Java
。 When I ran my Java
code, I didn't get the expected output 当我运行Java
代码时,没有得到预期的输出
Expected from my C++
code: 预期从我的C++
代码:
Minimum distance for source vertex 0 to reach vertex 0 is 0
Minimum distance for source vertex 0 to reach vertex 1 is 4
Minimum distance for source vertex 0 to reach vertex 2 is 12
Minimum distance for source vertex 0 to reach vertex 3 is 19
Minimum distance for source vertex 0 to reach vertex 4 is 21
Minimum distance for source vertex 0 to reach vertex 5 is 11
Minimum distance for source vertex 0 to reach vertex 6 is 9
Minimum distance for source vertex 0 to reach vertex 7 is 8
Minimum distance for source vertex 0 to reach vertex 8 is 14
Actual from Java
code: 从Java
代码实际:
Minimum distance for source vertex 0 to reach vertex 0 is 0
Minimum distance for source vertex 0 to reach vertex 1 is 4
Minimum distance for source vertex 0 to reach vertex 2 is 2
Minimum distance for source vertex 0 to reach vertex 3 is 7
Minimum distance for source vertex 0 to reach vertex 4 is 9
Minimum distance for source vertex 0 to reach vertex 5 is 2
Minimum distance for source vertex 0 to reach vertex 6 is 1
Minimum distance for source vertex 0 to reach vertex 7 is 1
Minimum distance for source vertex 0 to reach vertex 8 is 2
I tried to hunt for a bug in my Java code, I double checked if I copied my C++ code properly, I didn't find anything different. 我试图在Java代码中寻找错误,我再次检查是否正确复制了C ++代码,但没有发现任何不同。
I already spend lots of time debugging my code. 我已经花了很多时间调试我的代码。
I don't understand what went wrong! 我不明白怎么了! I'm desperate for some help, thanks! 我迫切需要帮助,谢谢!
Code: 码:
import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;
import java.util.PriorityQueue;
import java.util.Queue;
public class Dijkstra
{
public static final int INF = 100000;
private static class IPair implements Comparable<IPair>
{
int first;
int second;
IPair(int first, int second)
{
this.first = first;
this.second = second;
}
public int compareTo(IPair that)
{
return this.first - that.first;
}
}
public static int[] dijkstra(List<IPair>[] adj, int source)
{
Queue<IPair> pq = new PriorityQueue<>();
int[] dist = new int[adj.length];
boolean[] visited = new boolean[adj.length];
Arrays.fill(dist, INF);
pq.add(new IPair(0, source));
dist[source] = 0;
while (!pq.isEmpty())
{
int u = pq.poll().second;
if (visited[u])
continue;
System.err.println(u);
visited[u] = true;
for (IPair pair : adj[u])
{
int v = pair.first;
int weight = pair.second;
if (dist[v] > dist[u] + weight)
{
dist[v] = dist[u] + weight;
pq.add(new IPair(dist[v], v));
System.err.println(Arrays.toString(dist));
}
}
}
return dist;
}
private static void addEdge(List<IPair>[] adj, int u, int v, int weight)
{
adj[u].add(new IPair(v, weight));
adj[v].add(new IPair(u, weight));
}
public static void main(String[] args)
{
int V = 9;
List<IPair>[] adj = new ArrayList[V];
Arrays.fill(adj, new ArrayList<IPair>());
addEdge(adj, 0, 1, 4);
addEdge(adj, 0, 7, 8);
addEdge(adj, 1, 2, 8);
addEdge(adj, 1, 7, 11);
addEdge(adj, 2, 3, 7);
addEdge(adj, 2, 8, 2);
addEdge(adj, 2, 5, 4);
addEdge(adj, 3, 4, 9);
addEdge(adj, 3, 5, 14);
addEdge(adj, 4, 5, 10);
addEdge(adj, 5, 6, 2);
addEdge(adj, 6, 7, 1);
addEdge(adj, 6, 8, 6);
addEdge(adj, 7, 8, 7);
int[] dist = dijkstra(adj, 0);
for (int i = 0; i < V; ++i)
System.out.println("Minimum distance for source vertex " + 0 + " to reach vertex " + i + " is " + dist[i]);
}
}
Arrays.fill(adj, new ArrayList<IPair>())
Is equivalent to: 等效于:
List<IPair> list = new ArrayList<>();
Arrays.fill(adj, list)
Which means you are storing the same List
object in all array elements. 这意味着您将在所有数组元素中存储相同的 List
对象。 When you change the List
object of adj[x]
, it changes the List
object of all adj
elements, since it is the same object. 当您更改List
的对象adj[x]
它改变了List
的所有对象adj
元素,因为它是同一个对象。
The solution is to store a new List
object in each adj
element: 解决方案是在每个adj
元素中存储一个新的List
对象:
//Arrays.fill(adj, new ArrayList<IPair>());
for (int i = 0; i < V; ++i) {
adj[i] = new ArrayList<>();
}
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.