有任何现实世界的理由使用 throw ex 吗？

Question

在 C# 中，throw ex 几乎总是错误的，因为它会重置堆栈跟踪。

我只是想知道，这在现实世界中有什么用途吗？我能想到的唯一原因是隐藏封闭库的内部结构，但这是一个非常薄弱的​​理由。除此之外，我从未在现实世界中遇到过。

编辑：我的意思是throw ex，就像抛出与捕获的完全相同的异常，但堆栈跟踪为空，就像做错了一样。我知道 throw ex 必须作为一种语言结构存在以允许抛出不同的异常 (throw new DifferentException("ex as innerException", ex))，我只是想知道是否存在 throw ex 没有错的情况。

Answer 1

我从未见过有人故意使用它，但这当然没有任何意义。

让我们看看替代方案：

catch(Exception ex)
{
    // do stuff (logging)
    throw;                                // a) continue ex
    throw new SomeException("blah", ex);  // b) wrap
    throw new SomeException("blah");      // c) replace
    throw ex;                             // d) reset stack-trace
}

c) 和 d) 都在删除堆栈跟踪，我可以看到的 d) 的唯一“优势”是 throw ex; 保留了 ex 的确切类型和可能的额外属性（SQL 错误）。

那么有没有一种情况是您想要保留除堆栈跟踪之外的所有异常信息？不是通常，而是一些猜测：

• 位于混淆库的边界。但是 c) 在这里看起来是一个更好的选择。
• 在调用站点对某些生成的代码，例如在 RegEx 类中。

Answer 2

如果你停下来想一想，throw ex 实际上经常被使用——除了一个未命名的局部变量。 （重新）设计语言是相当困难的：

throw new ArgumentException();

是有效的，但是

var ex = new ArgumentException();
throw ex;

无效（然后扩展它以跟踪更复杂的变量分配）。 IE。 throw ex 是 throw 语句的“正常”形式，但在重新抛出已经捕获的异常时通常是错误的。

Answer 3

没有正当理由重新抛出异常对象，即“throw ex”。

这是一个可能的语言\编译器功能，尽管在实践中并没有增加它所具有的负面影响的任何价值。这与能够编写代码来访问空引用上的成员的情况类似——这当然是可能的，但没有价值，例如：

    //Possible but not useful.
    object ref1 = null;

    ref1.ToString();

能够重新抛出异常对象是不幸的，并且经常被新手和经验丰富的编码人员误解，直到您在生产中遇到未处理的异常并记录截断的堆栈跟踪。

有一种微妙的感觉，它可以用来故意隐藏堆栈跟踪信息，但抛出一个新的异常是正确的做法。

我想说的是，能够重新抛出异常对象（即“throw ex”）是一个设计缺陷——做错事太容易了。但是，我怀疑这是出于性能原因在编译器中进行的设计权衡，因为它会产生开销来验证“throw ex”不是“re-throw”。我敢肯定有很多这样的取舍。

Answer 4

throw ex 只是不知道或忘记throw; 的人所犯的错误或错字。

它可能仅用于一种情况，其他人在他们的回答中指出：您不希望将正确的堆栈跟踪发送给调用者的情况，但您仍然希望保留之前抛出的异常。在实践中，我很难想象有人会需要它，并且不会使用throw new SomeCustomException(...)。

FxCop 规则CA2200RethrowToPreserveStackDetails 也是如此，通过使用throw; 正确邀请您到rethrow an exception：

try
{
    // ...
}
catch (SomeException ex)
{
    Logger.AddException(ex);
    throw;
}

Answer 5

以下是重新抛出现有异常对象的两个实际应用：

反射

通过反射调用代码时，任何抛出的异常都会被包裹在TargetInvocationException中。但是您可能希望/需要处理超出反射“障碍”的异常。在这种情况下，包装将使异常处理代码复杂化。相反，您可以解开原始异常并重新抛出。

如果您使用的是 .NET 4.5+，则可以使用 ExceptionDispatchInfo 保留原始堆栈跟踪：

MethodInfo someMethod = ...
try
{
    someMethod.Invoke();
}
catch (TargetInvocationException e)
{
    ExceptionDispatchInfo.Capture(e.InnerException).Throw();
}

任务 API

在 .NET 任务 API 中，任务执行期间引发的异常被包装在 AggregateException 中。使用 async/await 时，框架将在幕后执行展开，以便抛出原始异常而不是包装器 AggregateException。

除了 BCL 代码在自动解包时需要重新抛出现有异常之外，在某些情况下您无法使用 async/await，因此需要手动解包并重新抛出。有关这方面的实际示例，请查看 Stephen Clearys 出色的 AsyncEx 项目中 AsyncContext 实现中 TaskExtensions.WaitAndUnwrapException 的内部使用。

Answer 6

您可能想要记录异常，记录它，然后将其作为“throw ex”扔到上层。您可能还想在此层中进行日志记录，但这可能与您遇到异常的层无关，或者您可能不需要异常的详细信息，因为您已经记录了它。

假设你有：

Public int GetAllCustomers()
{
     try
     {
          return data.GetAllCustomers()
     }
     catch()
     {
         Logger.log("Error calling GetAllCustomers");
     }
}
     //data
Public static int GetAllCustomers()
{
     try
     {
          //db operations...
     }
     catch(Exception ex)
     {
          Logger.log(ex.Stacktrace);
          throw ex;
     }
}