使用BindingOperations.EnableCollectionSynchronization后，ICollectionView是否需加锁？

结论：在DoesViewContainShape方法中，必须使用同一个_lock对象加锁，原因如下：

• CollectionViewSource的View并非独立维护一份内部列表，它是基于源集合（即你的_shapes）构建的视图层，所有查询、过滤操作最终都会关联到源集合的状态。
• BindingOperations.EnableCollectionSynchronization仅负责让WPF的绑定系统在访问源集合时自动获取锁，但你手动调用View的Contains方法不属于绑定系统的自动处理流程，必须手动加锁来保证线程安全。
• 多线程环境下，如果源集合正在被修改（比如AddShape方法中的操作），此时调用View的方法可能会出现集合枚举冲突、数据不一致等线程安全问题，加锁能避免这类情况。

修改后的DoesViewContainShape方法示例：

public bool DoesViewContainShape(Shape shape)
{
    lock(_lock)
        return _cvsShapes.View.Contains(shape);
}

额外注意：

• 所有直接或间接涉及源集合的操作（包括View的查询、遍历、刷新等），都需要持有同一个锁，确保线程同步。
• 过滤器FilterShapes方法通常会在UI线程执行（由WPF绑定系统触发），但如果有其他线程触发视图刷新，也要确保过滤器内的操作不会引发线程问题——不过因为源集合已通过锁同步，这部分风险会大幅降低。

内容的提问来源于stack exchange，提问作者Joe