BackgroundWorker 在 DoWork 之前完成

Question

我正在使用后台工作程序来处理文件的加载以阻止我的 ui 冻结但是似乎RunWorkerCompleted 在我的DoWork 事件完成之前完成（退出对话框时导致错误）.. .有什么我做错了吗？我是否最好在一项任务中执行此操作？

public static <T> LoadDesign(string xmlPath)
{
    PleaseWait pw = new PleaseWait(xmlPath);
    pw.ShowDialog();
    return pw.design;
}


private PleaseWait(string xmlFile)
{
   InitializeComponent();
   bw = new BackgroundWorker();
   bw.WorkerSupportsCancellation = true;
   bw.DoWork += (s, e) =>
   {
      design = (Cast)DllCall containing XmlSerializer.Deserialize(...,xmlFile);
   };
   bw.RunWorkerCompleted += (s, e) => {
   //Exit please wait dialog
      this.Close(); 
   };
   if (!bw.IsBusy)
       bw.RunWorkerAsync();
}

我认为问题可能归结于我的后台工作人员正在调用 dll 并且没有等待响应。我尝试添加诸如while(design == null) 之类的检查无济于事..

编辑2 错误是 NRE，因为设计尚未加载，我可以轻松解决此问题，但宁愿让线程工作。

Answer 1

有很多小错误。鉴于我们可能没有查看真实代码，并且我们没有带有调用堆栈窗口的调试器来查看它实际崩溃的位置，因此其中任何一个都可能是一个因素。

• 测试 bw.IsBusy 和 not 当它为真时启动工人是一个严重的错误。它永远不会忙于发布的代码，但如果它实际上可能是真的，那么你的代码中有一个令人讨厌的错误。因为您实际上确实订阅了忙碌工作人员的事件。现在 RunWorkerCompleted 事件处理程序将运行两次。

• 使用 Close() 方法关闭对话框是不正确的。应通过分配其 DialogResult 属性来关闭对话框。不是最严重的错误，但仍然是错误的。

• 代码中存在竞争，工作人员可以在对话框显示之前完成。对话框只能在创建其本机窗口时关闭。换句话说，IsHandleCreated 必须为真。您必须将其联锁以确保这永远不会发生。订阅对话框的 Load 事件以启动工作器。

• 您盲目地假设工人将完成工作并产生结果。当它的 DoWork 方法死于异常时，情况就不是这样了。它被 BackgroundWorker 捕获并作为 e.Error 属性传递给 RunWorkerCompleted 事件处理程序。您必须检查此属性，如果它不为 null，请执行合理的操作。

从 cmets 来看，我猜是后者的原因。您可以使用 Debug + Exceptions 进行调试，勾选 CLR 异常的 Throw 复选框。现在，当抛出异常时，调试器将停止，以便您找出问题所在。

Answer 2

在显示对话框之前，您的后台工作人员可能实际上并没有花费太多时间并完成。我建议将后台worker初始化和启动代码转移到PleaseWait的Form_Load或Form_Shown