使用Dijkstra算法的最短路径

Question

我正在恢复一个旧的家庭作业,我正在编写一个程序,其中包括使用Dijkstra算法在图形中找到最短路径.

我认为我在大多数时候都做对了,但是NullPointerException在执行时我一直在第58行if(currentNode.getAktuell()).

我一直在尝试几种解决方案,但似乎无法弄清楚什么是错误但在队列为空时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;
    }

Answer 1

我无法重建您告诉我们的 NullPointerException。正如 Leandro 指出的，问题可能出在 ListEdge 和 Graph 的实现上。

\n\n

我自己实现了这两个类来测试您的代码。

\n\n

我发现的唯一问题是最后创建结果列表的位置：

\n\n

for(int i=0;i<samling.size();i++){\n        if(samling.get(i).getNode()!=fr\xc3\xa5n){\n

\n\n

这将始终导致 aNullPointerException因为get()需要一个密钥，而在您的情况下，它是 a String，而不是 a int。要迭代地图，请使用类似的东西

\n\n

List<String> djikstaList = new ArrayList<String>();\nfor(String key : samling.keySet()){\n    if(samling.get(key).getNode()!=fr\xc3\xa5n){\n        System.out.println(samling.get(key).getNode());\n        djikstaList.add(samling.get(key).getNode());\n    }       \n}\n

\n\n

from此外，我假设您不想返回从到 的实际路径to，因此您需要添加一个 getter getFr\xc3\xa5n()，DijkstraObjekt然后像这样构建列表：

\n\n

   String fromNode = samling.get(to).getNode();\n   djikstaList.add(to);\n   while(fromNode != from){   \n       fromNode = samling.get(fromNode).getFr\xc3\xa5n();\n       djikstaList.add(fromNode);\n   }\n

\n\n

此后，列表将以相反的顺序包含完整路径（包括开始和结束节点）。

\n\n

如果需要，我可以发布我用于测试/调试的所有课程。

\n\n

干杯\ntannerli

\n