如何避免重复代码初始化hashmap的hashmap？

每个客户都有一个ID，以及许多带有日期的发票，这些日期按ID存储为客户的Hashmap，即按日期的发票哈希图：

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对于这种特殊情况是一个很好的解决方案。总的来说，我想指出以下几点，因为还没有人提及：

“外部”哈希表仅存储对“内部”哈希表的引用，因此您可以对操作进行重新排序以避免代码重复：

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，可以确保该映射中的项目不会为空。

这比其他答案更长，但是恕我直言，可读性更高：

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对象。