Czystość metody Guava do obsługi kolekcji o możliwej wartości zerowej

Question

Mam metodę, która przyjmuje argument Collection<Foo> foos, który może mieć wartość NULL. Chcę skończyć z lokalną kopią danych wejściowych jako ImmutableSet. Teraz mój kod wygląda następująco:

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

Czy istnieje lepszy sposób na zrobienie tego? Jeśli foos byłby prostym parametrem, mogłem zrobić coś takiego, jak Objects.firstNonNull(foos, Optional.of()), ale nie jestem pewien, czy istnieje coś podobnego do obsługi kolekcji.

Answer 1

ja nie rozumiem, dlaczego nie można używać Objects.firstNonNull:

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

Można zaoszczędzić trochę pisanie z importu statycznych, jeśli to twoja sprawa:

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

Collection jest odniesienie jak każdy inny, więc można zrobić:

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

Ale to staje się dość garstka pisać. Bardziej prosty:

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