我目前正在恢复一项旧的家庭作业,我正在编写一个程序,其中包括使用 Dijkstra 算法在图中查找最短路径的功能。
我想我大部分时间都做对了,但是在执行 if(currentNode.getAktuell())
时,我在第 58 行不断收到 NullPointerException
。
我一直在来回尝试几种解决方案,但似乎无法找出问题所在,但是 prioQueue.poll();
返回 null
当队列是空的。我已经尝试处理最后一个 currentNode
,它最终变成 null 但未能找到可行的解决方案,所以我开始认为我在这里错过了一些东西。
如果熟悉 dijkstras 算法的人可以帮助我,我将不胜感激。该算法可能有更好的解决方案,但我只想帮助找出我编写的算法有什么问题,而不是使用其他人的算法“答案”。
public static List<String> shortestPath(Graph<String> graph, String från, String till){
//if(!pathExists(graph, från, till))
//return null;
PriorityQueue<DjikstraObjekt<String>> prioQueue = new PriorityQueue<DjikstraObjekt<String>>();
LinkedHashMap<String, DjikstraObjekt<String>> samling = new LinkedHashMap<String, DjikstraObjekt<String>>();
for(String bla : graph.getNodes())
samling.put(bla, new DjikstraObjekt<String>(bla, Integer.MAX_VALUE, null, false));
samling.get(från).updateVikt(0);
prioQueue.add(samling.get(från));
while(!samling.get(till).getAktuell())
{
DjikstraObjekt<String> currentNode = prioQueue.poll();
if(currentNode==null)
break;
if(currentNode.getAktuell())
continue;
currentNode.aktuellNod();
for(ListEdge<String> edge : graph.getEdgesFrom(currentNode.getNode()))
{
System.out.println("get edges from");
int nyVikt = edge.getVikt() + currentNode.getVikt();
DjikstraObjekt<String> toNode = samling.get(edge.getDest());
if(!toNode.getAktuell() && nyVikt < toNode.getVikt()) {
toNode.updateVikt(nyVikt);
toNode.setFrån(currentNode.getNode());
prioQueue.add(toNode);
}
}
}
List<String> djikstaList = new ArrayList<String>();
for(int i=0;i<samling.size();i++){
if(samling.get(i).getNode()!=från){
System.out.println(samling.get(i).getNode());
djikstaList.add(samling.get(i).getNode());
}
}
return djikstaList;
}
public class DjikstraObjekt<E> implements Comparable<DjikstraObjekt<E>> {
private E nod;
private int vikt;
private E frånNod;
private boolean aktuellNod=false;
public DjikstraObjekt(E nod, int vikt, E frånNod, boolean aktuellNod){
this.nod=nod;
this.vikt=vikt;
this.frånNod=frånNod;
this.aktuellNod=aktuellNod;
}
public E getNode() {
return nod;
}
public void updateVikt(int nyvikt){
vikt=nyvikt;
}
public int getVikt() {
return vikt;
}
public boolean getAktuell() {
return aktuellNod;
}
public void aktuellNod(){
aktuellNod=true;
}
public void setFrån(E från)
{
frånNod = från;
}
public int compareTo(DjikstraObjekt<E> other) {
return getVikt() - other.getVikt();
}
}
这是我的 listEdge 类:
public class ListEdge<E> {
private E dest;
private String namn;
private Integer vikt;
public ListEdge(E dest, String namn, Integer vikt){
this.dest=dest;
this.namn=namn;
this.vikt=vikt;
}
public E getDest(){
return dest;
}
public void ändraVikt(Integer nyVikt){
if(vikt<0)
throw new IllegalArgumentException();
vikt=nyVikt;
}
public String getNamn(){
return namn;
}
public int compareTo(ListEdge other) {
return this.vikt.compareTo(other.getVikt());
}
public int getVikt(){
return vikt;
}
public String toString(){
return "till " + dest + " med " + namn +" "+ vikt;
}
}
这些应该是我的 ListGraph 类中的相关方法:
public List<E> getNodes(){
List<E> temp = new ArrayList<E>();
for(E test : noder.keySet()){
temp.add(test);
}
return temp;
}
public List<ListEdge<E>> getEdgesFrom(E nod) {
List<ListEdge<E>> temp = new ArrayList<ListEdge<E>>();
if(noder.containsKey(nod)){
try{
for(Map.Entry<E, List<ListEdge<E>>> test : noder.entrySet()){
if(test.getKey().equals(nod)){
System.out.println(nod+" "+test.getKey());
for(ListEdge<E> e: test.getValue()){
temp.add(e);
}
}
}
}
catch(NoSuchElementException E){
}
}
return temp;
}
最佳答案
我无法重建您告诉我们的 NullPointerException。正如 Leandro 指出的那样,问题可能在于您对 ListEdge 和 Graph 的实现。
我自己实现了这两个类来测试您的代码。
我能找到的唯一问题是最后创建结果列表的地方:
for(int i=0;i<samling.size();i++){
if(samling.get(i).getNode()!=från){
这将始终导致 NullPointerException
,因为 get()
需要一个键,在您的情况下,它是一个 String
,而不是一个 整数
。要遍历 map ,请使用类似的东西
List<String> djikstaList = new ArrayList<String>();
for(String key : samling.keySet()){
if(samling.get(key).getNode()!=från){
System.out.println(samling.get(key).getNode());
djikstaList.add(samling.get(key).getNode());
}
}
此外,我假设您不想返回从 from
到 to
的实际路径,因此您需要添加一个 getter getFrån()
到 DijkstraObjekt
然后像这样构建列表:
String fromNode = samling.get(to).getNode();
djikstaList.add(to);
while(fromNode != from){
fromNode = samling.get(fromNode).getFrån();
djikstaList.add(fromNode);
}
在此之后,列表将以相反的顺序包含完整路径(包括开始和结束节点)。
如果需要,我可以发布我用于测试/调试的所有类。
干杯 坦纳利
关于java - 使用 Dijkstra 算法的最短路径,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/6585223/