ハッシュマップのハッシュマップを初期化するコードの繰り返しを回避するにはどうすればよいですか？

すべてのクライアントにはIDと多くの請求書があり、日付ごとに請求書のハッシュマップのIDごとのクライアントのハッシュマップとして保存されています。

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（date、invoice）を実行したいので、「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、そのマップ内のアイテムをnullにならないように保証できます。

これは他の回答よりも長いですが、imhoははるかに読みやすくなっています。

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

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

ここでは2つの別々のことを行っています。 HashMap存在するそれに新しいエントリを追加します。

既存のコードでは、ハッシュマップを登録する前に必ず新しい要素を挿入する必要がありますが、これは必要ありません HashMapここでは順序を気にしないはません。どちらのバリアントもスレッドセーフではないため、何も失うことはありません。

したがって、@ Heinziが提案したように、これら2つのステップを分割することができます。

私はまたどうなることの創造オフロードでHashMapのallInvoicesAllClientsオブジェクトなので、getこの方法は返すことはできませんnull。

これにより、nullポインタを取得し、単一のエントリで新しいスレッドにget決定する可能性のある別々のスレッド間の競合の可能性も減少します。putHashMapputInvoice