记一个 BUG 的排查过程:一个类里有一个成员方法,用着用着, GC 把这个方法杀了,类实例还活着 - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
yulitian888
V2EX    C#

记一个 BUG 的排查过程:一个类里有一个成员方法,用着用着, GC 把这个方法杀了,类实例还活着

  •  
  •   yulitian888 2019-06-18 12:48:00 +08:00 8700 次点击
    这是一个创建于 2360 天前的主题,其中的信息可能已经有所发展或是发生改变。

    挺有意思的一个 bug,症状是运行了一小段时间之后,程序进程直接死掉,哪怕所有代码都用了 try...catch 都管不住。

    故障发生的原因在于, [直接] 使用了实例方法当作非托管函数的回调函数参数使用。 类代码如下: public class SnapInfo { internal int CallBack_KaKou(IntPtr lAnalyzerHandle, uint dwEventType, IntPtr pEventInfo, IntPtr pBuffer, uint dwBufSize, IntPtr dwUser, int nSequence, IntPtr reserved) { ...... }

    }

    调用大华卡口摄像头抓拍照片并识别车牌的时候用的,调用 SDK 代码如下: NETClient.RealLoadPicture(loginId, ChannelNum, (uint)EM_EVENT_IVS_TYPE.TRAFFIC_MANUALSNAP, true, currentSnap.CallBack_KaKou, loginId, IntPtr.Zero);

    其中 currentSnap 是 SnapInfo 类的实例。 执行一段时间之后,呵呵呵~~~

    解决办法:

    public class SnapInfo {

    public SnapInfo(CameraDevice cameraDevice) { this.卡口回调 = new fAnalyzerDataCallBack(this.CallBack_KaKou); }

    public fAnalyzerDataCallBack 卡口回调 { get; private set; }

    internal int CallBack_KaKou(IntPtr lAnalyzerHandle, uint dwEventType, IntPtr pEventInfo, IntPtr pBuffer, uint dwBufSize, IntPtr dwUser, int nSequence, IntPtr reserved) { ...... }

    }

    把成员方法包装成成员属性,对非托管方法传参的时候替换为属性“卡口回调”,问题解决。

    挺有趣的一个排错过程,记一下

    4 条回复    2019-06-18 16:01:29 +08:00
    geelaw
        1
    geelaw  
       2019-06-18 12:58:09 +08:00 via iPhone
    这并不会让你的代码正确。

    真正的原因是非托管代码和 GC 之间互相不认识,所以你需要 GC.KeepAlive(currentSnap) 或其他做得 currentSnap 仍然可达的方式来确保 currentSnap 在成员方法没有被取消监听之前不会被回收。
    yulitian888
        2
    yulitian888  
    OP
       2019-06-18 13:21:18 +08:00
    @geelaw GC.KeepAlive(currentSnap) 和 GC.KeepAlive(currentSnap.CallBack_KaKou) 都试过了,修改成属性才是最后生效的方式。
    实际上,使用 KeepAlive 甚至都没有能够做到延缓进程死亡时间的效果。
    当委托作为成员属性存在的时候,只要实例不被回收,GC 是不会对单独的属性下黑手的。而 GC 显然是在形参实参传递的过程中误判了委托的引用关系才会杀掉方法的引用。
    geelaw
        3
    geelaw  
       2019-06-18 14:45:23 +08:00
    @yulitian888 #2 你需要的是

    DelegateType dlg = new DelegateType(currentSnap.CallBack_KaKou);
    ExternalObject.SetCallback(dlg);
    ...
    GC.KeepAlive(dlg);

    因为是委托对象被咔擦了。把委托对象作为属性是一种不符合对象本身表达意思的方式。
    yulitian888
        4
    yulitian888  
    OP
       2019-06-18 16:01:29 +08:00
    @geelaw 然而你说的这种写法并没有生效,这也是我第一时间想到的写法,然并卵
    关于     帮助文档     自助推广系统     博客 &nbs;   API     FAQ     Solana     3200 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 27ms UTC 11:31 PVG 19:31 LAX 03:31 JFK 06:31
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86