如何修复在另一个类中被修改的私有列表

Question

对于一个学校项目，他们要求我们制作dijkstra的算法。我有类：节点，边缘，图形和Dijkstra使用图形.get节点（返回列表），并将其存储在查找路径（在dikstra）中的局部变量中使用查找路径.java图形.java。public List<Node> getNodes() {    return nodes;}public void findPath (Node s, Node d) {        System.out.println("nodeSize at begining of findpath : "+graph.getNodes().size());        List<Node> nodes = graph.getNodes();        dijkstraTable = new Map[nodes.size()];        //verify paramaters        if(!nodes.contains(s)) throw new InvalidParameterException("the source node is not valid");        if(!nodes.contains(d)) throw new InvalidParameterException("the destination node is not valid");        if(s == null || d == null ) throw new NullPointerException("s or d are null");        Edge edge0 = new Edge(s,s,0);        Map<Node,Edge> map = new HashMap<Node,Edge>();        map.put(s, edge0);        int i =0;        dijkstraTable[i] = map;        //iteration > 0         while(!nodes.isEmpty()) {            i++;            if (i==nodes.size()) break;            Map<Node,Edge> newMap = new HashMap<Node,Edge>();            map = dijkstraTable[i-1]; //last iteration map             //get minimum last iteration            Node min = getMinimum(map);            //mark it as "visited"             nodes.remove(min);            //add it to the path    }输出是节点查找路径开头的大小 ： 7节点循环中大小：6节点循环中大小：5节点循环中大小：5节点循环中大小：5节点查找路径末尾的大小 ： 5每次都应该是7分

Answer 1

您的问题就在这里：

public List<Node> getNodes() {

    return nodes;

}

...

List<Node> nodes = graph.getNodes();

您正在修改方法中的“私有”列表。如果要避免修改，请使用 int 或在其中创建新列表并传递给列表构造函数。findPathCollections.unmodifiableList(nodes)getNodes()findPathgraph.getNodes()

Answer 2

改变

List<Node> nodes = graph.getNodes();

自

List<Node> nodes = new ArrayList<>(graph.getNodes());

这将在 findPath 方法中创建列表的本地副本。

或者你可以改变

public List<Node> getNodes() {

    return nodes;

}

自

public List<Node> getNodes() {

    return new ArrayList<>(nodes);

}

这将返回 getNodes（） 方法中列表的副本。