BindingOperations.EnableCollectionSynchronization子集合跨线程使用方案

现有方案的潜在缺陷 #

你当前通过统一接口+全局集合注册事件自动开启同步的思路是可行的，也是生产环境比较常用的实现方式，但存在几个容易踩的陷阱：

• 锁逻辑脱节问题：EnableCollectionSynchronization方法仅能保证WPF绑定引擎读取、遍历集合时会使用你传入的锁对象，不会自动为集合本身的增删改操作加锁。如果业务代码里有任何修改集合的逻辑没有对SyncRoot加锁，依然会出现枚举修改冲突、跨线程访问UI对象异常。
• 内存泄漏与重复注册问题：CollectionRegistering是全局静态事件，如果你在短生命周期的视图、控件中注册该事件后没有手动注销，会导致关联对象无法被垃圾回收。此外同一个集合实例如果多次调用EnableCollectionSynchronization，会重复注册同步回调，额外增加锁开销，极端场景下甚至会触发死锁。
• 子集合注册时机滞后问题：嵌套在父集合中的子集合，只有第一次被WPF绑定引擎访问到时才会触发注册事件。如果子集合在首次被UI加载前，就已经收到后台线程的更新操作，此时同步机制还未生效，依然会抛出跨线程更新异常。
• 类型误判问题：你最初定义的ISynchronized接口继承自非泛型IEnumerable，如果有实现该接口但本质不是可绑定集合的类型（比如自定义迭代器、延迟执行的LINQ序列）被误识别，会给不需要同步的对象错误开启同步机制，带来不必要的性能损耗。

修正后的实现要点 #

• 所有实现ISynchronized的自定义集合，需要在内部所有修改数据的方法中自动对SyncRoot加锁，不要把加锁逻辑交给外部调用方，从根源避免漏锁：

public interface ISynchronized
{
    object SyncRoot { get; }
}

public class SynchronizedObservableCollection<T> : ObservableCollection<T>, ISynchronized
{
    public object SyncRoot { get; } = new object();

    protected override void InsertItem(int index, T item)
    {
        lock (SyncRoot)
        {
            base.InsertItem(index, item);
        }
    }

    protected override void RemoveItem(int index)
    {
        lock (SyncRoot)
        {
            base.RemoveItem(index);
        }
    }

    protected override void SetItem(int index, T item)
    {
        lock (SyncRoot)
        {
            base.SetItem(index, item);
        }
    }

    protected override void ClearItems()
    {
        lock (SyncRoot)
        {
            base.ClearItems();
        }
    }

    protected override void MoveItem(int oldIndex, int newIndex)
    {
        lock (SyncRoot)
        {
            base.MoveItem(oldIndex, newIndex);
        }
    }
}

• 增加已注册集合的缓存判断，避免重复注册，同时在对应视图生命周期结束时注销全局事件，避免内存泄漏：

// 存储已经开启过同步的集合实例，不会阻止对象GC回收
private static readonly ConditionalWeakTable<IEnumerable, object> _syncedCollections = new();

private void OnCollectionRegistering(object sender, CollectionRegisteringEventArgs e)
{
    if (e.Collection is ISynchronized syncTarget && !_syncedCollections.TryGetValue(syncTarget, out _))
    {
        BindingOperations.EnableCollectionSynchronization(syncTarget, syncTarget.SyncRoot);
        _syncedCollections.Add(syncTarget, null);
    }
}

// 在视图卸载逻辑中注销事件，例如Window.Unloaded、Page.Unloaded事件中执行
// BindingOperations.CollectionRegistering -= OnCollectionRegistering;

• 对于创建后立刻会被后台线程修改的子集合，不要等绑定事件触发才注册同步，在集合实例创建完成后就主动调用EnableCollectionSynchronization完成注册，避免注册空窗期的跨线程异常。
• 收紧ISynchronized接口的实现范围，仅让需要跨线程更新的可绑定集合实现该接口，不要给普通IEnumerable类型扩展该接口，避免误判。

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