Perché Iterables.find() in Guava genera NoSuchElementException, invece di restituire null?

amo Google Guava e lo uso molto, ma c'è un metodo che ho sempre trovato me scrivere ..Perché Iterables.find() in Guava genera NoSuchElementException, invece di restituire null?

public static <T> T tryFind(Iterable<T> iterable, Predicate<T> predicate){ 
    for(T t : iterable){ 
     if(predicate.apply(t)){ 
       return t; 
     } 
    } 
    return null; 
    }

A me questo sembra essere un accessorio molto utile per Iterables (anche per Iterators per quella materia), quindi mi chiedo perché manchi. Inoltre, mentre posso vedere il punto di avere un metodo che genera NoSuchElementException, forse per distinguere tra ricerca di un nullo e non trovare l'elemento, questa situazione viene solo su se il predicato si sta utilizzando è

public boolean apply(T t){ 
    return t==null; 
}

che doesn sembra essere un caso comune.

Quindi, perché i progettisti guava hanno scelto di avere questo comportamento, invece di restituire null se non riesce a trovarlo?

Ecco javadoc per [Iterables.find()] [1]

[1]: http://google-collections.googlecode.com/svn/trunk/javadoc/com/google/common/collect/Iterables.html#find(java.lang.Iterable, com.google.common.base.Predicate)

fonte

2010-06-23 Enno Shioji

Forse sarebbe stato meglio lanciare un'eccezione controllata per impedire ai client di dimenticarlo ... – thSoft

Eventuali duplicati di [Iterables.find e Iterators.find - invece di gettare eccezione, ottenere null] (https: //stackoverflow.com/questions/2543052/iterables-find-and-iterators-find-instead-of-throwing-exception-get-null) – Stewart

Stiamo aggiungendo un altro overload di find() che accetta un valore predefinito.

fonte

2010-06-23 06:13:53

Grazie per l'ottimo lavoro, Kevin! –

'Iterables.find (iterable, predicate, defaultValue) ':) –

Che dire di un Optional tryFind (Iterable iterable, int position)? A volte non ho un valore predefinito disponibile e vorrei comunicare che il chiamante della mia API ha bisogno di gestirlo. – jontejj

probabilmente perché nullo è un ritorno valido valore. In generale, a meno che non vi sia una buona ragione per non supportare null, dovrebbe essere supportato. Se è supportato, devi gestire il caso in cui esiste.

fonte

2010-06-23 05:52:30 TofuBeer

La necessità di un'operazione find() che restituisce null è altamente discutibile. Sicuramente se tu sai che se c'è un null nella collezione, otterrai nullo di nuovo, quindi potresti usare semplicemente contiene, o qualche altro metodo più semplice esistente. – Trejkaz

Invece di tryFind() è possibile utilizzare il filtro e verificare se restituisce una raccolta vuota.

Ho scoperto che lavorare sempre con le raccolte è più semplice che chiedere direttamente gli oggetti.

fonte

2010-06-23 06:03:07

Hm, ma poi starai iterando due volte, giusto? Vedo che è più pulito, ma .. –

In molti casi il controllo non è più necessario e puoi immediatamente ripetere i risultati. Non succederà nulla quando non ci sono risultati, che nella mia esperienza è spesso il comportamento desiderato. –

A mio parere, NoSuchElementException è meglio di un ritardo e molto difficile da eseguire il debug NPE ... Nella maggior parte dei casi, quando si cerca un oggetto in una "raccolta", si sa che probabilmente lo si troverà. Se l'oggetto che stai cercando non è nella "collezione", stai affrontando un caso eccezionale ... Secondo me, il feedback NoSuchElementException è più esplicito del "null" senza significato.

Il valore predefinito che verrà introdotto in una versione futura di guava sarebbe un collegamento efficiente per trattare il caso eccezionale.

fonte

2010-06-23 22:56:07

Trovare avrebbe senso se non fosse possibile trovare piuttosto il valore predefinito.

Optional<T> Iterables.find(Iterable<T>, Predicate<? super T>)

fonte

2013-04-10 14:03:34

Perché Iterables.find() in Guava genera NoSuchElementException, invece di restituire null?

risposta

Problemi correlati