7

我有一个扩展方法来订阅PropertyChanged实现的对象的事件INotifyPropertyChanged

我希望事件只触发一次。不多。

这是我的方法。

public static void OnPropertyChanged<T>(this  INotifyPropertyChanged target, string    propertyName, Action action)
{
    if (target == null)
    {
        return;
    }

    PropertyChangedEventHandler handler = (obj, e) =>
    {

        if (propertyName == e.PropertyName)
        {
            action();
        }

    };


    target.PropertyChanged -= handler;
    target.PropertyChanged += handler;

}

但它不起作用。我无法删除事件处理程序,因此每次调用此方法时都会触发事件。

我尝试了不同的方法。而不是使用匿名方法,而是使用更传统的方法,例如:

public static void OnPropertyChanged<T>(this  INotifyPropertyChanged target, string    propertyName, Action action)
{
    if (target == null)
    {
        return;
    }

    target.PropertyChanged -= target_PropertyChanged;
    target.PropertyChanged += target_PropertyChanged;

}

static void target_PropertyChanged(object sender, PropertyChangedEventArgs e)
    {
        //do stuff here
    }

它工作正常。该事件只触发一次,但我还需要 Action 参数。我不能用这种方法使用它。

任何解决方法或不同的方法来解决这个问题?静态方法中的匿名方法有什么奇怪的吗?

提前致谢。

4

3 回答 3

3

这是使用匿名方法作为事件处理程序的限制。它们不能像普通方法(技术上是通过方法组转换自动创建的委托实例)那样被删除,因为匿名方法被编译到编译器生成的容器类中,并且每次都会创建该类的新实例。

为了保留 action 参数,您可以创建一个容器类,其中包含事件处理程序的委托。该类可以在您正在使用的其他类的内部声明为私有 - 或设为内部,可能在“Helpers”命名空间中。它看起来像这样:

class DelegateContainer
{
    public DelegateContainer(Action theAction, string propName)
    {
         TheAction = theAction;
         PopertyName = propName;
    }

    public Action TheAction { get; private set; }
    public string PropertyName { get; private set; }

    public void PropertyChangedHandler(object sender, PropertyChangedEventArgs e)
    {
        if(PropertyName == e.PropertyName)
            TheAction();
    }
}

然后,在您的类中创建并存储对容器的引用。您可以创建一个静态成员currentContainer,然后像这样设置处理程序:

private static DelegateContainer currentContainer;

public static void OnPropertyChanged<T>(this  INotifyPropertyChanged target, string    propertyName, Action action)
{
   if (target == null)
   {
       return;
   }

   if(currentContainer != null)         
       target.PropertyChanged -= currentContainer.PropertyChangedHandler;

   currentContainer = new DelegateContainer(action, propertyName);
   target.PropertyChanged += currentContainer.PropertyChangedHandler;
}
于 2013-02-11T16:49:49.560 回答
3

如果您从事件处理程序本身中取消订阅,您可以让您的第一个示例工作。

public static void OnPropertyChanged<T>(this  INotifyPropertyChanged target, string    propertyName, Action action)
{
    if (target == null)
    {
        return;
    }

    // Declare the handler first, in order to create
    // a concrete reference that you can use from within
    // the delegate
    PropertyChangedEventHandler handler = null;  
    handler = (obj, e) =>
    {
        if (propertyName == e.PropertyName)
        {
            obj.PropertyChanged -= handler; //un-register yourself
            action();
        }

    };
    target.PropertyChanged += handler;
}

上面的代码用作“一劳永逸”的事件处理程序。您可以注册无限数量的这些,并且每个在取消注册之前只会执行一次。

请记住,如果您在多个线程中短时间连续引发事件,则这些处理程序之一可能会多次执行。为了防止这种情况,您可能需要创建一个静态Dictionary(T,T)映射对象实例到“锁定对象”,并添加一些哨兵代码以确保处理程序只执行一次。但是,这些实现细节似乎有点超出您当前编写的问题的范围。

于 2013-02-11T16:52:08.643 回答
0

从技术上讲,这与您尝试取消订阅的匿名方法不同。.NET 每次OnPropertyChanged调用时都会创建该方法的新实例。这就是为什么取消订阅不起作用的原因。

于 2013-02-11T16:51:20.707 回答