在我的代码中,我遇到了抛出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
}
}
我知道抛出异常通常很慢,所以我觉得第一种方法可能会更快。或者,有没有更好的方法来做到这一点,我没有想到?
答案 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;
}
}