使用MessageBox在多线程应用程序中显示异常信息

Question

嗨，我正在使用winform并尝试使用MessageBox进行异常处理。 这里奇怪的是，MessageBox只在主窗体（下面代码中的“Form1”）关闭后出现。

public class Worker {
    /* edited (see below)
    public void doWork() {
        try {
            // do something
            client.Connect(serverAddress);
            stream = client.GetStream();
        }
        catch(Exception e) {
            MessageBox.Show(e.ToString(), 
                "This will not show up until Form1 is closed");
        }
    }
    */
}

public class Form1 {
    /* edited (see below)
     * public void threadProc() {
     *    Worker worker = new Worker();
     *    worker.doWork();
     * }
     */
     void button_Click(object sender, EventArgs e) {
        // create a thread that will end up throwing an exception
        Thread thread = new Thread(threadProc);
        thread.Start();
     }
}

什么是使用MessageBox进行异常处理的更好方法？

---

...所以我在UI线程中为MessageBox-ing添加了一些代码，但问题仍然存在。

public class WorkExceptionArgs : EventArgs {
    public Exception e;
    public WorkExceptionArgs (Exception e) { this.e = e; }
}
public partial class Worker1 { // renamed (Worker->Worker1)
    /* (edited) Now Worker1 doesn't trigger any event (see below)
       public event EventHandler<WorkExceptionArgs> workException;
    */
    public void doWork() {
        try {
            // do something
            client.Connect(serverAddress);
            stream = client.GetStream();
        }
        catch(Exception e) {
            /* (edited) suppose Worker1 never throws any exception (see below)
             *  // trigger event that will cause MessageBox-ing by UI thread
             *  workException(this, new WorkExceptionArgs(e));
             */
        }
    }
}
public partial class Form1 {
    public void threadProc() {
       Worker1 worker1 = new Worker();
      /* (edited) Now Worker1 never throws any exception
       * worker.workException += new EventHandler<WorkException>(worker_WorkException);
       */
       worker1.doWork();
       // (added) After doWork() is done, Form1 creates Worker2
       Worker2 w2 = new Worker2(this, this.form2);
       w2.workException += new EventHandlerArgs<WorkExceptionArgs>(form2.worker2_WorkException);
       w2.doSomeOtherWork();
    }
    /* public void worker_WorkException(object sender, WorkExceptionArgs eArg) {
     *   MessageBox.Show(eArg.e.ToString(), "Still not showing");
     * } */
    Form2 form2 = new Form2(); // (added) At first form2 is hidden (see below)
}

---

实际上还有另一种形式和另一种工人。 一旦Worker（Worker1）连接到服务器，Form1隐藏（.Hide（）），Form2显示（.Show（）），并且Worker2开始使用连接Worker1。

public class Worker2 {
    Worker2(Worker1 w1, Form2 frm2) { this.w1=w1; this.frm2=frm2; }
    public Worker1 w1;
    public Form2 frm2;
    public event EventHandler<WorkExceptionArgs> workException;
    public void doSomeOtherWork() { // do some other, using data in Worker 1.
        try { // This will throw an exception
            BinaryFormatter formatter = new BinaryFormatter();
            MyObj mo = (MyObj)formatter.Deserialize(w1.getStream());
        }
        catch(Exception e) {
            workException(this, new WorkExceptionArgs(e));
        }
    }             
}

public class Form2 {
    public Form2(Form1 frm1) { // to switch from frm1 to frm2
        InitializeComponent();
        this.frm1 = frm1;
    }
    public Frm1 frm1 {get;set;}
    public void worker2_WorkException(object sender, WorkExceptionArgs ea) {
       MessageBox.Show(this, ea.e.ToString(), "SHOWS ONLY IF FORM2 IS CLOSED");
    }

}     

public partial class Form1 {
    delegate void switchWindow_Callback();
    public void switchWindow() { this.Hide(); form2.Show(); }
    public void switchWindowCb(object sender, EventArgs e) {
        if(this.InvokeRequired) {
            SwitchWindow_Callback hcb = new SwitchWindow_Callback(switchWindow);
            this.Invoke(hcb, new object[] {});
        }
        else { this.switchWindow(); }
    }
}

Answer 1

实际上我敢打赌MessageBox出现在主窗体后面 ，你只是在关闭它之前看不到它。

让UI线程（创建并拥有Form1线程）执行MessageBox更好。 您要么在事务中创建事件，要么在工作者类中有错误回调委托。

但是， BackgroundWorker可能值得在这里查看，而不是试图自己滚动。 假设它是一个致命的异常，您可以保存并检索错误状态，并在线程完成时自动调用事件。

Answer 2

您应该真正锁定doWork方法，以便多个线程无法同时访问它，它们需要排队。

查看“加入主题”。 想象一下，如果你同时得到两个例外。 你的应用程序会崩溃。 锁定将重复复制的代码区域将形成一个队列，供您的线程访问处理异常的代码区域。

Answer 3

如lc所述，您的消息框很可能出现在主窗体后面，因此您只能在主窗体关闭时看到它。

我用来处理Windows窗体应用程序中未处理的异常的模型如下所示：

// Switch-off the Windows Forms default handler for unhandled exceptions.
// NB From .NET 4 upwards, this won't work if the process state is corrupted.
Application.SetUnhandledExceptionMode(UnhandledExceptionMode.CatchException);

// Setup event handler to intercept an unhandled exception on a UI thread .
// NB The exception will still terminate the application. 
// But you can show a MessageBox in the event handler and log the exception. 
Application.ThreadException += 
    new ThreadExceptionEventHandler(App_UiThreadException);

// Setup event handler to intercept an unhandled exception on a non-UI thread.
AppDomain.CurrentDomain.UnhandledException += new 
    UnhandledExceptionEventHandler(App_NonUiThreadException);

// Run the application (open main form etc).

第一行表示你想捕获任何未处理的异常并自己处理它，而不是让WinForms基础设施处理它。 请注意，从.NET 4开始，此设置不适用于破坏进程状态的异常（例如OutOfMemory）。

如果UI线程上有未处理的异常，则第二行将触发您创建的名为* App_UiThreadException *的过程。 这就是你的MesssageBox代码应该去的地方。

如果在非UI线程上有未处理的异常，则最后一行将触发您创建的名为* App_NonUiThreadException *的过程。 这就是你的MesssageBox代码应该去的地方。