วิธีการสร้างใหม่ InnerException โดยไม่สูญเสียการติดตามสแต็คใน C # ได้อย่างไร

ฉันกำลังเรียกวิธีสะท้อนซึ่งอาจทำให้เกิดข้อยกเว้น ฉันจะส่งข้อยกเว้นไปยังผู้โทรโดยไม่มีการสะท้อนของผู้คลุมรอบได้อย่างไร
ฉันกำลัง rethrowing InnerException แต่สิ่งนี้จะทำลายการติดตามสแต็ก
รหัสตัวอย่าง:

public void test1()
{
    // Throw an exception for testing purposes
    throw new ArgumentException("test1");
}

void test2()
{
    try
    {
        MethodInfo mi = typeof(Program).GetMethod("test1");
        mi.Invoke(this, null);
    }
    catch (TargetInvocationException tiex)
    {
        // Throw the new exception
        throw tiex.InnerException;
    }
}

ใน. NET 4.5ขณะนี้มีExceptionDispatchInfoคลาส

สิ่งนี้ช่วยให้คุณสามารถดักจับข้อยกเว้นและทำการโยนใหม่โดยไม่เปลี่ยน stack-trace:

try
{
    task.Wait();
}
catch(AggregateException ex)
{
    ExceptionDispatchInfo.Capture(ex.InnerException).Throw();
}

นี้ทำงานบนข้อยกเว้นใด ๆ AggregateExceptionที่ไม่เพียง

มันได้รับการแนะนำเนื่องจากawaitคุณสมบัติภาษา C # ซึ่งไม่รวมข้อยกเว้นภายในจากAggregateExceptionอินสแตนซ์เพื่อให้คุณสมบัติทางภาษาแบบอะซิงโครนัสมากกว่าคุณสมบัติทางภาษาแบบซิงโครนัส

มันเป็นไปได้ที่จะรักษากองติดตามก่อน rethrowing โดยไม่ต้องสะท้อนให้เห็นถึง:

static void PreserveStackTrace (Exception e)
{
    var ctx = new StreamingContext  (StreamingContextStates.CrossAppDomain) ;
    var mgr = new ObjectManager     (null, ctx) ;
    var si  = new SerializationInfo (e.GetType (), new FormatterConverter ()) ;

    e.GetObjectData    (si, ctx)  ;
    mgr.RegisterObject (e, 1, si) ; // prepare for SetObjectData
    mgr.DoFixups       ()         ; // ObjectManager calls SetObjectData

    // voila, e is unmodified save for _remoteStackTraceString
}

สิ่งนี้ทำให้สิ้นเปลืองวงจรจำนวนมากเมื่อเทียบกับการโทรInternalPreserveStackTraceผ่านผู้แทนแคช แต่มีข้อได้เปรียบจากการพึ่งพาฟังก์ชั่นสาธารณะเท่านั้น ต่อไปนี้เป็นรูปแบบการใช้งานทั่วไปสองแบบสำหรับฟังก์ชั่นการปกป้องแบบกองซ้อน:

// usage (A): cross-thread invoke, messaging, custom task schedulers etc.
catch (Exception e)
{
    PreserveStackTrace (e) ;

    // store exception to be re-thrown later,
    // possibly in a different thread
    operationResult.Exception = e ;
}

// usage (B): after calling MethodInfo.Invoke() and the like
catch (TargetInvocationException tiex)
{
    PreserveStackTrace (tiex.InnerException) ;

    // unwrap TargetInvocationException, so that typed catch clauses 
    // in library/3rd-party code can work correctly;
    // new stack trace is appended to existing one
    throw tiex.InnerException ;
}

ฉันคิดว่าทางออกที่ดีที่สุดของคุณคือเพียงแค่ใส่สิ่งนี้ลงในบล็อก catch ของคุณ:

throw;

แล้วแยกความสว่างของแสงในภายหลัง

public static class ExceptionHelper
{
    private static Action<Exception> _preserveInternalException;

    static ExceptionHelper()
    {
        MethodInfo preserveStackTrace = typeof( Exception ).GetMethod( "InternalPreserveStackTrace", BindingFlags.Instance | BindingFlags.NonPublic );
        _preserveInternalException = (Action<Exception>)Delegate.CreateDelegate( typeof( Action<Exception> ), preserveStackTrace );            
    }

    public static void PreserveStackTrace( this Exception ex )
    {
        _preserveInternalException( ex );
    }
}

เรียกวิธีการส่วนขยายในข้อยกเว้นของคุณก่อนที่คุณจะโยนมันจะรักษาสแต็คร่องรอยเดิม

ไม่มีใครอธิบายความแตกต่างระหว่างExceptionDispatchInfo.Capture( ex ).Throw()และธรรมดาthrowดังนั้นนี่คือ

วิธีที่สมบูรณ์ในการสร้างข้อยกเว้นที่ถูกดักจับใหม่คือการใช้ExceptionDispatchInfo.Capture( ex ).Throw()(ใช้ได้เฉพาะจาก. Net 4.5)

ด้านล่างมีกรณีที่จำเป็นในการทดสอบนี้:

1

void CallingMethod()
{
    //try
    {
        throw new Exception( "TEST" );
    }
    //catch
    {
    //    throw;
    }
}

2

void CallingMethod()
{
    try
    {
        throw new Exception( "TEST" );
    }
    catch( Exception ex )
    {
        ExceptionDispatchInfo.Capture( ex ).Throw();
        throw; // So the compiler doesn't complain about methods which don't either return or throw.
    }
}

