何时为构造函数抛出异常

Question

public Neocortex(Region rootRegion, ConnectionInterface functor) {
this.rootRegion = rootRegion;
this.currentRegion = this.rootRegion;
this.functor = functor;
}

嘿上面我有一个类的构造函数。 我的问题是我应该在构造函数中添加空指针异常还是不必要？ 老实说，我只是不明白何时应该为我的代码添加例外。 但在这种情况下，我应该使用哪个构造函数？

public Neocortex(Region rootRegion, ConnectionInterface functor) {
    if (rootRegion == null) {
    throw new NullPointerException("rootRegion cannot be null");
} else if (functor == null) {
        throw new NullPointerException("functor cannot be null");
    }
this.rootRegion = rootRegion;
this.currentRegion = this.rootRegion;
this.functor = functor;
}

Answer 1

我会为构造函数的那些参数抛出一个IllegalArgumentException ，如果被接受，会导致一些不好的东西（包括但不限于null参数）。 也就是说，它将导致完成一个没有意义的物体的构造。

不要抛出NullPointerException ，因为您还没有尝试在null上调用某个方法。

来自IllegalArgumentException javadocs ：

抛出以指示方法已被传递非法或不适当的参数。

如果null不合适，则抛出IllegalArgumentException 。

Answer 2

嗯......这是一个品味问题。

如果类的前提条件是必须提供rootRegion，那么保护类实现不需要在整个地方进行空检查是有意义的。

因此，回答“何时应该在构造函数中抛出异常”的问题：我会在所有情况下执行此操作，其中来自使用者的参数使您的实现处于无效状态，委托问题（即抛出异常）。

如果你试图将角色作为消费者一段时间，并且你选择不进行空检查，他将拥有如下代码：

Neocortex n = new Neocortex(null,null);
n.doSomeething();

如果他到达第二行，并且这里的实现抛出NullPointerException，那么他将不清楚它是由于他提供的参数。

Answer 3

是的，如果null不是rootRegion等可接受的值， rootRegion构造函数中抛出NullPointerException 。 从NullPointerException文档中 ，它讨论了JVM何时抛出它，但也说：

应用程序应抛出此类的实例以指示null对象的其他非法使用。

您可以将IllegalArgumentException用于对构造函数无效的更一般的参数情况，但NullPointerException完全适用于参数为null且不应该为null的情况。 但是， IllegalArgumentException确实具有优势，因为它非常清楚错误的原因。 :-)

Answer 4

在这个特定的情况下，我会抛出一个IllegalArgumentException ，这意味着你的类的用户比NullPointerException 。

构造函数抛出异常并不是一种坏习惯。 但是，您应该确保它们有良好的文档记录（特别是在运行时异常的情况下），并且对调用者有意义。