事件处理程序未取消订阅

Question

在我的silverlight应用程序中，我将方法GetListCallBack传递给Repository类中的另一个方法GetEmployees的委托参数，该参数将该委托作为事件处理程序附加到异步服务调用的完成事件。

EmpViewModel类：

public class EmpViewModel 
{ 
    private IRepository EMPRepository = null; 

    //constructor 
    public EmpViewModel 
    { 
    this.EMPRepository= new Repository(); 
    } 

    public void GetList() 
    { 
    this.EMPRepository.GetEmployees(xyz, this.GetListCallBack); 

    } 

    public void GetAnotherList() 
    { 
    this.EMPRepository.GetEmployees(pqr, this.GetAnotherListCallBack); 

    } 


    private void GetListCallBack(object sender, GetListCompletedEventArgs args) 
    { 
     if (args.Error == null) 
     { 
      this.collection1.Clear(); 
      this.collection1 = args.Result; 
     } 
     else 
     { 
      //do sth 
     } 
    } 

    public void GetAnotherListCallback(object sender, GetListCompletedEventArgs args) 
    { 
    //do sth with collection1 

    } 

} 

仓储类：

public class Repository : IRepository 
{ 

    private readonly ServiceClient _client=null ; 

    public Repository() 
    { 
     _client = new ServiceClient(Binding, Endpoint); 
    } 

    public void GetEmployees(int xyz, EventHandler<GetListCompletedEventArgs> eventHandler) 
    { 
     _client.GetListCompleted -= eventHandler; 
     _client.GetListCompleted += new EventHandler<GetListCompletedEventArgs>(eventHandler); 
     _client.GetListAsync(xyz); 
    } 
} 

现在，当GetList()已经完成，然后，如果我调用另一个方法GetAnotherList()在同一类EmpViewModel，则该方法的调用在调用GetAnotherListCallBack之前，再次调用方法GetListCallBack。

这可能是因为这两种方法都订阅了该事件。

正如你所看到的，我已经明确地从回调事件中取消订阅事件处理器，但仍然调用事件处理器。 任何人都可以请建议我可能会出错的地方吗？

编辑：

当我使用的，而不是使用this.EMPRepository调用Repository方法效果很好既是回调方法传递给Repository类的不同实例，只有attched回调方法被炒鱿鱼

一个局部变量

public class EmpViewModel 
{ 

public void GetList() 
{ 
    EMPRepository = new Repository(); 
    EMPRepository.GetEmployees(xyz, this.GetListCallBack); 
} 

public void GetAnotherList() 
{ 
EMPRepository = new Repository(); 
EMPRepository.GetEmployees(pqr, this.GetAnotherListCallBack); 
} 

-------- 

Answer 1

第一：

_client.GetListCompleted += new EventHandler<GetListCompletedEventArgs>(eventHandler); 

被（从功能的POI看到nt of view）相同：

_client.GetListCompleted += eventHandler; 

而现在你马上看到了问题。您的代码是：

_client.GetListCompleted -= eventHandler; 
_client.GetListCompleted += eventHandler; 

为什么要删除事件处理程序，如果您将其添加到下一行中。

我想你想删除旧的事件处理程序并添加一个新的。所以你的函数应该让一个委托给旧的事件处理器来删除。类似这样的：

public void GetEmployees(int xyz, EventHandler<GetListCompletedEventArgs> oldEventHandler, EventHandler<GetListCompletedEventArgs> newEventHandler) 
{ 
    _client.GetListCompleted -= oldEventHandler; 
    _client.GetListCompleted += newEventHandler; 
    _client.GetListAsync(xyz); 
} 

但是，这甚至可能吗？

如果你有控制权ServiceClient.GetListCompleted，为什么不删除event关键字，只是分配一个委托，如：

public void GetEmployees(int xyz, EventHandler<GetListCompletedEventArgs> eventHandler) 
{ 
    _client.GetListCompleted = eventHandler; 
    _client.GetListAsync(xyz); 
} 

或者......如果委托只调用一次每GetListASync：

public void GetEmployees(int xyz, EventHandler<GetListCompletedEventArgs> eventHandler) 
{ 
    _client.GetListCompleted += (sender, args) => 
    { 
     eventHandler(sender, e); 
     _client.GetListCompleted -= eventHandler; 
    }; 
    _client.GetListAsync(xyz); 
}