un approccio diverso sul FirstOrDefault

Question

Il metodo di estensione IEnumerable FirstOrDefault non esattamente fare come volevo così ho creato FirstOrValue. E 'un buon modo per fare questo o c'è un modo migliore?

public static T FirstOrValue<T>(this IEnumerable<T> source, Func<T, bool> predicate, T value) 
{ 
    T first = source.FirstOrDefault(predicate); 
    return Equals(first, default(T)) ? value : first; 
} 

Answer 1

Il tuo codice è probabilmente errato; probabilmente non hai considerato tutti i casi.

Naturalmente, non possiamo sapere se esiste il codice è corretto o non corretto finché non avremo uno spec. Quindi inizia scrivendo una specifica su una riga:

"FirstOrValue<T> prende una sequenza di T, un predicato e un valore di T, e restituisce il primo elemento nella sequenza che corrisponde al predicato se ce n'è uno, oppure , se non c'è, il valore dichiarato. "

Il vostro tentativo effettivamente attuare tale spec? Certamente no! Provalo:

int x = FirstOrValue<int>(new[] { -2, 0, 1 }, y=>y*y==y, -1); 

restituisce -1. La risposta corretta in base alle specifiche è 0. Il primo elemento che corrisponde al predicato è zero, quindi deve essere restituito.

Una corretta applicazione delle specifiche sarà simile:

public static T FirstOrValue<T>(this IEnumerable<T> sequence, Func<T, bool> predicate, T value) 
{ 
    if (sequence == null) throw new ArgumentNullException("sequence"); 
    if (predicate == null) throw new ArgumentNullException("predicate"); 
    foreach(T item in sequence) 
     if (predicate(item)) return item; 
    return value; 
} 

Sempre scrivere una specifica prima, anche se è solo una sola frase.

Answer 2

default(T) tornerà null per impostazione predefinita per i tipi di riferimento.

farei questo

public static T FirstOrValue<T>(this IEnumerable<T> source, Func<T, bool> predicate, T value) 
{ 
    T first = source.FirstOrDefault(predicate); 
    return first ?? value; 
} 

Answer 3

mi sembra ragionevole se si desidera modificare la leggibilità invece di utilizzare DefaultIfEmpty.

Si potrebbe anche creare un override che utilizza una lambda se la creazione del valore di default è costoso, creandola solo se necessario.

public static T FirstOrValue<T>(this IEnumerable<T> source, Func<T, bool> predicate, Func<T> getValue) 
{ 
    T first = source.FirstOrDefault(predicate); 
    return Equals(first, default(T)) ? getValue() : first; 
} 

Answer 4

Poiché si tratta di un sovraccarico, vale la pena menzionare la versione senza predicato.

public static T FirstOrValue<T>(this IEnumerable<T> sequence, T value) 
{ 
    if (sequence == null) throw new ArgumentNullException("sequence"); 
    foreach(T item in sequence) 
     return item; 
    return value; 
}