V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
yulitian888
V2EX  ›  C#

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

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

    挺有意思的一个 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 然而你说的这种写法并没有生效,这也是我第一时间想到的写法,然并卵
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   2559 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 24ms · UTC 15:56 · PVG 23:56 · LAX 08:56 · JFK 11:56
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.