我应该抛出当前方法还是让后面的方法抛出它

Question

我在好设计方面遇到了困难。
我不确定是否应该让稍后的方法抛出异常，或者是否应该自己抛出异常。
事情是：

异常的类型会根据被调用的方法而变化。在后面的方法中，它可能是 InvalidOperation，但在当前方法中，ArgumentException 可能更适合。

现在我应该真正执行所有检查以抛出正确类型的异常，还是应该让更深层次的方法抛出不正确类型的异常？

考虑一下：

private bool _canWaste = false;



/// <summary>
///    Runs around and wastes something.
/// </summary>
/// <exception cref="InvalidOperationException">
///     Thrown when <paramref name="CanPower"/> is false.
/// </exception>
public void Run(bool CanWaste) {
   _canWaste = CanWaste;
   // <- Should I check if we can waste and throw an ArgumentException here or 
   //    throw the 'wrong' type later in Waste?
   Waste();
}



/// <summary>
///    Wastes something.
/// </summary>
/// <exception cref="InvalidOperationException">
///     Thrown when we cannot waste.
/// </exception>
private void Waste() {
   if(!_canWaste) throw new InvalidOperationException("We cannot waste.");
   /* ... */
}

这是一个很容易完成的简单示例（没有意义）。
但是，当然，有些方法的评估要困难得多。

另一种选择是捕获异常并抛出正确类型的新异常。
但我想这会影响性能并在输出中记录一些不酷的东西。

Answer 1

一般来说，你应该尽可能快地投掷。这样，您可以防止代码在可能导致重大问题的无效状态下执行（例如，由于过滤器不正确，您删除了所有记录而不是一条记录）。

另外，如果你快速抛出，你可以确保调用者得到最有意义的消息。

public void Run(bool CanWaste)
{
    if(!CanWaste)
        throw new ArgumentException("CanWaste must be true", "CanWaste");
    Waste();
    // ...
}

但是，如果您不知道参数是否无效并且确定是否昂贵（例如，您需要执行它），那么让更深层的方法抛出。 您可以在调用方法中处理InvalidOperationException：

public void Run(bool CanWaste)
{
    _canWaste = CanWaste;

    try
    {
        Waste();
    }catch(InvalidOperationException ioe)
    {
        throw new ArgumentException("CanWaste must be true", "CanWaste", ioe);
    }
}

当然，在这种情况下，很容易确定参数是否有效。这也不是最好的例子，因为只能是 true 的 bool 参数没有多大意义。

Answer 2

不要抛出或让代码抛出不正确的类型异常，这对任何人都没有帮助。如果你得到一个无效的参数抛出InvalidArgumentException，否则让异常从库中冒泡到调用者。

从描述中，听起来 您正试图通过异常来控制执行流程。不要那样做。例外就是“例外”，当出现无法避免的例外情况时使用它们。

除非您想自己重置堆栈跟踪，否则捕获并重新抛出该异常是没有意义的。