Jak mogę uniknąć powtarzania kodu inicjującego skrót mapy?

Każdy klient ma identyfikator i wiele faktur, wraz z datami, przechowywanymi jako Hashmap klientów według identyfikatorów, skrótów faktur według dat:

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);
}

Wydaje się, że rozwiązaniem Java jest użycie getOrDefault:

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

Ale jeśli get nie ma wartości zerowej, nadal chcę wykonać polecenie put (data, faktura), a także dodanie danych do „allInvoicesAllClients” jest nadal potrzebne. Więc to niewiele pomaga.

Jest to doskonały przypadek użycia dla Map#computeIfAbsent. Twój fragment kodu jest zasadniczo równoważny z:

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

Jeśli idnie jest obecny jako klucz allInvoicesAllClients, utworzy mapowanie z idna nowy HashMapi zwróci nowy HashMap. Jeśli idjest obecny jako klucz, wówczas zwróci istniejący HashMap.

computeIfAbsentjest doskonałym rozwiązaniem dla tego konkretnego przypadku. Ogólnie chciałbym zwrócić uwagę na następujące kwestie, ponieważ nikt jeszcze o tym nie wspominał:

„Zewnętrzna” mapa skrótów po prostu przechowuje odniesienie do „wewnętrznej” tablicy skrótów, dzięki czemu można po prostu zmienić kolejność operacji, aby uniknąć powielania kodu:

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

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

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

Prawie nigdy nie powinieneś używać inicjalizacji mapy „podwójny nawias klamrowy”.

{{  put(date, invoice); }}

W takim przypadku powinieneś użyć computeIfAbsent

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

Jeśli nie ma mapy dla tego identyfikatora, wstawisz ją. Wynikiem będzie istniejąca lub obliczona mapa. Możesz następnie putprzedmioty na tej mapie z gwarancją, że nie będzie ona zerowa.

Jest to dłuższe niż inne odpowiedzi, ale znacznie bardziej czytelne:

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

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

Robisz tutaj dwie oddzielne rzeczy: upewnienie się, że HashMapistnieje i dodanie do niej nowego wpisu.

Istniejący kod upewnia się, że najpierw wstawisz nowy element przed zarejestrowaniem mapy skrótu, ale nie jest to konieczne, ponieważ tutaj HashMapnie zależy na zamówieniu. Żaden wariant nie jest bezpieczny dla wątków, więc nic nie tracisz.

Tak jak sugerował @Heinzi, możesz po prostu podzielić te dwa kroki.

Co chciałbym także zrobić to odciążyć stworzenia HashMapdo allInvoicesAllClientsobiektu, więc getmetoda ta nie może powrócić null.

Zmniejsza to również możliwość wyścigów między osobnymi wątkami, które mogłyby zarówno pobrać nullwskaźniki, geta następnie zdecydować się putna nowy HashMapz jednym wpisem - drugi putprawdopodobnie odrzuciłby pierwszy, tracąc Invoiceobiekt.