如何在Dispose()中正确处理IDisposable对象列表？异常排查与疑问

关于集合修改异常的原因与解决 #

你遇到的System.InvalidOperationException（集合已修改；枚举操作可能无法执行），即便构造函数和Dispose是唯一修改_mWatchers的地方，大概率是多线程并发访问冲突导致的：

• 场景1：类实例还在构造过程中（构造函数正在往集合添加元素），另一个线程就调用了Dispose方法，此时枚举集合的操作与添加元素的操作并发，触发异常。
• 场景2：Dispose被多个线程同时调用，第一个线程在枚举/修改集合时，第二个线程的枚举操作检测到集合已变更，抛出异常。

解决办法 #

1. 确保实例构造完成后再对外暴露，避免构造过程中被调用Dispose。
2. 给集合的所有访问操作加锁同步：

   private readonly object _syncLock = new object();
   private bool _isDisposed;
   
   // 构造函数中添加元素
   lock (_syncLock)
   {
       _mWatchers.Add(new FileSystemWatcher(...));
   }
   
   public void Dispose()
   {
       if (_isDisposed) return;
       lock (_syncLock)
       {
           if (_isDisposed) return;
           // 先复制到数组，避免迭代原集合时被修改
           var watchers = _mWatchers.ToArray();
           foreach (var watcher in watchers)
           {
               watcher.Dispose();
           }
           _mWatchers.Clear();
           _isDisposed = true;
       }
   }
   

3. 改用线程安全集合（如ConcurrentBag<FileSystemWatcher>），它允许并发修改与枚举，不会抛出此类异常（注意：枚举结果可能不反映最新集合状态）。

你的四个具体疑问解答 #

1. 是否需要在public Dispose()方法中调用GC.SuppressFinalize(this)？ #

不需要，除非你的类实现了终结器（~MyDisposableListOfFileSystemWatchers()）。这个方法的作用是告诉GC：该实例已经手动清理了资源，无需再执行终结器。如果你的类只管理托管资源（比如FileSystemWatcher是托管对象），根本不需要写终结器，自然也不需要调用这个方法。

2. 迭代集合时Dispose其中的对象是否可行？ #

直接用foreach迭代原集合并Dispose对象存在风险：如果Dispose过程中集合被其他线程修改，或者你在Dispose后从集合移除元素，都会触发集合修改异常。更安全的做法是先把集合元素复制到临时数组，再迭代数组执行Dispose，比如var watchers = _mWatchers.ToArray(); foreach (var w in watchers) w.Dispose();，这样原集合的变更不会影响迭代过程。

3. 是否可以清理集合，还是应交给垃圾回收器处理？ #

建议手动清理集合（比如调用_mWatchers.Clear()）。原因有二：一是手动清理可以立即释放集合对已Dispose对象的引用，让GC更早回收这些资源；二是避免后续代码误访问集合中已释放的FileSystemWatcher实例。

4. 释放作为事件发布者的FileSystemWatcher后，是否无需手动取消事件订阅？ #

是的。当FileSystemWatcher被Dispose且没有其他引用时，GC会回收它。如果订阅者是你的MyDisposableListOfFileSystemWatchers实例，当实例被Dispose并回收后，订阅关系会自动解除。唯一需要注意的情况：如果事件处理方法是静态方法，静态引用不会随实例回收，此时最好手动取消订阅；其他场景无需额外操作。

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