C#：方法调用永不返回

Question

我有一个永远不会返回的线程调用调用。

线程一直运行良好，直到我以这种方式调用该行，“owner.Invoke(methInvoker);”

调试的时候，我可以慢慢地一步，一步，一步，但是一旦我点击owner.Invoke...就结束了！

Control owner;
public event ReportCeProgressDelegate ProgressChanged;

public void ReportProgress(int step, object data) {
  if ((owner != null) && (ProgressChanged != null)) {
    if (!CancellationPending) {
      ThreadEventArg e = new ThreadEventArg(step, data);
      if (owner.InvokeRequired) {
        MethodInvoker methInvoker = delegate { ProgressChanged(this, e); };
        owner.Invoke(methInvoker);
      } else {
        ProgressChanged(this, e);
      }
    } else {
      mreReporter.Set();
      mreReporter.Close();
    }
  }
}

仅供参考：这是一个模仿 BackgroundWorker 类的自定义类，它在没有表单的控件上不可用。

认为Invoke可能不需要，我在调试器中手动将光标移到那部分代码上并尝试直接调用ProgressChanged，但是VS2010的调试器抛出了跨线程异常。

编辑：

由于我收到了前 3 个 cmets，我想用我的 ProgressChanged 方法更新：

worker.ProgressChanged += delegate(object sender, ThreadEventArg e) {
  if (progressBar1.Style != ProgressBarStyle.Continuous) {
    progressBar1.Value = 0;
    object data = e.Data;
    if (data != null) {
      progressBar1.Maximum = 100;
    }
    progressBar1.Style = ProgressBarStyle.Continuous;
  }
  progressBar1.Value = e.ProgressPercentage;
};

匿名方法的第一行有一个断点，但它也永远不会被命中。

编辑 2

这里是线程调用的更完整列表：

List<TableData> tList = CollectTablesFromForm();
if (0 < tList.Count) {
  using (SqlCeReporter worker = new SqlCeReporter(this)) {
    for (int i = 0; i < tList.Count; i++) {
      ManualResetEvent mre = new ManualResetEvent(false);
      worker.StartThread += SqlCeClass.SaveSqlCeDataTable;
      worker.ProgressChanged += delegate(object sender, ThreadEventArg e) {
        if (progressBar1.Style != ProgressBarStyle.Continuous) {
          progressBar1.Value = 0;
          object data = e.Data;
          if (data != null) {
            progressBar1.Maximum = 100;
          }
          progressBar1.Style = ProgressBarStyle.Continuous;
        }
        progressBar1.Value = e.ProgressPercentage;
      };
      worker.ThreadCompleted += delegate(object sender, ThreadResultArg e) {
        Cursor = Cursors.Default;
        progressBar1.Visible = false;
        progressBar1.Style = ProgressBarStyle.Blocks;
        if (e.Error == null) {
          if (e.Cancelled) {
            MessageBox.Show(this, "Save Action was Cancelled.", "Save Table " + tList[i].TableName);
          }
        } else {
          MessageBox.Show(this, e.Error.Message, "Error Saving Table " + tList[i].TableName, MessageBoxButtons.OK, MessageBoxIcon.Error);
        }
        mre.Set();
      };
      worker.RunWorkerAsync(tList[i]);
      progressBar1.Value = 0;
      progressBar1.Style = ProgressBarStyle.Marquee;
      progressBar1.Visible = true;
      Cursor = Cursors.WaitCursor;
      mre.WaitOne();
    }
  }
}

我希望这不是矫枉过正！我讨厌提供太多信息，因为那样我会让人们批评我的风格。 :)

Answer 1

  worker.RunWorkerAsync(tList[i]);
  //...
  mre.WaitOne();

这是一个有保证的死锁。您传递给 Control.Begin/Invoke() 的委托只能在 UI 线程空闲且重新进入消息循环时运行。您的 UI 线程不是空闲的，它在 WaitOne() 调用中被阻塞。在您的工作线程完成之前，该调用无法完成。在 Invoke() 调用完成之前，您的工作线程无法完成。在 UI 线程空闲之前，该调用无法完成。死锁城市。

阻塞 UI 线程从根本上说是错误的做法。不仅仅是因为 .NET 管道，COM 已经要求它永不阻塞。这就是 BGW 有 RunWorkerCompleted 事件的原因。

Answer 2

您很可能已经死锁了 UI 和工作线程。 Control.Invoke 通过将消息发布到 UI 线程的消息队列，将委托的执行编组到 UI 线程，然后等待该消息被处理，这反过来意味着委托的执行必须在 Control.Invoke 返回之前完成。但是，如果您的 UI 线程忙于分派和处理消息之外的其他事情怎么办？我可以从您的代码中看到ManualResetEvent 可能在这里起作用。您的 UI 线程是否在偶然调用 WaitOne 时被阻塞？如果是这样，那肯定是问题所在。由于WaitOne 不发送消息，它会阻塞 UI 线程，当从工作线程调用 Control.Invoke 时会导致死锁。

如果您希望 ProgressChanged 事件的行为与 BackgroundWorker 一样，那么您需要调用 Control.Invoke 以将这些事件处理程序放到 UI 线程上。这就是BackgroundWorker 的工作方式。当然，只要您准备好让调用者在处理 ProgressChanged 事件时进行自己的编组，您就必须在这方面完全模仿 BackgroundWorker 类。