4

編集:元の質問を間違えました。メソッドLastおよびLastOrDefault (またはSingleおよびSingleOrDefault、またはFirstおよびFirstOrDefault - それらの多く!) に関するものである必要があります。

この質問に触発されて、私はReflectorを開いてコードを見ました

Enumerable.Last<T>(this collection)

それから私はのコードにジャンプしました

Enumerable.LastOrDefault<T>(this collection)

そして、最後の1行だけが異なるまったく同じコード(約20行)を見ました(最初のメソッドはデフォルト(T)を返し、2番目のメソッドは例外をスローします)。

私の質問は、なぜそうなのかということです。Microsoft の担当者が、.Net フレームワーク内の重要なコードの複製を許可するのはなぜですか? コードレビューはありませんか?

4

1 に答える 1

4

実際、それらはまったく同じではありません。最初は次のようになります。

public static TSource Last<TSource>(this IEnumerable<TSource> source)
{
    if (source == null) throw Error.ArgumentNull("source");

    IList<TSource> list = source as IList<TSource>;
    if (list != null) {
        int count = list.Count;
        if (count > 0) return list[count - 1];
    } else {
        using (IEnumerator<TSource> enumerator = source.GetEnumerator()) {
            if (enumerator.MoveNext()) {
                TSource current;
                do { current = enumerator.Current;}
                while (enumerator.MoveNext());

                return current;
            }
        }
    }
    throw Error.NoElements();
}

そしてもう一つは次のようなものです。

public static TSource Last<TSource>(this IEnumerable<TSource> source,
Func<TSource, bool> predicate)
{
    if (source == null) throw Error.ArgumentNull("source");
    if (predicate == null) throw Error.ArgumentNull("predicate");

    TSource last = default(TSource);
    bool foundOne = false;
    foreach (TSource value in source) {
        if (predicate(value)) {
            last = value;
            foundOne = true;
        }
    }
    if (!foundOne) throw Error.NoMatch();
    return last;
}

PS .: 現在、著作権を侵害していないことを願っています。:S

于 2009-09-05T04:27:56.200 に答える