WeakReference - 我做得对吗？

Question

我有一个静态类，它公开一个事件：

public static class MyStaticClass
{
    static bool myBool= false;
    public static bool MyBool
    {
        get { return myBool; }
        private set
        {
            myBool= value;

            var handler = MyBoolChanged;
            if (handler != null)
                handler(null, null);
        }
    }

    public static event EventHandler MyBoolChanged;
}

然后我使用以下模式注册它：

class AnotherClass
{    
    WeakReference _me;

    public MyMethodInAnotherClass()
    {
        _me = new WeakReference(this);
        MyStaticClass.MyBoolChanged+= 
                (_me.Target as AnotherClass).MyMethodInAnotherClassCallback;    
    }

    private void MyMethodInAnotherClassCallback(some arguments)
    {
    }
}

我想要实现的是 MyStaticClass 仅在 AnotherClass 的实例存在时才执行处理程序 尚未处置（且尚未注销）。

Answer 1

我认为使用它的最好方法是忘记一个事件，并使用某种列表来代替；假设List；然后，您可以使用：

interface IFoo {
    void Bar(some args);
}

with：

static class Whatever {
    private static readonly List<WeakReference> items=new List<WeakReference>();
    public static void Add(IFoo foo) {
        if(foo != null) {
            var newRef = new WeakReference(foo);
            lock(items) { items.Add(newRef); }
        }
    }
    public static void DoIt(some args) {
        lock(items) {
           foreach(var item in items) {
              IFoo foo = item.IsAlive ? item.Target as IFoo : null;
              if(foo != null) foo.Bar(some args);
           }
        }
    }
}

以及附加机制来删除特定的 IFoo，并删除所有剩余的死 foo 待办事项。

然后，您只需要 AnotherClass : IFoo，以及应用您的回调的 Bar() 实现。

额外强调：静态集合（包括事件）是相当危险的；您必须偶尔进行某种清理以删除空项目，并在可能的情况下尝试立即取消订阅（例如，在 Dispose() 中）。举例来说：

public static void Remove(IFoo foo) {
    lock (items) { // also remove any dead debris
        items.RemoveAll(x => !x.IsAlive || x.Target == foo || x.Target == null);
    }
}