Làm thế nào tôi có thể tránh lặp lại mã khởi tạo một hashmap của hashmap?

Mỗi khách hàng có một id và nhiều hóa đơn, có ngày tháng, được lưu dưới dạng Hashmap của khách hàng theo id, của một hashmap hóa đơn theo ngày:

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

Giải pháp Java dường như được sử dụng getOrDefault:

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

Nhưng nếu get không phải là null, tôi vẫn muốn đặt (ngày, hóa đơn) để thực thi và cũng cần thêm dữ liệu vào "allInvoices ALLCl Client". Vì vậy, nó dường như không giúp được gì nhiều.

Đây là một trường hợp sử dụng tuyệt vời cho Map#computeIfAbsent. Đoạn mã của bạn về cơ bản tương đương với:

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

Nếu idkhông có mặt như một khóa trong allInvoicesAllClients, thì nó sẽ tạo ánh xạ từ idmới HashMapvà trả lại cái mới HashMap. Nếu idcó mặt như một khóa, thì nó sẽ trả về cái hiện có HashMap.

computeIfAbsentlà một giải pháp tuyệt vời cho trường hợp cụ thể này. Nói chung, tôi muốn lưu ý những điều sau, vì chưa ai đề cập đến nó:

Hashmap "bên ngoài" chỉ lưu trữ một tham chiếu đến hashmap "bên trong", vì vậy bạn chỉ có thể sắp xếp lại các hoạt động để tránh trùng lặp mã:

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

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

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

Bạn nên không bao giờ sử dụng khởi tạo bản đồ "cú đúp".

{{  put(date, invoice); }}

Trong trường hợp này, bạn nên sử dụng computeIfAbsent

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

Nếu không có bản đồ cho ID này, bạn sẽ chèn một bản đồ. Kết quả sẽ là bản đồ hiện có hoặc được tính toán. Sau đó, bạn có thể putcác mục trong bản đồ đó với đảm bảo rằng nó sẽ không có giá trị.

Câu này dài hơn các câu trả lời khác, nhưng imho dễ đọc hơn nhiều:

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

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

Bạn đang làm hai điều riêng biệt ở đây: đảm bảo rằng sự HashMaptồn tại và thêm mục mới vào đó.

Mã hiện tại đảm bảo chèn phần tử mới trước khi đăng ký bản đồ băm, nhưng điều đó không cần thiết, vì HashMapkhông quan tâm đến việc đặt hàng ở đây. Không có biến thể nào là an toàn, vì vậy bạn không mất gì cả.

Vì vậy, như @Heinzi đề xuất, bạn chỉ có thể chia hai bước này.

Những gì tôi cũng sẽ làm là offload việc tạo ra HashMapđể các allInvoicesAllClientsđối tượng, do đó getphương pháp không thể trở về null.

Điều này cũng làm giảm khả năng các cuộc đua giữa các luồng riêng biệt có thể nhận được nullcon trỏ từ getđó và sau đó quyết định putmột cuộc đua mới HashMapvới một mục duy nhất - lần thứ hai putcó thể sẽ loại bỏ Invoiceđối tượng đầu tiên, làm mất đối tượng.