Question

我正在尝试迭代一个对象列表，这样在迭代后我可以从中创建一个Map of String and List<String>。

下面是我的Machine类，它有dataCenter，hostName和hostId。

public class Machine { 

    private String dataCenter;
    private String hostName;
    private String hostId;

    // getters here

    public Machine(String dataCenter, String hostName, String hostId) {
        // some code here

    }
}

如果您看到我的下面的代码，我有一个machines的列表，其中我有6个机器对象。然后迭代machines对象并从中创建一个新的Map。

    Map<String, List<String>> holder = new LinkedHashMap<String, List<String>>();

    List<Machine> machines = new ArrayList<Machine>();

    Machine machine1 = new Machine("DC1", "machineA", "hostA");
    Machine machine2 = new Machine("DC1", "machineB", "hostB");
    Machine machine3 = new Machine("DC2", "machineC", "hostC");
    Machine machine4 = new Machine("DC2", "machineD", "hostD");
    Machine machine5 = new Machine("DC3", "machineE", "hostE");
    Machine machine6 = new Machine("DC3", "machineF", "hostF");

    machines.add(machine1);
    machines.add(machine2);
    machines.add(machine3);
    machines.add(machine4);
    machines.add(machine5);
    machines.add(machine6);

    // some other code here

    // iterate machines object
    for (Machine machine : machines) {
        LinkedList<String> ppMachine = new LinkedList<String>();

        // below code doesn't work as it shows only last machine for that Datacenter
        ppMachine.add(machine.getHostId());
        holder.put(machine.getDatacenter(), ppMachine);
    }

    System.out.println(holder);

新地图应该是这样的 -

DC1 - hostA,hostB
DC2 - hostC,hostD
DC3 - hostE,hostF

此处DC1是新地图的关键，hostA,hostB是该地图的字符串值列表。与其他人相似。

但不知何故，在我的持有人地图中，我总是看到该数据中心的最新机器价值，而不是该数据中心的所有机器。

我在某种程度上做了非常愚蠢的错误。

Answer 1

    LinkedList<String> ppMachine = new LinkedList<String>();

    // below code doesn't work as it shows only last machine for that Datacenter
    ppMachine.add(machine.getHostId());
    holder.put(machine.getDatacenter(), ppMachine);

您正在替换与machine.getColo()相关联的整个列表，而不是添加新条目。

可能最简单的解决方法是

    LinkedList<String> dynMachine = holder.get(machine.getDatacenter());
    if (dynMachine == null) {
      dynMachine = new LinkedList<String>();
      holder.put(machine.getDatacenter(), dynMachine);
    }
    dynMachine.add(machine.getDatacenter());

Answer 2

那是因为你正在为每次迭代创建一个新列表，这就是你如何实现它：

    // iterate machines object
    for (Machine machine : machines) {
        LinkedList<String> dynMachine = holder.get(machine.getColo());
        if (null == dynMachine) {
           dynMachine = new LinkedList<String>();
           holder.put(machine.getColo(), dynMachine);
        }
        dynMachine.add(machine.getHostId());
    }

如何迭代对象列表以从中创建新Map？

2 个答案: