>
>Use the Application.ThreadException() to set up a new handler. You can also use the Application.SetUnhandledExceptionMode() to set up a handler as well.
>>
>>Of course you should have try{}...catch{} around sections of code and not only rely on these global error handlers.>
>Hey Derek,
>
>Have you actually gotten this to work correctly? I looked up info on Application.ThreadException() in the docs and copied the example into my test app. I have a button that throws an Exception, without a try/catch ... I end up getting two errors dialogs getting displayed.
>
>One from the actual thrown exception, the dialog that .NET would normally show when an Exception is detected and there is no try/catch. And then the Dialog that I've coded in the Exception Handler. This is not exactly what I expected to happen. Have you discovered something different?
>
>Here's what I have:
>
>
>[STAThread]
>static void Main()
>{
> // Creates an instance of the methods that will handle the exception.
> CustomExceptionHandler eh = new CustomExceptionHandler();
>
> // Adds the event handler to to the event.
> Application.ThreadException += new ThreadExceptionEventHandler(eh.OnThreadException);
>
> Application.Run(new Form1());
>}
>
>// The following was copied directly from the docs
>#region Exception Handling
>// Creates a class to handle the exception event.
>internal class CustomExceptionHandler
>{
>
> // Handles the exception event.
> public void OnThreadException(object sender, ThreadExceptionEventArgs t)
> {
> DialogResult result = DialogResult.Cancel;
> try
> {
> result = this.ShowThreadExceptionDialog(t.Exception);
> }
> catch
> {
> try
> {
> MessageBox.Show("Fatal Error", "Fatal Error", MessageBoxButtons.AbortRetryIgnore, MessageBoxIcon.Stop);
> }
> finally
> {
> Application.Exit();
> }
> }
>
> // Exits the program when the user clicks Abort.
> if (result == DialogResult.Abort)
> Application.Exit();
> }
>
> // Creates the error message and displays it.
> private DialogResult ShowThreadExceptionDialog(Exception e)
> {
> string errorMsg = "An error occurred please contact the adminstrator with the following information:\n\n";
> errorMsg = errorMsg + e.Message + "\n\nStack Trace:\n" + e.StackTrace;
> return MessageBox.Show(errorMsg, "Application Error", MessageBoxButtons.AbortRetryIgnore, MessageBoxIcon.Stop);
> }
>}
>#endregion
>
>// And then the button click that simply throws an exception:
> throw new Exception("Test Exception Handling");
>>
>I've also tried it with the try/catch in the Main, but it still does the same thing:
>
>
>[STAThread]
>static void Main()
>{
> try
> {
> // Creates an instance of the methods that will handle the exception.
> CustomExceptionHandler eh = new CustomExceptionHandler();
>
> // Adds the event handler to to the event.
> Application.ThreadException += new ThreadExceptionEventHandler(eh.OnThreadException);
>
> Application.Run(new Form1());
> }
> catch (Exception ex)
> {
> MessageBox.Show(ex.Message);
> }
>}
>>
>Did I miss something? Like I said, this is exactly how the docs say to do it. I'm assuming you played around with it a bit more and got it to work properly.
Hi,
Were you testing this in Debug mode ? If so test without....
