Riprogettazione del metodo per utilizzare Java 8 Map.computeIfAbsent() con eccezione generata

Question

Sto cercando di rielaborare alcuni dei miei metodi per renderli più concisi utilizzando Java 8, le cui nuove funzionalità sto cercando di assorbire lentamente.

È un metodo con lo scopo di aggiungere uno value a Map<Key, Set<Value>>. Esistono tre possibilità:

1. La chiave non esiste: viene aggiunta e ad essa è associata una nuova serie contenente il valore.
2. La chiave esiste: il valore viene aggiunto al set esistente. Si noti che un set non sarà mai null, perché ho alcune condizioni preliminari per affrontarlo.
3. La chiave esiste e il valore è già contenuto nel set: viene emesso un numero IllegalArgumentException.

Il codice che implementa questo comportamento è la seguente, e non fa uso di Java 8 caratteristiche:

public void addValue(Key key, Value value) { 
    // irrelevant preconditions... 

    Set<Value> valuesForKey = myMap.get(key); 
    if (valuesForKey != null && valuesForKey.contains(value)) 
     throw new IllegalArgumentException("Association exists already"); 

    if (valuesForKey == null) 
     myMap.put(key, new HashSet<Value>(Arrays.asList(value))); 
    else 
     valuesForKey.add(value); 
} 

Vorrei ridurre questo codice utilizzando Java 8 metodi come computeIfAbsent.

potessi riassumere l'ultimo if-else blocco, ma non posso andare oltre la ridondanza del valore impostato al quale key mappe ad avere già in recuperati Nell'effettuare i controlli di presupposto.

public void addValue(Key key, Value value) { 
    // irrelevant preconditions... 

    Set<Value> valuesForKey = myMap.get(key); 
    if (valuesForKey != null && valuesForKey.contains(value)) 
     throw new IllegalArgumentException("Association exists already"); 

    myMap.computeIfAbsent(key, v -> new HashSet<Value>()).add(value); 
} 

C'è comunque una possibilità di unire tutto ciò in un'unica istruzione?

Answer 1

È possibile sfruttare il fatto che il metodo add(element) restituirà true se il set non contiene l'elemento specificato. Se questa chiamata restituisce false, significa che l'elemento non è stato aggiunto perché era già presente. Pertanto, è possibile utilizzare:

public void addValue(Key key, Value value) { 
    boolean added = myMap.computeIfAbsent(key, k -> new HashSet<>()).add(value); 
    if (!added) { 
     throw new IllegalArgumentException("Association exists already"); 
    } 
}