もっと使いやすい方法デリゲートをデリゲートするデリゲート

Question

スレッドセーフな方法でコレクションを反復処理したいと思っています。私は一つの方法を持っているconvieniantです見つける

/// <summary> 
/// Saves each item in the list to the database 
/// </summary> 
protected static void SaveListToDatabase() { 
    this.VisitAllItems(item => { 
     bool itemSavedSuccessfully = item.SaveToDB(); 
     if(!itemSavedSuccessfully) { 
      //log error message 
     } 
    }); 
} 

それの使用の別の例は、

/// <summary> 
/// Get the number of special items in the list. 
/// </summary> 
protected int GetNumberOfUnsynchronisedItems() { 
    int numberOfSpecialItems = 0; 
    this.VisitAllItems((item) => { 
     numberOfSpecialItems += item.IsItemSpecial() ? 1 : 0; 
    }); 
    return numberOfSpecialItems; 
} 

だろう、それの例が使用だ

/// <summary> 
/// This visits all items and performs an action on them in a thread manner 
/// </summary> 
/// <param name="visitAction">The action to perform on the item</param> 
public void VisitAllItems(Action<Item> visitAction) { 
    lock (listLock) { 
     foreach (Item item in this.ItemList) { 
      visitAction.Invoke(item); 
     } 
    } 
} 

と呼ばれるのかもしれない。しかし、私はそこに確信していますAction<>デリゲートではなくFunc<>を使用してこの値を返すように、VisitAllItemsメソッドを書き込む方がよい方法です。私はいくつかのことを試しましたが、コンパイルエラーで終わります。

誰もこの方法を実装するためのより良い方法はありますか？

おかげで、 Alasdair

Answer 1

これは、すべての使用状況によって異なりますので、それは正確に何を伝えるのは難しいですあなたのために役立つでしょう。しかし、1つの方法は、コレクションの各アイテムに対して1つの結果を返し、LINQを使用して結果を結合することです。

public IEnumerable<TResult> VisitAllItems<TResult>(
    Func<Item, Result> visitfunction) 
{ 
    var result = new List<TResult>(); 
    lock (listLock) 
    { 
     foreach (Item item in ItemList) 
      result.Add(visitfunction(item)); 
    } 
    return result; 
} 

通常、私は手動でList<T>を作成するのではなく、yield returnを使用しますが、私はそれが理由lockの、この方法の方が良いと思います。

使い方は次のようになります：svickと@jonskeet両方に

protected int GetNumberOfUnsynchronisedItems() 
{ 
    return VisitAllItems(item => item.IsItemSpecial()) 
        .Count(isSpecial => isSpecial); 
} 

Answer 2

あなたはAggregateモデルに従うことができるが、使用するひどく快適ではないでしょう。

public TAccumulate VisitAllItems<TAccumulate>(TAccumulate seed, 
    Func<Item, TAccumulate, TAccumulate> visitor) { 
    TAccumulate current = seed; 
    lock (listLock) { 
     foreach (Item item in this.ItemList) { 
      current = visitor(current, item); 
     } 
    } 
    return current; 
} 

... 

protected int GetNumberOfUnsynchronisedItems() { 
    return VisitAllItems(0, 
     (count, item) => count + item.IsItemSpecial() ? 1 : 0); 
} 

をあなたが実際にどのように多くの異なる集計機能が必要なのでしょうか？たとえば、あなただけカウントしている時間のほとんど場合、あなたが必要になることがあります。その後、

public void VisitAndCount<TAccumulate>(Func<Item, bool> visitor) { 
    int count = 0; 
    lock (listLock) { 
     foreach (Item item in this.ItemList) { 
      if (visitor(item)) { 
       count++; 
      } 
     } 
    } 
    return count; 
} 

：

protected int GetNumberOfUnsynchronisedItems() { 
    return VisitAndCount(item => item.IsItemSpecial()); 
}