Question

public Map<String, List<String>> getContactMap() {
    Set<String> keys = contactMap.keySet(); //Get Key Set
    for(String key: keys){
        List<String> a=contactMap.get(key); //Get List for each Key
        a.forEach(System.out::println);     //Print the list
    }
    return null;
}

我有一个Hashmap，单个键有多个值。我使用此代码来打印这些值，但这似乎不起作用。我做错了什么？

我只将最后插入的值作为输出。

这是我的addContact方法：

private Map<String, List<String>> contactMap = new HashMap<String, List<String>>();

@Override
public void addContact(String name, List<String> list) {
    // TODO Auto-generated method stub
    contactMap.put(name, list);
}

这是我的主要（）：

List <String> list= new ArrayList<String>();
    list.add("0321564988");
    list.add("7891268429");
    contacts.addContact("Name1",list);
    list.clear();
    list.add("1891219122");
    list.add("9495198929");
    contacts.addContact("Name2",list);
    list.clear();
    list.add("8949219912");
    contacts.addContact("Name3",list);

这是我想要的输出：

Name1   Phone1
        Phone2
        Phone3
Name2   Phone1
        Phone2
        .....

Answer 1

您的代码：

List <String> list= new ArrayList<String>();
list.add("0321564988");
list.add("7891268429");
contacts.addContact("Name1",list);
list.clear();
list.add("1891219122");
list.add("9495198929");
contacts.addContact("Name2",list);
list.clear();
list.add("8949219912");
contacts.addContact("Name3",list);

错了。

您正在重复使用相同 List。

看一个简单的例子：

List <String> list = new ArrayList<String>();
List <String> list2 = list;

list.add("0321564988");
list.add("7891268429");
list.clear();

list2.add("1891219122");
System.out.println(list2);

输出：

1891219122

因为在Java中，List是对包含List的内存位置的引用。 Java传递引用（按值），这样当您将List传递给Map时，只会将引用传递给List }。当您clear List清除内存位置时，Map中的引用仍指向同一位置。

我建议您将方法更改为：

public void addNumberForContact(String name, String number) {
    contactMap.computeIfAbsent(name, k -> new ArrayList<>()).add(number);
}

通过这种方式，您可以将List的创建抽象为控制Map的类，从而避免出现此问题。

此外，按键遍历Map然后获取值是反模式。 Map存储密钥 - ＆gt;值组合在一起，因此您可以使用EntrySet：

contactMap.entrySet().forEach(System.out::println);

Answer 2

如果您只想打印值的值，您可以这样做：

contactMap.entrySet().stream()
          .map(Entry::getValue)
          .flatMap(List::stream)
          .forEach(System.out::println);

此外，如果这是该方法的唯一目的，则不必返回任何内容，因此请将return-type更改为void并删除return语句。

Answer 3

您只包含最后一个列表的原因是因为您的插入方法：

List <String> list= new ArrayList<String>();
    list.add("0321564988");
    list.add("7891268429");
    contacts.addContact("Name1",list);
    list.clear();
    list.add("1891219122");
    list.add("9495198929");
    contacts.addContact("Name2",list);
    list.clear();
    list.add("8949219912");
    contacts.addContact("Name3",list);

Map通过将键（在您的情况下为String键）引用到对象的内存地址（在您的情况下为List <String>）来绑定键值对。您的地图包含三个不同的键，但每个键都指向内存中完全相同的对象。因此，当您修改一个Map条目时，更改对所有密钥都有效。要更正代码，每次要添加代码时都应初始化列表：

List <String> list= new ArrayList<String>();
    list.add("0321564988");
    list.add("7891268429");
    contacts.addContact("Name1",list);
List <String> list1= new ArrayList<String>();
    list1.add("1891219122");
    list1.add("9495198929");
    contacts.addContact("Name2",list1);
List <String> list2= new ArrayList<String>();
    list2.add("8949219912");
    contacts.addContact("Name3",list2);

最后值得指出的是，“内存引用”对于从Object类派生的元素有效。基本类型（即int, boolean, double）由值引用，因此对于此类数据，您的代码可以正常工作。

在Java Hashmap中显示多个键的多个值

3 个答案: