是否有必要在 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 将被触发，动作类型为Replace，OldItems 将包含替换项（旧的 myCollection[0]），NewItems 将包含new MyObject() 项。第一个代码示例将完全忽略此事件。第二个代码示例将使用addAction 和removeAction 处理这两个项目。如果你这样做：

myCollection.Move(0,1);

它将触发带有动作Move 的事件，其中OldItems 和NewItems 都将包含移动的项目。第一个样本将再次忽略它，第二个样本将在移动的同一项目上执行 addAction 和 removeAction，我猜这可能会导致令人惊讶的结果。