16

さまざまな大きなパフォーマンス上の利点（私の場合）のために、自分自身の動的配列の実装をロールバックしなければならない立場にいることに気づきました。ただし、自分のバージョンの列挙子を作成し、その効率をListが使用しているものと比較した後、私は少し戸惑いました。リスト1は、はるかに複雑ですが、私のバージョンよりも約30〜40％高速です。

リスト列挙子の実装の重要な部分は次のとおりです。

public struct Enumerator : IEnumerator<T>, IDisposable, IEnumerator
{
    private List<T> list;
    private int index;
    private int version;
    private T current;
    internal Enumerator(List<T> list)
    {
        this.list = list;
        this.index = 0;
        this.version = list._version;
        this.current = default(T);
        return;
    }

    public bool MoveNext()
    {
        List<T> list;
        list = this.list;
        if (this.version != list._version)
        {
            goto Label_004A;
        }
        if (this.index >= list._size)
        {
            goto Label_004A;
        }
        this.current = list._items[this.index];
        this.index += 1;
        return 1;
        Label_004A:
        return this.MoveNextRare();
    }

    public T Current
    {
        get {  return this.current; }
    }
}

そして、これが私の非常にベアボーンバージョンです：

internal struct DynamicArrayEnumerator<T> : IEnumerator<T> where T : class
{
     private readonly T[] internalArray;
     private readonly int lastIndex;
     private int currentIndex;

     internal DynamicArrayEnumerator(DynamicArray<T> dynamicArray)
     {
          internalArray = dynamicArray.internalArray;
          lastIndex = internalArray.Length - 1;
          currentIndex = -1;
     }

     public T Current
     {
          get { return internalArray[currentIndex]; }
     }

     public bool MoveNext()
     {
          return (++currentIndex <= lastIndex);
     }
}

これがマイクロ最適化であることは知っていますが、実際には、リスト列挙子が私のものよりもはるかに高速である理由を理解することに興味があります。何か案は？ありがとう！

編集：要求に応じて; DynamicArrayクラス（関連する部分）：列挙子はこの内部クラスです。

public struct DynamicArray<T> : IEnumerable<T> where T : class
{
    private T[] internalArray;
    private int itemCount;

    internal T[] Data
    {
        get { return internalArray; }
    }

    public int Count
    {
        get { return itemCount; }
    }

    public DynamicArray(int count)
    {
        this.internalArray = new T[count];
        this.itemCount = 0;
    }

    public IEnumerator<T> GetEnumerator()
    {
        return new DynamicArrayEnumerator<T>(this);
    }

    IEnumerator IEnumerable.GetEnumerator()
    {
        return this.GetEnumerator();
    }

}

私がテストしている方法については：

 List<BaseClass> list = new List<BaseClass>(1000000);
 DynamicArray<BaseClass> dynamicArray = new DynamicArray<BaseClass>(1000000);

// Code for filling with data omitted.

   int numberOfRuns = 0;
   float p1Total = 0;
   float p2Total = 0;
   while (numberOfRuns < 100)
   {
        PerformanceAnalyzer p1 = new PerformanceAnalyzer(() =>
        {
             int u = 0;
             foreach (BaseClass b in list)
             {
                  if (b.B > 100)   // Some trivial task
                      u++;
             }
        });
        p1.ExecuteAndClock();
        p1Total += p1.TotalElapsedTicks;

        PerformanceAnalyzer p2 = new PerformanceAnalyzer(() =>
        {
             int u = 0;
             foreach (BaseClass b in dynamicArray)
             {
                  if (b.B > 100)  // Some trivial task
                       u++;
             }
        });
        p2.ExecuteAndClock();
        p2Total += p2.TotalElapsedTicks;

        numberOfRuns++;
    }

    Console.WriteLine("List enumeration: " + p1Total / totalRuns + "\n");
    Console.WriteLine("Dynamic array enumeration: " + p2Total / totalRuns + "\n");

PerformanceAnalyzerクラスは基本的にストップウォッチを開始し、提供されたアクションデリゲートを実行し、その後ストップウォッチを停止します。

編集2（Ryan Gatesへのクイックアンサー）：自分でロールしたい理由はいくつかあります。最も重要なのは、非常に高速なRemoveAt（int index）メソッドが必要なことです。

特定のケースではリスト要素の順序を気にする必要がないので、.Net組み込みリストの方法を回避できます。

public void RemoveAt(int index)
{
    T local;
    if (index < this._size)
    {
        goto Label_000E;
    }
    ThrowHelper.ThrowArgumentOutOfRangeException();
Label_000E:
    this._size -= 1;
    if (index >= this._size)
    {
        goto Label_0042;
    }
    Array.Copy(this._items, index + 1, this._items, index, this._size - index);
Label_0042:
    this._items[this._size] = default(T);
    this._version += 1;
    return;
}

そして、代わりに次の線に沿って何かを使用します：

public void RemoveAt(int index)
{
     // overwrites the element at the specified index with the last element in the array and decreases the item count.
     internalArray[index] = internalArray[itemCount];  
     itemCount--;
}

私の場合、長いリストの最初の1000個の要素をインデックスで削除する必要があるとすると、時間を大幅に節約できる可能性があります。

4

1 に答える 1

16

さて、ベンチマークの問題は別として、DynamicArrayクラスをもっと次のようにする方法は次のList<T>とおりです。

public DynamicArrayEnumerator<T> GetEnumerator()
{
    return new DynamicArrayEnumerator<T>(this);
}

IEnumerator<T> IEnumerable<T>.GetEnumerator()
{
    return GetEnumerator();
}

IEnumerator IEnumerable.GetEnumerator()
{
    return this.GetEnumerator();
}

これで、動的配列で動作していることを認識しているコードはDynamicArrayEnumerator<T> 、ボックス化なしで、仮想ディスパッチなしで反復できます。これはまさに何をするかList<T>です。コンパイラーは、型がカスタムの方法でパターンを実装するときに気づき、インターフェースの代わりに関連する型を使用します。

現在のコードでは、-をボックス化するため、-を作成してもメリットはありませstructんGetEnumerator()。

上記の変更を試して、ベンチマークを修正して、より長く機能するようにしてください。大きな違いが見られると思います。

于 2012-12-06T20:56:29.450 に答える