Bir hashmap hashmapını başlatan kodun tekrarlanmasını nasıl önleyebilirim?

Her müşterinin bir kimliği ve tarihle birlikte müşterilere kimliğe göre Hashmap olarak depolanan birçok faturası vardır.

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 çözümü aşağıdaki gibi görünüyor getOrDefault:

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

Ancak get boş değilse, hala koymak (tarih, fatura) yürütmek istiyorum ve ayrıca "allInvoicesAllClients" için veri eklemek hala gereklidir. Yani pek yardımcı olmuyor.

Bu mükemmel bir kullanım durumudur Map#computeIfAbsent. Snippet'iniz aslında aşağıdakilere eşdeğerdir:

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

Eğer idbir anahtar olarak bulunmaz allInvoicesAllClients, o zaman gelen eşlemesi oluştururuz idbir yeniye HashMapve yeni dönün HashMap. idBir anahtar olarak varsa , var olanı döndürür HashMap.

computeIfAbsentbu özel durum için harika bir çözümdür. Genel olarak, aşağıdakileri not etmek isterim, çünkü henüz kimse bundan bahsetmedi:

"Dış" hashmap sadece "iç" hashmap için bir referans depolar , böylece sadece kod çoğalmasını önlemek için işlemleri yeniden sıralayabilirsiniz:

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

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

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

Hemen hemen hiçbir zaman "çift ayraç" harita başlatma kullanmamalısınız.

{{  put(date, invoice); }}

Bu durumda, computeIfAbsent

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

Bu kimlik için harita yoksa bir tane ekleyeceksiniz. Sonuç mevcut veya hesaplanmış harita olacaktır. Daha sonra putbu haritadaki öğeleri, boş olmayacağını garanti ederek yapabilirsiniz.

Bu, diğer cevaplardan daha uzun, ancak daha okunabilir:

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

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

Burada iki ayrı şey yapıyorsunuz: var olmasını sağlamak HashMapve yeni girişi eklemek.

Mevcut kod, karma haritayı kaydetmeden önce yeni öğeyi eklediğinizden emin olur, ancak bu gerekli değildir, çünkü HashMapburadaki siparişle ilgilenmez. Her iki değişken de güvenli değildir, bu yüzden hiçbir şey kaybetmezsiniz.

Yani, @Heinzi'nin önerdiği gibi, bu iki adımı bölebilirsiniz.

Ne yapmam da olur yaratılması boşaltması olduğu HashMapiçin allInvoicesAllClientsbu nedenle, nesne getyöntemi döndüremez null.

Bu aynı zamanda, hem nullişaretçilerden hem de tek bir girişle yeni bir noktaya getkarar verebilen ayrı dişler arasındaki ırkların olasılığını azaltır - ikincisi nesneyi kaybederek ilkini atacaktır .putHashMapputInvoice