Bewährte Verfahren für das Abfangen und erneute Auslösen von .NET-Ausnahmen

Question

Was sind die besten Praktiken, die beim Abfangen von Ausnahmen und dem erneuten Auswerfen von Ausnahmen zu berücksichtigen sind? Ich möchte sicherstellen, dass die Exception das Objekt InnerException und die Stapelverfolgung werden beibehalten. Gibt es einen Unterschied zwischen den folgenden Codeblöcken in der Art und Weise, wie sie dies handhaben?

try
{
    //some code
}
catch (Exception ex)
{
    throw ex;
}

Vs:

try
{
    //some code
}
catch
{
    throw;
}

Answer 1

Als Faustregel gilt, dass man das Fangen und Werfen der grundlegenden Exception Objekt. Dies zwingt Sie dazu, mit Ausnahmen etwas klüger umzugehen; mit anderen Worten, Sie sollten einen expliziten Catch für ein SqlException damit Ihr Bearbeitungscode nicht etwas falsch macht mit einer NullReferenceException .

In der realen Welt ist das Fangen von und Protokollierung die Basisausnahme ist auch eine gute Übung, aber vergessen Sie nicht, die ganze Sache zu Fuß zu erledigen, um alle InnerExceptions könnte es haben.

Answer 2

Sie sollten immer "throw;" verwenden, um die Ausnahmen in .NET erneut zu werfen,

Siehe dies, http://weblogs.asp.net/bhouse/archive/2004/11/30/272297.aspx

MSIL (CIL) hat im Wesentlichen zwei Anweisungen - "throw" und "rethrow":

• C#s "throw ex;" wird in MSILs "throw" kompiliert
• C#s "throw;" - in MSIL "rethrow"!

Grundsätzlich kann ich den Grund sehen, warum "throw ex" den Stack-Trace außer Kraft setzt.

Answer 3

Ein paar Leute haben einen sehr wichtigen Punkt übersehen - "throw" und "throw ex" tun zwar dasselbe, aber sie geben Ihnen keine entscheidenden Informationen, nämlich die Zeile, in der die Ausnahme passiert ist.

Betrachten Sie den folgenden Code:

static void Main(string[] args)
{
    try
    {
        TestMe();
    }
    catch (Exception ex)
    {
        string ss = ex.ToString();
    }
}

static void TestMe()
{
    try
    {
        //here's some code that will generate an exception - line #17
    }
    catch (Exception ex)
    {
        //throw new ApplicationException(ex.ToString());
        throw ex; // line# 22
    }
}

Wenn Sie entweder ein 'throw' oder 'throw ex' tun, erhalten Sie den Stack-Trace, aber die Zeilennummer wird #22 sein, so dass Sie nicht herausfinden können, welche Zeile genau die Ausnahme ausgelöst hat (es sei denn, Sie haben nur 1 oder wenige Codezeilen im Try-Block). Um die erwartete Zeile #17 in Ihrer Ausnahme zu erhalten, müssen Sie eine neue Ausnahme mit der ursprünglichen Ausnahme Stack Trace werfen.

Answer 4

Ich würde es auf jeden Fall verwenden:

try
{
    //some code
}
catch
{
    //you should totally do something here, but feel free to rethrow
    //if you need to send the exception up the stack.
    throw;
}

So bleibt Ihr Stapel erhalten.

Answer 5

Sie können auch verwenden:

try
{
// Dangerous code
}
finally
{
// clean up, or do nothing
}

Alle ausgelösten Ausnahmen werden an die nächste Ebene weitergegeben, die sie bearbeitet.