CollectionChangedイベントでNotifyCollectionChangedActionをチェックする必要がありますか？

Question

CollectionChangedイベントに登録するときに、NotifyCollectionChangedEventArgsの NotifyCollectionChangedAction列挙をチェックする必要があるかどうかは疑問です。このようにそれをしない時に、私がつまずいたすべての例：

myCollection.CollectionChanged += (sender, eventArgs) => 
{ 
    if (eventArgs.Action == NotifyCollectionChangedAction.Add) 
    { 
     foreach (SampleClass sampleObject in eventArgs.NewItems) 
     { 
      addAction(sampleObject); 
     } 
    } 
    else if (eventArgs.Action == NotifyCollectionChangedAction.Remove) 
    { 
     foreach (SampleClass sampleObject in eventArgs.OldItems) 
     { 
      removeAction(sampleObject); 
     } 
    } 
    // ... 
}; 

はそれがNotifyCollectionChangedActionを無視して、ちょうどこのようなコードを簡素化することが可能です：

myCollection.CollectionChanged += (sender, eventArgs) => 
{ 
    eventArgs.NewItems?.OfType<SampleClass>() 
     .ToList() 
     .ForEach(addAction); 

    eventArgs.OldItems?.OfType<SampleClass>() 
     .ToList() 
     .ForEach(removeAction); 
}; 

このアイデアの欠点は何ですか？私が逃したものはありますか？

Answer 1

これらのコードサンプルは同等ではないため、実行しようとする内容によって異なります。ちょうどAddとRemoveより多くの行動タイプがあります。例えば、Replaceアクションがあります。私はこれを行う場合：

myCollection[0] = new MyObject(); 

CollectionChangedはOldItemsが交換アイテムが含まれている（旧myCollection [0]）とNewItemsnew MyObject()項目が含まれていますので、アクションタイプReplaceで起動されます。最初のコードサンプルはこのイベントを完全に無視します。 2番目のコードサンプルは、addActionとremoveActionの両方の項目を処理します。そうした場合：

myCollection.Move(0,1); 

これは、両方のOldItemsとNewItemsが移動した項目が含まれていますアクションMoveとイベントを発生します。最初のサンプルでは無視され、同じアイテム上でaddActionとremoveActionが実行され、驚くべき結果につながる可能性があります。