3

void CallingMethod()
{
    try
    {
        throw new Exception( "TEST" );
    }
    catch
    {
        throw;
    }
}

4

void CallingMethod()
{
    try
    {
        throw new Exception( "TEST" );
    }
    catch( Exception ex )
    {
        throw new Exception( "RETHROW", ex );
    }
}

กรณีที่ 1 และกรณีที่ 2 จะให้การติดตามสแต็กแก่คุณโดยที่หมายเลขบรรทัดซอร์สโค้ดสำหรับCallingMethodวิธีคือหมายเลขบรรทัดของthrow new Exception( "TEST" )บรรทัด

อย่างไรก็ตามกรณีที่ 3 จะให้การติดตามสแต็กแก่คุณโดยที่หมายเลขบรรทัดซอร์สโค้ดสำหรับCallingMethodวิธีคือหมายเลขบรรทัดของการthrowโทร ซึ่งหมายความว่าหากthrow new Exception( "TEST" )บรรทัดถูกล้อมรอบด้วยการดำเนินการอื่นคุณไม่มีความคิดว่าหมายเลขบรรทัดใดที่มีข้อยกเว้นเกิดขึ้นจริง

กรณีที่ 4 คล้ายกับ case 2 เนื่องจากหมายเลขบรรทัดของข้อยกเว้นดั้งเดิมถูกเก็บรักษาไว้ แต่ไม่ใช่การสร้างใหม่จริงเนื่องจากจะเปลี่ยนชนิดของข้อยกเว้นดั้งเดิม

สะท้อนเพิ่มเติม ...

catch (TargetInvocationException tiex)
{
    // Get the _remoteStackTraceString of the Exception class
    FieldInfo remoteStackTraceString = typeof(Exception)
        .GetField("_remoteStackTraceString",
            BindingFlags.Instance | BindingFlags.NonPublic); // MS.Net

    if (remoteStackTraceString == null)
        remoteStackTraceString = typeof(Exception)
        .GetField("remote_stack_trace",
            BindingFlags.Instance | BindingFlags.NonPublic); // Mono

    // Set the InnerException._remoteStackTraceString
    // to the current InnerException.StackTrace
    remoteStackTraceString.SetValue(tiex.InnerException,
        tiex.InnerException.StackTrace + Environment.NewLine);

    // Throw the new exception
    throw tiex.InnerException;
}

โปรดทราบว่าการดำเนินการนี้อาจแตกได้ตลอดเวลาเนื่องจากเขตข้อมูลส่วนตัวไม่ได้เป็นส่วนหนึ่งของ API ดูการอภิปรายเพิ่มเติมเกี่ยวกับBugzilla โมโน

ข้อแรก: อย่าเสีย TargetInvocationException - เป็นข้อมูลที่มีค่าเมื่อคุณต้องการแก้ไขข้อบกพร่อง
ข้อสอง: ตัด TIE เป็น InnerException ในประเภทยกเว้นของคุณและใส่คุณสมบัติ OriginalException ที่เชื่อมโยงกับสิ่งที่คุณต้องการ (และทำให้ callstack ทั้งหมดยังคงอยู่)
ประการที่สาม: ปล่อยให้ฟอง TIE ออกจากวิธีการของคุณ

พวกคุณเท่ห์ .. ฉันจะเป็นหมอผีในไม่ช้า

    public void test1()
    {
        // Throw an exception for testing purposes
        throw new ArgumentException("test1");
    }

    void test2()
    {
            MethodInfo mi = typeof(Program).GetMethod("test1");
            ((Action)Delegate.CreateDelegate(typeof(Action), mi))();

    }

ตัวอย่างโค้ดอีกอันที่ใช้การยกเว้นอนุกรม / ดีซีเรียลไลเซชัน มันไม่จำเป็นต้องมีชนิดของข้อยกเว้นตามจริงที่จะต่อเนื่องกันได้ นอกจากนี้ยังใช้วิธีสาธารณะ / การป้องกันเท่านั้น

    static void PreserveStackTrace(Exception e)
    {
        var ctx = new StreamingContext(StreamingContextStates.CrossAppDomain);
        var si = new SerializationInfo(typeof(Exception), new FormatterConverter());
        var ctor = typeof(Exception).GetConstructor(BindingFlags.NonPublic | BindingFlags.Instance, null, new Type[] { typeof(SerializationInfo), typeof(StreamingContext) }, null);

        e.GetObjectData(si, ctx);
        ctor.Invoke(e, new object[] { si, ctx });
    }

จากคำตอบของพอลเทอร์เนอร์ฉันทำวิธีการขยาย

    public static Exception Capture(this Exception ex)
    {
        ExceptionDispatchInfo.Capture(ex).Throw();
        return ex;
    }

return exist ไม่ถึง แต่ข้อดีก็คือว่าผมสามารถใช้throw ex.Capture()เป็นหนึ่งซับเพื่อให้คอมไพเลอร์จะไม่ยกnot all code paths return a valueข้อผิดพลาด

    public static object InvokeEx(this MethodInfo method, object obj, object[] parameters)
    {
        {
            return method.Invoke(obj, parameters);
        }
        catch (TargetInvocationException ex) when (ex.InnerException != null)
        {
            throw ex.InnerException.Capture();
        }
    }