委托订阅取消不掉的问题？

newmankind 2013-10-18 10:53:41

string[] arr = GlobalStatic.marketChannel.dicMarketTransfer.Keys.ToArray();
foreach (string _InstrumentID in arr)
{
GlobalStatic.marketChannel.dicMarketTransfer[_InstrumentID].PushMarketInfoEvent -= dicMarketTransfer[_InstrumentID].ReciveMarketInfo;

}

GlobalStatic.marketChannel.dicMarketTransfer[_InstrumentID] 和 dicMarketTransfer[_InstrumentID]. 均继承了以下接口（已实现）
public interface IMarketTransfer
{
event PushMarketInfoEventHandler PushMarketInfoEvent;
void ReciveMarketInfo(DataRow drMarket);
}

按照官方文档，非匿名委托订阅，应该可以用-=取消的啊，为什么取消不掉，求解，谢谢！

...全文

317 11 打赏收藏转发到动态举报

写回复

用AI写文章

11 条回复

切换为时间正序

请发表友善的回复…

发表回复

qldsrx 2013-10-23

打赏
举报

引用 8 楼 u011303459 的回复:

//订阅
GlobalStatic.marketChannel.dicMarketTransfer[_instrumentID].PushMarketInfoEvent
                += new PushMarketInfoEventHandler(dicMarketTransfer[_instrumentID].ReciveMarketInfo);
//取消
GlobalStatic.marketChannel.dicMarketTransfer[_InstrumentID].PushMarketInfoEvent -= dicMarketTransfer[_InstrumentID].ReciveMarketInfo;
看看右边的项，订阅多了new，这样2个对象就不是同一个了。订阅改成下面的试试。 GlobalStatic.marketChannel.dicMarketTransfer[_instrumentID].PushMarketInfoEvent += dicMarketTransfer[_instrumentID].ReciveMarketInfo;

那样写没错，你自己对委托的概念不清楚，委托不是类，不能用类的思考方式去处理。

引用 9 楼 newmankind 的回复:

我疑惑就是注册和取消注册同时指向，同一个dictionary 的同一个实例的同一个函数，dicMarketTransfer[_InstrumentID].ReciveMarketInfo 应该这种写法表述不会有问题啊，为啥就取消不掉呢？

有几个疑点：一、RegEvent()方法内部有注册事件，但没有看到取消，虽然注册的是空方法，不代表你实际方法也是空的。二、pushMarketInfo可能为null，你却直接在ReciveMarketInfo内部使用，这是不安全的。三、看定义，dicMarketTransfer似乎不是静态的，GlobalStatic.marketChannel.dicMarketTransfer似乎又是静态的，这两者似乎不是同一个对象，这里面是否存在变量偷换的情况。四、GlobalStatic.marketChannel.dicMarketTransfer.Keys在注册和取消时，是否保证不曾发生过任何改变。

newmankind 2013-10-23

打赏
举报

我疑惑就是注册和取消注册同时指向，同一个dictionary 的同一个实例的同一个函数，dicMarketTransfer[_InstrumentID].ReciveMarketInfo 应该这种写法表述不会有问题啊，为啥就取消不掉呢？

feiniao19830822 2013-10-23

打赏
举报


//订阅
GlobalStatic.marketChannel.dicMarketTransfer[_instrumentID].PushMarketInfoEvent
                += new PushMarketInfoEventHandler(dicMarketTransfer[_instrumentID].ReciveMarketInfo);
//取消
GlobalStatic.marketChannel.dicMarketTransfer[_InstrumentID].PushMarketInfoEvent -= dicMarketTransfer[_InstrumentID].ReciveMarketInfo;

看看右边的项，订阅多了new，这样2个对象就不是同一个了。订阅改成下面的试试。 GlobalStatic.marketChannel.dicMarketTransfer[_instrumentID].PushMarketInfoEvent += dicMarketTransfer[_instrumentID].ReciveMarketInfo;

newmankind 2013-10-23

打赏
举报

顺序是，先初始化，填值，注册，（到此处这个流程正常的），取消注册，（这里取消不掉）。

newmankind 2013-10-23

打赏
举报

