5

我有两种形式，一种是MainForm，第二种是DebugForm。MainForm 有一个按钮，可以像这样设置和显示 DebugForm，并将引用传递给已经打开的 SerialPort：

private DebugForm DebugForm; //Field
private void menuToolsDebugger_Click(object sender, EventArgs e)
{
    if (DebugForm != null)
    {
        DebugForm.BringToFront();
        return;
    }

    DebugForm = new DebugForm(Connection);

    DebugForm.Closed += delegate
    {
        WindowState = FormWindowState.Normal;
        DebugForm = null;
    };

    DebugForm.Show();
}

在 DebugForm 中，我附加了一个方法来处理DataReceived串行端口连接的事件（在 DebugForm 的构造函数中）：

public DebugForm(SerialPort connection)
{
    InitializeComponent();
    Connection = connection;
    Connection.DataReceived += Connection_DataReceived;
}

然后在Connection_DataReceived方法中，我在 DebugForm 中更新了一个 TextBox，即使用 Invoke 进行更新：

private void Connection_DataReceived(object sender, SerialDataReceivedEventArgs e)
{           
    _buffer = Connection.ReadExisting();
    Invoke(new EventHandler(AddReceivedPacketToTextBox));
}

但我有一个问题。一旦我关闭 DebugForm，它就会ObjectDisposedException在Invoke(new EventHandler(AddReceivedPacketToTextBox));Line 上抛出一个。

我怎样才能解决这个问题？欢迎任何提示/帮助！

更新

我发现如果我在按钮事件 click 中删除事件，并在该按钮单击中关闭表单，一切都很好，我的调试表单毫无例外地被关闭......多么奇怪！

private void button1_Click(object sender, EventArgs e)
{
    Connection.DataReceived -= Connection_DebugDataReceived;
    this.Close();
}
4

3 回答 3

6

关闭表单会处理 Form 对象，但不能强制删除其他类对其的引用。当您为事件注册表单时，您基本上是将表单对象的引用提供给事件的源（本SerialPort例中的实例）。

这意味着，即使您的表单已关闭，事件源（您的SerialPort对象）仍在向表单实例发送事件，并且处理这些事件的代码仍在运行。那么问题是，当这段代码尝试更新已处理的表单（设置其标题、更新其控件、调用Invoke等）时，您将收到此异常。

因此，您需要做的是确保在您的表单关闭时取消注册该事件。这就像检测表单正在关闭并取消注册Connection_DataReceived事件处理程序一样简单。您可以通过覆盖该OnFormClosing方法并在其中取消注册事件来方便地检测表单正在关闭：

protected override OnFormClosing(FormClosingEventArgs args)
{
    Connection.DataReceived -= Connection_DataReceived;
}

我还建议将事件注册移动到方法的覆盖，OnLoad否则它可能会在表单完全构造之前接收事件，这可能会导致混乱的异常。

于 2012-10-18T14:25:13.543 回答
2

您尚未显示该AddReceivedPacketToTextBox方法的代码。

您可以尝试在该方法中检查已处理的表单：

private void AddReceivedPacketToTextBox(object sender, EventArgs e)
{
    if (this.IsDisposed) return;

    ...
}

在关闭表单时分离DataReceived事件处理程序可能是一个好主意，但这还不够：仍然存在竞争条件，这意味着您AddReceivedPacketToTextBox可以在表单关闭/处置后调用。序列将类似于：

  • 工作线程：触发 DataReceived 事件，Connection_DataReceived 开始执行
  • UI 线程：表单关闭并释放，DataReceived 事件分离。
  • 工作线程：调用 Invoke
  • UI 线程：在处理表单时执行 AddReceivedPacketToTextBox。

我发现如果我在按钮事件 click 中删除事件，并在该按钮单击中关闭表单，一切都很好，我的调试表单毫无例外地被关闭......多么奇怪！

这并不奇怪。多线程错误（“Heisenbugs”）是与时间相关的，像这样的小变化会影响时间。但这不是一个强大的解决方案。

于 2012-10-20T14:47:58.510 回答
0

这个问题可以通过添加一个计时器来解决：

  bool formClosing = false;
    private void Connection_DataReceived(object sender, SerialDataReceivedEventArgs e)
    {
      if (formClosing) return;
      _buffer = Connection.ReadExisting();
      Invoke(new EventHandler(AddReceivedPacketToTextBox));
    }
    protected override void OnFormClosing(FormClosingEventArgs e)
    {
      base.OnFormClosing(e);
      if (formClosing) return;
      e.Cancel = true;
      Timer tmr = new Timer();
      tmr.Tick += Tmr_Tick;
      tmr.Start();
      formClosing = true;
    }
    void Tmr_Tick(object sender, EventArgs e)
    {
      ((Timer)sender).Stop();
      this.Close();
    }

感谢MSDN的 JohnWein

于 2012-10-18T19:30:45.637 回答