Как добавить в массив в мапе?

Question

[Рус Лан]

коллеги, подскажите плз.
Есть, класс и мапа.

class Payload1 {
     String summary;
     String issueKey;
     String description;
     String displayName;
}

HashMap<String,ArrayList<Payload1>> mapa1 = new HashMap<>();

Пытаюсь написать условие, если уже есть ключ в мапе. То нужно не затирать массив Payload1 а добавить к нему елемент.

if(mapa1.containsKey(assigneeMail)){
     mapa1.put(assigneeMail, mapa1.get(assigneeMail).add(new Payload1( it.getSummary(),  it.getKey(),  it.getDescription(), it.getAssignee().getDisplayName())))
   }
    else mapa1.put(assigneeMail,[new Payload1( it.getSummary(),  it.getKey(),  it.getDescription(),  it.getAssignee().getDisplayName())])

Первое условие не добавляет к нему элемент, а изменяет велью мапы на true. Не могу понять где затык.

Comments

[Wan-Derer]

1. Оформи код соответствующим тэгом.
2. .add не возвращает новый List, а добавляет значение в существующий (по ссылке).

[Рус Лан]

Wan-Derer,
1)оформил
2)так разве я не получаю уже существующий с помощью mapa1.get()

[Alex K.]

Получаете, но метод add возвращает boolean (true если элемент успешно добавлен, иначе false), а мапа хочет ArrayList.

[Рус Лан]

Wan-Derer,

if(mapa1.containsKey(assigneeMail)){
     mapa1.put(assigneeMail, [mapa1.get(assigneeMail), new Payload1( it.getSummary(),  it.getKey(),  it.getDescription(), it.getAssignee().getDisplayName())])
   }
    else mapa1.put(assigneeMail,[new Payload1( it.getSummary(),  it.getKey(),  it.getDescription(),  it.getAssignee().getDisplayName())])
  }

у меня получилась конструкция [[old payload], New Payload]
Можно ли как-то избавиться от доп[]?

[Рус Лан]

Alex, посмотри выше

[mayton2019]

Обычно вот это

HashMap<String,ArrayList<Payload1>> mapa1 = new HashMap<>();

заменяют на

Map<String,List<Payload1>> mapa1 = new HashMap<>();

Это более красиво для интерфейсов. Обобщенно. И старуха Варвара Лискова будет довольна.

[Рус Лан]

mayton2019, спасибо, учту

Answer 1

[Alex K.]

Ну, например так

if (mapa1.containsKey(assigneeMail)) {
    ArrayList<Payload1> payloads = mapa1.get(assigneeMail);
    payloads.add(new Payload1(it.getSummary(), it.getKey(), it.getDescription(), it.getAssignee().getDisplayName()));
    mapa1.replace(assigneeMail, payloads);
} else {
    mapa1.put(assigneeMail, new ArrayList<>(List.of(new Payload1(it.getSummary(), it.getKey(), it.getDescription(), it.getAssignee().getDisplayName()))));
}

С Java 8 в Map появился метод merge ещё.

Comments

[Рус Лан]

пасибо, все работает