对 FirstOrDefault 的不同看法

Question

IEnumerable 扩展方法 FirstOrDefault 并没有完全按照我的意愿行事，因此我创建了 FirstOrValue。这是解决这个问题的好方法还是有更好的方法？

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

您的代码可能不正确；您可能还没有考虑所有情况。

当然，在我们有规范之前，我们无法知道任何代码是否正确或不正确。因此，首先编写一个单行规范：

“FirstOrValue 接受 T 序列、谓词和 T 值，并返回序列中与谓词匹配的第一项如果有的话，或者如果没有，则为规定值。”

您的尝试实际上实现了该规范吗？当然不是！测试一下：

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

返回-1。根据规范，正确答案是 0。与谓词匹配的第一个项目为零，因此应该返回它。

规范的正确实现如下所示：

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

始终先编写规范，即使它只是一个句子。

Answer 2

对于引用类型，default(T) 默认返回 null。

我会这样做

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

由于这是一个重载，因此值得一提的是没有谓词的版本。

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

Answer 4

如果您想调整可读性而不是使用 DefaultIfEmpty，这对我来说似乎是合理的。

如果默认值的创建成本很高，您还可以创建使用 lambda 的覆盖，仅在必要时创建它。

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