将元素添加到包含 ArrayList 的 HashMap

Question

这个简单的方法应该是向 HashMap 添加元素。这些元素是从 txt 文件中读取的，我已经对其进行了测试，并且该区域的所有内容都运行良好。当我打印地图中的值时会出现问题。因此，我认为添加它们时可能会出现一些问题。

txt文件的内容是

SW: (scores, 90); (scores, 91); (scores, 92);

权限为“SW”，属性为“(scores, 90)... (scores, 92)”。这些是以下方法中使用的元素。

public Map<Permissions, ArrayList<Attributes>> paMap; 

public void addMap(Permissions per, Attributes att) {
    if(paMap.containsKey(per)) {
        paMap.get(per).add(att);            
    }
    else{
        ArrayList<Attributes> attList = new ArrayList<Attributes>();
        attList.add(att);
        paMap.put(per, attList);
    }
}

 //Attributes contain variables (Scores) and values(90-93), that is just for printing

public void printMap() {
    ArrayList<Attributes> list;

    for(Permissions p: paMap.keySet()) {
        list = paMap.get(p);
        System.out.print(p.name + ": ");

        for(Attributes l: list) {
            System.out.print("<" + l.variable +", " + l.value + ">; ");
        }
        System.out.println();
    }

当我打印地图时，我希望得到与 txt 文件中完全相同的内容，但我却得到了

SW：（得分，92）； （得分，92）； （得分，92）

即使我从 addMap 方法中删除 paMap.get(per).add(att) 行，我最终也会得到 SW: (scores, 92);

另外，我已经测试了是否正确调用了 for 循环。

因此，我对 ArrayList 上的属性如何更新以及正确的做法感到困惑。您是否发现我添加或打印值的方式有任何错误？

感谢您的帮助

Answer 1

我不知道 read 方法看起来如何，但似乎你一直在添加相同的属性，所以也许在 read 函数中你不是创建新属性而是一直使用相同的对象，只是在这个上调用集合，所以在数组 lsit 的末尾是相同的对象添加了 3 次

顺便说一句： 方法 addmap 也可以替换为：

paMap.computeIfAbsent(per,x->new ArrayList<Attributes>()).add(att);

如果您使用的是 java8 及以上版本 ;)

Answer 2

几个建议：

• 在Map 声明中使用接口：

public Map<Permissions, List<Attributes>> paMap; 

• 为了用作Map 中的键，请确保Permissions 是 不可变并且正确定义了equals 和hashCode

所以，假设 Permissions 只有 1 个字符串字段，它应该看起来像这样：

public final class Permissions {
  private final String permission;
  public Permissions(String permission) {
    this.permission = permission;
  }
  public String getPermission() {
    return this.permission;
  }
  @Override
  public boolean equals(Object o) {
    if (this == o) {
      return true;
    }
    if (o == null || getClass() != o.getClass()) {
      return false;
    }
    Permissions that= (Permissions ) o;
    return Objects.equals(this.permission, that.permission);
  }
  @Override
  public int hashCode() {
    return Objects.hash(this.permission);
  }
}

• 尝试另一种实现printMap 方法。例如，使用 Java 8：

  private void printMap() {
    paMap.entrySet().stream().map(this::format).forEach(System.out::println);
  }
  private String format(Map.Entry<Permissions, List<Attributes>> entry) {
    String key = formatPermissions(entry.getKey());
    String value = formatAttributes(entry.getValue());
    return String.format("%s: %s", key, value);
  }
  private String formatPermissions(Permissions permissions) {
    return permissions.name;
  }
  private String formatAttributes(List<Attributes> attributes) {
    return attributes.stream()
            .map(attr -> String.format("(%s, %s)", attr.variable, attr.value))
            .collect(Collectors.joining("; "));
  }