3

可能重复：
何时选择已检查和未检查的异常

你好！

因此，我仍然对何时抛出已检查或未检查的异常感到满意。我想知道在这种情况下其他人认为最合适的是：

class Correlation<T>
{
    private final T object1, object2;
    private final double correlationCoefficient;

    public Correlation(T object1, T object2, double correlationCoefficient)
    {
        if(Math.abs(correlationCoefficient) > 1.0 || (object1.equals(object2) && correlationCoefficient != 1.0))
            throw new IllegalArgumentException();

        this.object1 = object1;
        this.object2 = object2;
        this.correlationCoefficient = correlationCoefficient;
    }
}

因此，在这种情况下，我想抛出一个运行时异常，因为我无法轻易地从用户传入错误数据的情况中恢复过来。我想事先指出，我无法控制传入的数据。如果可以的话，我会创建一个接口来保证构造函数中的条件为真。但是，对于已经计算过的相关性来说，这是一个便利类，所以我必须相信用户提供了准确的信息。

好吧，让我知道你们的想法！

4

3 回答 3

6

我认为这是正确的回应。您正在有效地进行障碍断言，即障碍检查，如果它们是错误的，您将拒绝创建实体。我会用一个 java 文档记录你可以抛出一个 IllegalArgumentException，但除此之外，它看起来是正确的。

Joshua Block 有一些关于检查和未检查异常的重要信息。基本前提是，除非您绝对希望有人检查异常，否则您应该抛出未经检查的异常。以这种方式思考可能会使一些编码和返回值复杂化，但通常它会使代码更清晰、更高效。对特殊情况使用例外，事情对你来说会更好。

只是我的2美分。

编辑

为了清楚起见，这是您应该拥有的类似于 java doc 的内容：

/**
 * <Something describing constructor, and what it does, ending with a period.>
 *
 * @param parameter <Describe the parameter - do one for each parameter of the constructor,
 *     and note which values may be illegal for that particular parameter.>
 * @throws IllegalArgumentException <the case for the illegal argument exception.>
于 2009-08-03T20:42:15.673 回答
4

在我看来，答案取决于：

  • 你期望调用者能够优雅地恢复吗？
  • 这个 API 是供公众使用还是内部使用？

有人会告诉你，你永远不应该使用受检异常。这纯粹是主观的。

于 2009-08-03T20:39:51.980 回答
2

您应该始终在您的例外中包含解释性文本。在这种特殊情况下，您甚至可以考虑进行两项检查：

    if(Math.abs(correlationCoefficient) > 1.0)
            throw new IllegalArgumentException("abs(correlationCoefficient) > 1.0 - " + correlationCoefficient);
    if((object1.equals(object2) && correlationCoefficient != 1.0))
            throw new IllegalArgumentException("object1==object2, but correlationCoefficient != 1.0, " + correlationCoefficient);

这允许那些真正看到堆栈跟踪的人能够识别确切的原因，而无需仔细查看代码。一个给定的异常应该只由一个条件触发，而不是几个，因为你不能确定发生了什么。还包括所有必要的信息，因为如果错误情况无法在测试场景中重现，这可能是至关重要的。

于 2009-08-03T21:11:14.880 回答