1

USBデバイスを使用しています。このデバイスはメッセージを受信しますが、いつ、どのくらいの頻度でメッセージを受信しますか。ドライバーに付属するAPIは、デバイスがメッセージを受信したときにコールバックを提供するsetreceiveCallBack関数を指定します。しかし、ランダムな時間または間隔で、garbagecollectedデリゲートの例外に関するコールバックを受け取ります。私は自分の問題の解決策を探しましたが、私の場合はどの解決策も機能していないようです。以下は私のコードの最大の部分です：

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.IO;
using System.Linq;
using System.Text;
using System.Runtime.InteropServices;
using System.Windows.Forms;

namespace CallBacktesting
{
    public unsafe delegate void callBack(Form1.CANMsg *pmsg);

    public partial class Form1 : Form
    {
        uint handle;
        static WriteLog log = new WriteLog();
        Boolean getCan = false;
        static int frameCount = 0;
        static CANMsg newmsg = new CANMsg();
        callBack _setCallBack;
        List<string> write = new List<string>();

        public Form1()
        {
            InitializeComponent();
        }


        private void buttonOpen_Click(object sender, EventArgs e)
        {
                // Open connection
        }

        private void buttonClose_Click(object sender, EventArgs e)
        {
               // Close connection
        }

        private void buttonCallBack_Click(object sender, EventArgs e)
        {
            if (!getCan)
            {
                int rv;
                unsafe
                {
                    callBack _setCallBack = new callBack(call);
                    rv = canusb_setReceiveCallBack(handle, _setCallBack);
                }
                label1.Text = rv.ToString();
            }
            else
            {
                _setCallBack = null;
                int rv = canusb_setReceiveCallBack(handle, _setCallBack);
                GC.KeepAlive(_setCallBack);
                label1.Text = rv.ToString();
            }
        }

        public unsafe void call(CANMsg *pmsg)
        {
            newmsg = *pmsg;
            update();
        }

        private void buttonExit_Click(object sender, EventArgs e)
        {
            GC.KeepAlive(_setCallBack);
            Application.Exit();
        }

        [DllImport("canusbdrv.dll", EntryPoint = "canusb_setReceiveCallBack")]
        public static extern int canusb_setReceiveCallBack(uint handle, callBack callBack);

        unsafe private void timer_Tick(object sender, EventArgs e)
        {
              // update the form with received messages
        }

        public void update()
        {
            CANMsg msgrec = newmsg;
            // Build str from messages with all data
            write.Add(str);
            log.logWrite(str);
            frameCount++;
        }
    }

    public class WriteLog
    {

        private void OpenFile()
        {     }

        public void logWrite(string log)
        {     }

        public void logAdd(string log)
        {     }

        private void logClose()
        {     }
    }
}
4

2 に答える 2

2

これは正しいですか/タイプミスですか？：

callBack setCallBack = new callBack(call);
rv = canusb_setReceiveCallBack(handle, call);

あなたはcallBackのインスタンスを作成しているように見えますが、それから何か他のものをcanusb_setReceiveCallBack渡します-setCallBack代わりに渡すつもりでしたか？

また、この行では、setCallBackをローカル変数として宣言しているため、のsetCallBack代わりに渡す場合でも、おそらくガベージコレクションされるローカルスコープの変数を渡すことになります（これを明示的に防ぐためにcall行うことに気づきました）GC.KeepAlive(setCallBack);

于 2009-11-30T13:30:16.470 に答える
2

あなたのコードで、あなたがするとき



                callBack setCallBack = new callBack(call);
                rv = canusb_setReceiveCallBack(handle, call);

コードのどこにもデリゲートが参照されていないため、「canusb_setReceiveCallBack」を呼び出した後、デリゲートはガベージ コレクションに使用できます。

プライベート フィールドに格納することで、これを回避できます。

元：


Class Form1
{

callBack _setCallBack;

private void buttonCallBack_Click(object sender, EventArgs e)
{


                _setCallBack = new callBack(call);
                rv = canusb_setReceiveCallBack(handle, _setCallBack);

}

}

ただし、ボタンをクリックするたびに新しいコールバックが作成されるため、これには問題が発生する可能性があります。以前のコールバックを参照する必要がある場合、これは問題になる可能性があります。

コードをリファクタリングして SafeHandle を使用し、 canusb_Openによって返されるハンドルを格納する必要があると思います。

このようなクラスを設計します。


class CanUsbSafeHandle : SafeHandle
{
    private EventHandler _receiveCallBack;
    private readonly object _receiveCallBackLock = new object();

    public event EventHandler ReceiveCallBack
    {
        add
        {
            lock (_receiveCallBackLock)
            {
                bool hasListeners = (_receiveCallBack != null);
                _receiveCallBack += value;
                //call canusb_setReceiveCallBack only when 1 or more listeners were added
                //and there were previously no listeners
                if (!hasListeners && (_receiveCallBack != null))
                {
                    canusb_setReceiveCallBack(this, setCallBack);
                }
            }
        }
        remove
        {
            lock (_receiveCallBackLock)
            {
                bool hasListeners = (_receiveCallBack != null);
                _receiveCallBack -= value;
                //call canusb_setReceiveCallBack only when there are no more listeners.
                if(hasListeners && (_receiveCallBack == null))
                {
                    canusb_setReceiveCallBack(this, null);
                }
            }
        }
    }

    public CanUsbSafeHandle()
        : base(IntPtr.Zero, true)
    {
    }

    public override bool IsInvalid
    {
        get { return handle == IntPtr.Zero; }
    }

    protected override bool ReleaseHandle()
    {
        return canusb_Close(handle);
    }

    protected override void Dispose(bool disposing)
    {
        if (disposing)
        {
            lock (_receiveCallBackLock)
            {
                _receiveCallBack = null;
            }
        }
        base.Dispose(disposing);
    }
}

そうすれば、SafeHandle は「受信コールバック」デリゲートの有効期間を管理し、SafeHandle によって管理されます。

于 2009-11-30T13:48:19.187 に答える