Control.Invoke rozpakowuje wyjątek zewnętrzny i propaguje wewnętrzny wyjątek zamiast:

Question

W poniższym wywołaniu MessageBox.Show pojawia się komunikat "Wewnętrzny". Czy to błąd?

private void Throw() 
{ 
    Invoke(new Action(() => 
    { 
     throw new Exception("Outer", new Exception("Inner")); 
    })); 
} 

private void button1_Click(object sender, EventArgs e) 
{ 
    try 
    { 
     Throw(); 
    } 
    catch (Exception ex) 
    { 
     MessageBox.Show(ex.Message); // Shows "Inner" 
    } 
} 

Answer 1

musiałem spojrzeć na źródło odniesienia dla System.Windows.Forms.Control, a kod, który zajmuje się Invoke wygląda następująco:

try { 
    InvokeMarshaledCallback(current); 
} 
catch (Exception t) { 
    current.exception = t.GetBaseException(); 
} 

GetBaseException:

public virtual Exception GetBaseException() 
{ 
    Exception inner = InnerException; 
    Exception back = this; 

    while (inner != null) { 
     back = inner; 
     inner = inner.InnerException; 
    } 

    return back; 
} 

więc widocznie to tak przez projekt. Komentarze w źródle nie zawierają żadnego wyjaśnienia, dlaczego tak robią.

EDIT: Niektóre witryny, która już nie ma roszczeń ten komentarz pochodzi od faceta w firmie Microsoft:

Na podstawie winform comfirmation w rekordzie, nasza analiza jest poprawne od przyczynę i to zachowanie jest zamierzony. Powodem było to, że użytkownik nie widział zbyt wiele mechanizmów wewnętrznych Windows.Forms. Dzieje się tak, ponieważ domyślne okno dialogowe błędu winformu również wykorzystuje Application.ThreadException, aby wyświetlić szczegóły wyjątku. .Net Winform zespół przycina inne informacje o wyjątkach, dzięki czemu domyślny błąd w oknie dialogowym nie wyświetli wszystkich szczegółów dla użytkownika końcowego.

Również niektóre MSFT sugerowały zmianę tego zachowania. Jednak .Net Zespół Winform uważa, że ​​zmiana wyjątku na throw jest przełamującą zmianą iz tego powodu WinForms będzie nadal wysyłać najbardziej wewnętrzny wyjątek do obsługi aplikacji Application.ThreadException.