您应该抛出一个IllegalArgumentException，因为这将使程序员清楚地知道他做了一些无效的事情。开发人员太习惯于看到VM抛出的NPE，以至于任何程序员都不会立即意识到自己的错误，并开始随机查看，或者更糟糕的是，指责你的代码“有bug”。

如果它是一个setter方法，并且null被传递给它，我认为抛出一个IllegalArgumentException会更有意义。NullPointerException似乎在尝试实际使用null的情况下更有意义。

如果你在使用它，它是空的，NullPointer。如果它被传入并且它为空，则为非法参数。

在这种情况下，IllegalArgumentException使用API向用户传达了“不应该为空”的明确信息。正如其他论坛用户指出的那样，只要你使用API向用户传达正确的信息，你就可以使用NPE。

GaryF和tweak放弃了“有效Java”(我发誓)的参考，建议使用NPE。看看其他好的API是如何构造的，这是了解如何构造你的API的最好方法。

Another good example is to look at the Spring APIs. For example, org.springframework.beans.BeanUtils.instantiateClass(Constructor ctor, Object[] args) has a Assert.notNull(ctor, "Constructor must not be null") line. org.springframework.util.Assert.notNull(Object object, String message) method checks to see if the argument (object) passed in is null and if it is it throws a new IllegalArgumentException(message) which is then caught in the org.springframework.beans.BeanUtils.instantiateClass(...) method.

这是一个“圣战”式的问题。换句话说，两种选择都是好的，但人们会有自己的偏好，他们会誓死捍卫这些偏好。

根据您的场景，IllegalArgumentException是最好的选择，因为null不是属性的有效值。

如果你不希望null是一个允许的值，似乎会调用IllegalArgumentException，如果你试图使用一个结果为null的变量，则会抛出NullPointerException。