使用SerialPort和C＃中的线程“关闭了安全句柄”

Question

大家下午好！

我有这个线程化的SerialPort包装器，它从串行端口读取一行。 这是我的线程代码。

protected void ReadData()
{
     SerialPort serialPort = null;
     try
     {
         serialPort = SetupSerialPort(_serialPortSettings);
         serialPort.Open();

         string data;
         while (serialPort.IsOpen)
         {
             try
             {

                 data = serialPort.ReadLine();
                 if (data.Length > 0)
                     ReceivedData(serialPort, new ReceivedDataEventArgs(data));

             }
             catch (TimeoutException)
             {
                 //  No action
             }
         }
     }
     catch (ThreadAbortException)
     {
         if (serialPort != null)
             serialPort.Close();
     }
}

当我调用myThread.Abort(); 我得到一个例外（没有代码的行或引用）“安全句柄已关闭”。 谁能发现我做错了什么？ 谢谢。

顺便说一下，我有一个Start()和一个Stop()来创建线程，并尊重地中止线程。

Answer 1

我怀疑这是因为你使用Thread.Abort来结束线程 - 这通常是不赞成的。 中止它时的线程行为是不可预测的。 因此，由于串行端口是本机代码的包装器，因此存在本机资源（由.NET中的SafeHandle表示），这些资源会被意外处理掉，因此您将获得异常。

您可以考虑这样的线程会发生什么：

• 你开始你的线程
• 打开串口（分配本机资源并使用SafeHandle来保存这些资源）
• 你开始从串口读取
• 然后在某些时候（你的线程意外），你调用Thread.Abort
• 很可能你的线程中的代码就是在那一点上试图访问串口（读取数据）
• 线程被杀死并且串行端口句柄被隐式销毁
• 你从串口的ReadLine（）函数内部的代码中抛出异常，因为它具有的句柄不再有效

你真的应该使用一种不同的方法来中止线程，这样你就有机会关闭和处理串口。

可以使用如下的ManualResetEvent实现关闭线程的正确方法：

protected ManualResetEvent threadStop = new ManualResetEvent(false);

protected void ReadData()
{
     SerialPort serialPort = null;
     try
     {
         serialPort = SetupSerialPort(_serialPortSettings);
         serialPort.Open();

         string data;
         while (serialPort.IsOpen)
         {
             try
             {

                 data = serialPort.ReadLine();
                 if (data.Length > 0)
                     ReceivedData(serialPort, new ReceivedDataEventArgs(data));

             }
             catch (TimeoutException)
             {
                 //  No action
             }

             // WaitOne(0) tests whether the event was set and returns TRUE
             //   if it was set and FALSE otherwise.
             // The 0 tells the manual reset event to only check if it was set
             //   and return immediately, otherwise if the number is greater than
             //   0 it will wait for that many milliseconds for the event to be set
             //   and only then return - effectively blocking your thread for that
             //   period of time
             if (threadStop.WaitOne(0))
                 break;
         }
     }
     catch (Exception exc)
     {
         // you can do something here in case of an exception
         // but a ThreadAbortedException should't be thrown any more if you
         // stop using Thread.Abort and rely on the ManualResetEvent instead
     }
     finally
     {
         if (serialPort != null)
             serialPort.Close();
     }
}

protected void Stop()
{
    // Set the manual reset event to a "signaled" state --> will cause the
    //   WaitOne function to return TRUE
    threadStop.Set();
} 

当然，当使用events方法来停止线程时，您必须小心地在所有长时间运行的循环或任务中包含事件状态检查 。 如果你不这样做，你的线程似乎不会响应你的设置事件 - 直到它退出长时间运行的循环，或任务并有机会“看到”事件已被设置。

Answer 2

遇到类似的情况，我尝试在单个方法的本地创建串行端口连接。

想法是创建最多四个serialPort对象（对于每个串行端口）。 一旦其中一个端口返回了良好的数据，我就知道我的设备连接到哪个端口。 我将丢弃所有“试用”对象并创建与特定COM端口的新连接。

每次通过该方法我都会创建/丢弃我的serialPort对象。 哎呀。 碰巧的是，如果GC在我再次调用该方法时没有运行，它将尝试创建第二个串行连接，替换第一个连接并且它们将发生冲突。 那会触发这个错误。

解决方案：使所有串行端口连接对象全局到类。 丑陋的哈克：是的，但它有效