在失败时正确删除 WCF 回调的模式

Question

我有一个存储连接实例的静态列表的类。在回调中，列表中的每个实例都会被调用，如果调用失败，该实例将从列表中删除。

细节很复杂，因为添加、删除和枚举列表项时需要同步。我目前正在使用这种模式：

class Foo : IDisposable
{
    private static readonly List<Foo> _Connections = new List<Foo> ();
    private readonly IFooCallback CallbackChannel;

    internal Foo ()
    {
        CallbackChannel = OperationContext.Current.GetCallbackChannel<IFooCallback> ();

        lock (_Connections) {
            _Connections.Add (this);
            OperationContext.Current.Channel.Closing += (s, e) => Dispose ();
            OperationContext.Current.Channel.Faulted += (s, e) => Dispose ();
        }
    }

    public void Dispose ()
    {
        lock (_Connections) {
            _Connections.Remove (this);
        }
    }

    private void RaiseCallback ()
    {
        List<Foo> connections;
        lock (_Connections) {
            connections = new List<Foo> (_Connections);
        }
        foreach (var con in connections) {
            try {
                con.CallbackChannel.SomeCallback ();
            }
            catch (CommunicationException) {
                OperationContext.Current.Channel.Abort ();
            }
            catch (TimeoutException) {
                OperationContext.Current.Channel.Abort ();
            }
        }
    }
}

我的想法：

1. 实例的静态列表，每个实例都存储 CB 通道。
2. 当通道关闭或中止时，实例将被删除。
3. 在回调时，将创建（同步）并枚举（不同步）列表的副本。
4. 当回调失败时，通道将中止。

失败的回调会导致通道中止，进而导致 Dispose 并从列表中删除。这可能会也可能不会发生在同一线程上（无法保证，因为可能会因不同的事件而随时引发 Dispose）。

我的问题是，这是处理实例存储和回调故障处理的常见模式，或者如何改进？在 RaiseCallback 中枚举之前复制列表是一个错误，还是正确的？