2017-05-09 65 views
1

我不确定散列表内容如何改变。散列表替换值

下面是我的代码:

public static void main(String args[]){ 

    HashMap<String, Double> map= new HashMap<>(); 
    map.put("com.x",30.00d); 
    map.put("com.y",70.00d); 
    map.put("com.z",70.00d); 

    List<String> readyServers=new ArrayList<>(); 
    readyServers.add("system-01"); 
    readyServers.add("system-02"); 

    returnServerMapping(2,170.00d,map,readyServers); 
} 



private static void returnServerMapping(int serverRequired, double totalExpectedExecutionTime, 
                    HashMap<String, Double> map, List<String> readyServers) { 
    Map<String,List<String>> testSuiteAndServerMap= new HashMap<>(); 
    List<String> suitesForServer = new ArrayList<>(); 

    Double executionTimePerServer = totalExpectedExecutionTime/serverRequired; 

    int currentServerNumber = 1; 
    Double currentTime = 0.0d; 
    for (Map.Entry<String, Double> entry : map.entrySet()) { 
     Double t = entry.getValue(); 

     System.out.println(currentServerNumber + " of " + serverRequired + ": " + entry.getKey()); 
     suitesForServer.add(entry.getKey()); 
     System.out.println("1.suites:"+suitesForServer); 

     currentTime += t; 

     if(currentServerNumber == serverRequired){// In the last case, let's get the all remaining suites & finally map this list with the last server 
     }else if(currentTime >= executionTimePerServer && currentServerNumber < serverRequired){ 
      System.out.println("Putting data into the map : server:"+readyServers.get(currentServerNumber-1)+"="+suitesForServer); 
      System.out.println("2.suites:"+suitesForServer); 
      testSuiteAndServerMap.put(readyServers.get(currentServerNumber-1), suitesForServer); 
      suitesForServer.removeAll(suitesForServer); 
      System.out.println("3.suites:"+suitesForServer); 
      currentServerNumber++;currentTime = 0.0d; 
     } 
    } 
    System.out.println("4.suites:"+suitesForServer); 
    testSuiteAndServerMap.put(readyServers.get(serverRequired-1), suitesForServer); 
    System.out.println("Final servers & suites:"+testSuiteAndServerMap); 
} 

和输出

1 of 2: com.z 
1.suites:[com.z] 
1 of 2: com.y 
1.suites:[com.z, com.y] 
Putting data into the map : server:system-01=[com.z, com.y] 
2.suites:[com.z, com.y] 
3.suites:[] 
2 of 2: com.x 
1.suites:[com.x] 
4.suites:[com.x] 
Final servers & suites:{system-01=[com.x], system-02=[com.x]} 

所以现在的问题是,当系统01 = [com.z,com.y]那么怎么来它是在最后设置为system-01 = [com.x]?

编辑:调试后,我发现hashmap的内容 之前suitesForServer.removeAll(suitesForServer); => key-system-01 & value 0-com.z,value 1-com.y

after suitesForServer.removeAll(suitesForServer); => key-system-01 & value size = 0

已解决:HashMap指的是列表。一旦列表内容被改变(在任何地方),hashmap也修改了内容。为了解决这个问题,我使用了一个只拷贝列表内容的浅拷贝。

List<String> suites= new ArrayList<>(suitesForServer); 
       testSuiteAndServerMap.put(readyServers.get(currentServerNumber-1), suites); 
+3

调试程序。然后你就可以一步一步看到发生了什么。 – f1sh

+3

简化了代码,并保留了您认为自己遇到问题的唯一部分。 – Ultraviolet

+1

我相信你的问题可能与此:'currentServerNumber-1' 我可能是错的,因为我没有测试它,但从一目了然似乎可能是罪魁祸首 – DCON

回答

1

您的地图的值始终是相同的列表实例。 将清单放入地图else后,您可以通过suitesForServer.removeAll(suitesForServer)清除地图。 我建议在您的if中将com.x添加到列表中。

在每次迭代中你应该创建一个新的List实例:

for (Map.Entry<String, Double> entry : map.entrySet()) { 
    List<String> suitesForServer = new ArrayList<>(); 
    ... 
+0

谢谢!即使我在循环内初始化它,在循环之后我仍然想要保留东西。相反,我使用列表 suites = new ArrayList <>(suitesForServer);内部在else循环中。这个浅表有助于hashmap不引用在顶部初始化的任何对象。 –