将元素添加到包含 ArrayList 的 HashMap

Question

这个简单的方法应该将元素添加到 HashMap。这些元素是从 txt 文件中读取的，我已经对其进行了测试，并且该区域的所有内容都运行良好。当我打印地图中的值时出现问题。因此，我认为添加它们时可能存在一些问题。txt文件的内容是SW: (scores, 90); (scores, 91); (scores, 92);其中权限是“SW”，属性是“（分数，90）...（分数，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 printingpublic 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 文件中完全相同的东西，但我得到了西南：（得分，92）；（得分，92）；（得分，92）即使我paMap.get(per).add(att)从 addMap 方法中删除该行，我最终也会得到 SW: (scores, 92);另外，我已经测试了是否正确调用了 for 循环，并且确实如此。因此，我对 ArrayList 上的属性如何更新以及正确的做法感到困惑。您在我添加或打印值的方式中看到任何错误吗？谢谢您的帮助

Answer 1

几个建议：

在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("; "));

  }

Answer 2

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

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

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

如果您使用的是 java8 及更高版本；)