해시 맵의 해시 맵을 초기화하는 코드 반복을 피하려면 어떻게해야합니까?

모든 고객은 날짜별로 인보이스의 해시 맵으로 ID와 고객의 해시 맵으로 저장된 날짜와 함께 많은 송장을 가지고 있습니다.

HashMap<LocalDateTime, Invoice> allInvoices = allInvoicesAllClients.get(id);

if(allInvoices!=null){
    allInvoices.put(date, invoice);      //<---REPEATED CODE
}else{
    allInvoices = new HashMap<>();
    allInvoices.put(date, invoice);      //<---REPEATED CODE
    allInvoicesAllClients.put(id, allInvoices);
}

Java 솔루션은 다음을 사용하는 것 같습니다 getOrDefault.

HashMap<LocalDateTime, Invoice> allInvoices = allInvoicesAllClients.getOrDefault(
    id,
    new HashMap<LocalDateTime, Invoice> (){{  put(date, invoice); }}
);

그러나 get이 null이 아닌 경우 여전히 put (날짜, 송장)을 실행하고 "allInvoicesAllClients"에 데이터를 추가해야합니다. 그래서별로 도움이되지 않는 것 같습니다.

이것은 훌륭한 유스 케이스입니다 Map#computeIfAbsent. 스 니펫은 기본적으로 다음과 같습니다.

allInvoicesAllClients.computeIfAbsent(id, key -> new HashMap<>()).put(date, invoice);

경우 id에 키와 존재하지 않는 allInvoicesAllClients, 그것은에서 매핑을 만듭니다 id새에 HashMap새를 반환합니다 HashMap. 이 id키로 존재하면 기존을 반환합니다 HashMap.

computeIfAbsent이 특별한 경우에 대한 훌륭한 솔루션입니다. 일반적으로 아무도 언급하지 않았으므로 다음 사항에 유의하고 싶습니다.

"outer"해시 맵 은 "inner"해시 맵에 대한 참조 만 저장 하므로 코드 중복을 피하기 위해 작업 순서를 다시 지정할 수 있습니다.

HashMap<LocalDateTime, Invoice> allInvoices = allInvoicesAllClients.get(id);

if (allInvoices == null) {           
    allInvoices = new HashMap<>();
    allInvoicesAllClients.put(id, allInvoices);
}

allInvoices.put(date, invoice);      // <--- no longer repeated

"이중 괄호"맵 초기화는 거의 사용하지 않아야합니다.

{{  put(date, invoice); }}

이 경우에는 computeIfAbsent

allInvoicesAllClients.computeIfAbsent(id, (k) -> new HashMap<>())
                     .put(date, allInvoices);

이 ID에 대한지도가 없으면지도를 삽입합니다. 결과는 기존 또는 계산 된 맵이됩니다. 그런 다음 put해당 맵의 항목이 null이되지 않도록 보장 할 수 있습니다.

이것은 다른 답변보다 길지만 훨씬 더 읽기 쉽습니다.

if(!allInvoicesAllClients.containsKey(id))
    allInvoicesAllClients.put(id, new HashMap<LocalDateTime, Invoice>());

allInvoicesAllClients.get(id).put(date, invoice);

여기서 두 가지 별도의 작업을 수행합니다. HashMap 존재 여부를 하고 새 항목을 추가합니다.

기존 코드는 해시 맵을 등록하기 전에 먼저 새 요소를 삽입해야하지만 필요하지는 않습니다. HashMap 여기서 순서를 신경 쓰지 하지 않습니다. 두 가지 변형 모두 스레드 안전하지 않으므로 아무것도 잃지 않습니다.

@Heinzi가 제안한 것처럼이 두 단계를 나눌 수 있습니다.

내가 또 할 것입니다 것은 창조 오프로드입니다 HashMap받는 allInvoicesAllClients소위, 객체 get메소드는 반환 할 수 없습니다null .

또한 null포인터를 가져 get오고 단일 항목 put으로 새 HashMap항목 을 결정할 수 있는 별도의 스레드 간 경쟁 가능성을 put줄입니다 . 두 번째 항목 은 첫 번째 항목을 버리고 Invoice개체를 잃을 수 있습니다.