Une prise de vue différente sur FirstOrDefault

Question

La méthode d'extension IEnumerable FirstOrDefault n'a pas fait exactement ce que je voulais, donc j'ai créé FirstOrValue. Est-ce un bon moyen d'y parvenir ou existe-t-il un meilleur moyen?

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

Votre code est probablement incorrect; vous n'avez probablement pas considéré tous les cas.

Bien sûr, nous ne pouvons pas savoir si un code est correct ou incorrect jusqu'à ce que nous ayons une spécification. Commencez par écrire une spécification d'une ligne:

"FirstOrValue<T> prend une séquence de T, un prédicat et une valeur de T, et renvoie le premier élément de la séquence qui correspond au prédicat s'il y en a un, ou , s'il n'y en a pas, la valeur déclarée. "

Est-ce que votre tentative implémente réellement cette spécification? Certainement pas! Testez-le:

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

cela renvoie -1. La réponse correcte selon la spécification est 0. Le premier élément qui correspond au prédicat est zéro, il doit donc être retourné.

Une bonne mise en œuvre de la spécification ressemblerait à ceci:

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; 
} 

toujours écrire une spécification d'abord, même si elle est seulement une seule phrase.

Answer 2

default(T) retournera null par défaut pour les types de référence.

Je ferais ce

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

semble raisonnable pour moi si vous souhaitez modifier la lisibilité au lieu d'utiliser DefaultIfEmpty.

Vous pouvez également créer un remplacement utilisant un lambda si la création de la valeur par défaut est coûteuse, en la créant uniquement si nécessaire.

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

Comme il s'agit d'une surcharge, il vaut la peine de mentionner la version sans prédicat.

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; 
}