//本地 dicMarketTransfer 初始化 dicMarketTransfer = new Dictionary<string, IMarketTransfer>(); //本地 dicMarketTransfer 填值， if (!dicMarketTransfer.ContainsKey(InstrumentID)) { MarketTransfer mt = new MarketTransfer(InstrumentID); mt.Init(); dicMarketTransfer.Add(InstrumentID, mt); } //MarketTransfer 实现接口IMarketTransfer public class MarketTransfer:IMarketTransfer { //public ControlInfo controlInfo = null; public string InstrumentID = ""; public MarketTransfer(string InstrumentID_p) { InstrumentID = InstrumentID_p; //InitMarketUnit(InstrumentID); } public void Init() { RegEvent(); } public void RegEvent() { PushMarketInfoEvent += new PushMarketInfoEventHandler(PushMarketInfo); } public PushMarketInfoEventHandler pushMarketInfo = null; public event PushMarketInfoEventHandler PushMarketInfoEvent { add { pushMarketInfo += value; } remove { pushMarketInfo -= value; } } public void PushMarketInfo(DataRow drMarket) { } public void ReciveMarketInfo(DataRow drMarket) { try { Delegate [] DelegateGroup = pushMarketInfo.GetInvocationList(); //20130721在此处注入数据感觉不合适，应该作为传输的drmarket //如果下面挂着接收单位，则证明此时有效channel，否则可能是实虚切换后废channel //有效之后则处理markunit诸多变量，赶在异步发送之前 // if (DelegateGroup.Length > 0) //{ // GlobalStatic.marketPools.DicMarketUnits[InstrumentID].ReciveMarket(drMarket); //} foreach (PushMarketInfoEventHandler g in DelegateGroup) { g.BeginInvoke(drMarket,DelegMarketCallBack,g); } } catch (System.Exception ex) { GlobalStatic.logInfo.SaveExceptionLog(ex); } } public void DelegMarketCallBack(IAsyncResult asyncResult) { //到这儿委托已经在异步线程中执行完毕 PushMarketInfoEventHandler deleg = (PushMarketInfoEventHandler)asyncResult.AsyncState; //委托执行的异常会在EndInvoke时抛出来 try { //使用BeginInvoke时传入委托的EndInvoke获得计算结果，这时候计算结果已经出来了，有异常的话也在这儿抛出来 deleg.EndInvoke(asyncResult); } catch (OverflowException ofex) { GlobalStatic.logInfo.SaveExceptionLog(ofex); } } //private void InitMarketUnit(string InstrumentID) //{ // if (!GlobalStatic.marketPools.DicMarketUnits.ContainsKey(InstrumentID)) // { // GlobalStatic.marketPools.DicMarketUnits.Add(InstrumentID, new MarketUnit(InstrumentID)); // } //} }

newmankind 2013-10-23

打赏
举报

注册部分 string[] arr = GlobalStatic.marketChannel.dicMarketTransfer.Keys.ToArray(); foreach (string _InstrumentID in arr) { RealMarketHookMarketTransfer(_InstrumentID); } //引用函数 private static void HookMarketTransfer(string _instrumentID) { GlobalStatic.marketChannel.dicMarketTransfer[_instrumentID].PushMarketInfoEvent += new PushMarketInfoEventHandler(dicMarketTransfer[_instrumentID].ReciveMarketInfo); }

引用 4 楼 qldsrx 的回复:

注册和取消的是不同的函数，即使函数名相同，但出现了继承关系后，不同对象就会产生不同的函数委托，不可能一样。你把最关键的注册部分省略了，这也是你犯错的根本所在，以为那个地方没错，其实就是错在那里。

newmankind 2013-10-23

打赏
举报

非常感谢qldsrx及其余诸位帮助提醒，终于找到原因了，自己的问题，有一个bug，跟进去才发现，太粗心了，因为静态全局调，HookMarketTransfer 调用了两次，所以取消订阅一次，还有一次。这个错误确实低级，最后删掉多余的调用，在MarketTransfer，然后设置了一个内部函数算是仅一次注册的门禁: public void HookInside(PushMarketInfoEventHandler _PushMarketInfoEvent) { if (!pushMarketInfo.GetInvocationList().Contains(_PushMarketInfoEvent)) { pushMarketInfo += _PushMarketInfoEvent; } }

qldsrx 2013-10-20