Bagaimana saya bisa menghindari mengulang kode yang menginisialisasi hashmap dari hashmap?

Setiap klien memiliki id, dan banyak faktur, dengan tanggal, disimpan sebagai Hashmap klien dengan id, dari hashmap faktur berdasarkan tanggal:

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

Solusi Java tampaknya menggunakan getOrDefault:

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

Tetapi jika get bukan null, saya masih ingin mengeksekusi (tanggal, faktur), dan juga menambahkan data ke "allInvoicesAllClients" masih diperlukan. Jadi sepertinya tidak banyak membantu.

Ini adalah kasus penggunaan yang sangat baik untuk Map#computeIfAbsent. Cuplikan Anda pada dasarnya setara dengan:

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

Jika idtidak hadir sebagai kunci allInvoicesAllClients, maka itu akan membuat pemetaan dari idke yang baru HashMapdan mengembalikan yang baru HashMap. Jika idada sebagai kunci, maka itu akan mengembalikan yang sudah ada HashMap.

computeIfAbsentadalah solusi tepat untuk kasus khusus ini. Secara umum, saya ingin mencatat yang berikut, karena belum ada yang menyebutkannya:

Hashmap "outer" hanya menyimpan referensi ke hashmap "inner", jadi Anda bisa menyusun ulang operasi untuk menghindari duplikasi kode:

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

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

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

Anda seharusnya tidak pernah menggunakan inisialisasi peta "penjepit ganda".

{{  put(date, invoice); }}

Dalam hal ini, Anda harus menggunakan computeIfAbsent

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

Jika tidak ada peta untuk ID ini, Anda akan memasukkan satu. Hasilnya adalah peta yang ada atau yang dihitung. Anda kemudian dapat putitem di peta itu dengan jaminan bahwa itu tidak akan nol.

Ini lebih panjang dari jawaban yang lain, tetapi jauh lebih mudah dibaca:

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

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

Anda sedang melakukan dua hal terpisah di sini: memastikan bahwa HashMapada, dan menambahkan entri baru ke sana.

Kode yang ada memastikan untuk memasukkan elemen baru terlebih dahulu sebelum mendaftarkan peta hash, tetapi itu tidak perlu, karena HashMaptidak peduli tentang pemesanan di sini. Varian tidak ada threadsafe, sehingga Anda tidak kehilangan apa pun.

Jadi, seperti yang disarankan @Heinzi, Anda bisa membagi dua langkah ini.

Apa yang saya juga akan lakukan adalah offload penciptaan HashMapke allInvoicesAllClientsobjek, sehingga getmetode tidak dapat kembali null.

Ini juga mengurangi kemungkinan balapan antara utas-utas terpisah yang keduanya bisa mendapatkan nullpointer dari getdan kemudian memutuskan untuk putyang baru HashMapdengan satu entri - yang kedua putmungkin akan membuang yang pertama, kehilangan Invoiceobjek.