Modo pulito Guava per gestire la raccolta eventualmente-null

Question

Ho un metodo che accetta un argomento Collection<Foo> foos, che potrebbe essere NULL. Voglio finire con una copia locale dell'input come ImmutableSet. In questo momento il mio codice assomiglia a questo:

if (foos == null) 
{ 
    this.foos = ImmutableSet.of(); 
} 
else 
{ 
    this.foos = ImmutableSet.copyOf(foos); 
} 

C'è un modo più pulito per fare questo? Se foos fosse un parametro semplice, potrei fare qualcosa come Objects.firstNonNull(foos, Optional.of()) ma non sono sicuro che ci sia qualcosa di simile a gestire le raccolte.

Answer 1

non vedo il motivo per cui non si poteva usare Objects.firstNonNull:

this.foos = ImmutableSet.copyOf(Objects.firstNonNull(foos, ImmutableSet.of())); 

è possibile risparmiare qualche digitazione con le importazioni statiche, se vi piace il genere:

import static com.google.common.collect.ImmutableSet.copyOf; 
import static com.google.common.collect.ImmutableSet.of; 
// snip... 
this.foos = copyOf(Objects.firstNonNull(foos, of())); 

Answer 2

Un Collection è un riferimento come un altro, così si potrebbe fare:

ImmutableSet.copyOf(Optional.fromNullable(foos).or(ImmutableSet.of())); 

Ma che sta diventando un bel pugno a scrivere. Più semplice:

foos == null ? ImmutableSet.of() : ImmutableSet.copyOf(foos);