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와 술어 및 T 값의 시퀀스를 취해 하나의 술어와 일치하는 시퀀스의 첫 번째 아이템을 반환하거나, 또는 하나의 라인 스펙을 작성하여 시작합니다. , 없다면, 명시된 가치. "

귀하의 시도가 실제로 해당 사양을 구현합니까? 확실히! 테스트 :

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

이 경우 -1을 반환합니다. 명세에 따라 정답은 0입니다. 술어와 일치하는 첫 번째 항목은 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를 반환합니다.

난 당신이 DefaultIfEmpty를 사용하는 대신 가독성을 조정할하려면이

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> source, Func<T, bool> predicate, Func<T> getValue) 
{ 
    T first = source.FirstOrDefault(predicate); 
    return Equals(first, default(T)) ? getValue() : first; 
} 

Answer 4

과부하이기 때문에 술어가없는 버전을 언급 할 가치가 있습니다.

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