7

ReSharperは、「の複数の列挙の可能性がある」ので、リストまたは配列にを列挙することを提案しましIEnumerable<T>た。IEnumerable<T>

提案された自動コードリファクタリングにIEnumerable<T>は、を呼び出す前にすでに配列であるかどうかを確認するための最適化が組み込まれていToArray()ます。

var list = source as T[] ?? source.ToArray();
  • この最適化は、元のLINQメソッドにすでに組み込まれていませんか？
  • そうでない場合、そうしない動機は何でしょうか？
4

1 に答える 1

6

いいえ、そのような最適化はありません。sourceがICollectionの場合、新しいアレイにコピーされます。配列を作成するためBuffer<T>に使用される構造体のコードは次のとおりです。Enumerable

internal Buffer(IEnumerable<TElement> source)
{    
    TElement[] array = null;
    int length = 0;
    ICollection<TElement> is2 = source as ICollection<TElement>;
    if (is2 != null)
    {
         length = is2.Count;
         if (length > 0)
         {
             array = new TElement[length]; // create new array
             is2.CopyTo(array, 0); // copy items
         }
    }
    else // we don't care, because array is ICollection<TElement>

    this.items = array;
}

そしてここにEnumerable.ToArray()方法があります：

public static TSource[] ToArray<TSource>(this IEnumerable<TSource> source)
{
    if (source == null)
    {
        throw Error.ArgumentNull("source");
    }
    Buffer<TSource> buffer = new Buffer<TSource>(source);
    return buffer.ToArray(); // returns items
}
于 2012-12-09T18:58:42.867 に答える