检查内部异常的类型

时间:2012-04-12 09:04:52

标签: c#

在我的代码中,我遇到了抛出System.Reflection.TargetInvocationException的情况。在一个特定的情况下,我知道我想如何处理根异常,但我想抛出所有其他异常。我可以想到两种方法,但我不确定哪种方式更好。

1

try
{
    //code
}
catch (System.Reflection.TargetInvocationException ex)
{
    if (typeof(ex.InnerException) == typeof(SpecificException))
    {
        //fix
    }
    else
    {
        throw ex.Innerexception;
    }
}

2

try
{
    //code
}
catch (System.Reflection.TargetInvocationException ex)
{
    try
    {
        throw ex.InnerException;
    }
    catch (SpecificException exSpecific)
    {
        //fix
    }
}

我知道抛出异常通常很慢,所以我觉得第一种方法可能会更快。或者,有没有更好的方法来做到这一点,我没有想到?

3 个答案:

答案 0 :(得分:24)

您提出的每个解决方案都有自己的问题。

第一种方法检查内部异常的类型是否正好您期望的类型。这意味着派生类型将不匹配,这可能不是您想要的。

第二种方法用当前的堆栈位置覆盖内部异常的堆栈跟踪,正如Dan Puzey所提到的那样。销毁堆栈跟踪可能会破坏您需要的一条线索以修复错误。

解决方案基本上是DarkGray发布的内容,有Nick的建议,还有我自己的建议(在else中):

try 
{ 
    // Do something
} 
catch (TargetInvocationException ex) 
{ 
    if (ex.InnerException is SpecificException) 
    { 
        // Handle SpecificException
    }
    else if (ex.InnerException is SomeOtherSpecificException)
    {
        // Handle SomeOtherSpecificException
    }
    else 
    { 
        throw;    // Always rethrow exceptions you don't know how to handle.
    } 
}

如果你想重新抛出一个你无法处理的异常,请不要throw ex;,因为这会覆盖堆栈跟踪。而是使用保留堆栈跟踪的throw;。它基本上意味着“我实际上不想输入这个catch条款,假装我从未捕获过异常”。

更新:C#6.0通过异常过滤器提供更好的语法:

try
{
    // Do something
}
catch (TargetInvocationException ex) when (ex.InnerException is SpecificException)
{
    // Handle SpecificException
}
catch (TargetInvocationException ex) when (ex.InnerException is SomeOtherSpecificException)
{
    // Handle SomeOtherSpecificException
}

答案 1 :(得分:1)

你的#2绝对是一个有趣的解决方案!

你确实要小心:TargetInvocationException通常在第一次抓住 InnerException时被另一个组件抛出。如果你throw ex.InnerException你要销毁它包含的一些信息(比如堆栈跟踪),因为你是从另一个地方重新抛出它。

所以在你提出的两个中,我肯定建议选择#1。我不知道你所拥有的结构中有另一种选择。但是,InnerException最初会被抛到其他地方 - 值得研究是否有一个更优雅的地方来处理这个失败,更接近抛出异常的地方。

答案 2 :(得分:-2)

try 
{ 
    //code 
} 
catch (System.Reflection.TargetInvocationException ex) 
{ 
    if (ex.InnerException is SpecificException) 
    { 
        //fix 
    } 
    else 
    { 
        throw ex.InnerException; 
    } 
} 

try 
{ 
    //code 
} 
catch (System.Reflection.TargetInvocationException ex) 
{ 
    SpecificException spExc = ex.InnerException as SpecificException;
    if (spExc != null) 
    { 
        bla-bla spExc 
    } 
    else 
    { 
        throw ex.InnerException; 
    } 
} 

try 
{ 
    //code 
} 
catch (System.Reflection.TargetInvocationException ex) 
{ 
    if (ex.InnerException.GetType() == typeof(SpecificException)) 
    { 
        //fix 
    } 
    else 
    { 
        throw ex.InnerException; 
    } 
}