如果它是一个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.

作为一个主观问题，这应该是封闭的，但它仍然是开放的:

这是我以前工作的地方使用的内部政策的一部分，效果非常好。这些都是我的记忆，所以我不记得确切的措辞。值得注意的是，他们没有使用受控异常，但这超出了问题的范围。他们使用的未检查异常主要分为3类。

NullPointerException:不故意抛出。npe只有在解引用空引用时才会被VM抛出。要尽一切可能的努力确保这些错误永远不会被抛出。@Nullable和@NotNull应该与代码分析工具一起使用来发现这些错误。

IllegalArgumentException:当函数的参数不符合公共文档时抛出，这样就可以根据传入的参数识别和描述错误。OP的情况就属于这一类。

IllegalStateException:当调用函数时，其实参在传递时是意外的，或者与方法所属对象的状态不兼容时抛出。

例如，在有长度的事物中使用IndexOutOfBoundsException的两个内部版本。一个是IllegalStateException的子类，在索引大于长度时使用。另一个是IllegalArgumentException的子类，用于索引为负的情况。这是因为您可以向对象添加更多的项，并且参数将是有效的，而负数永远无效。

正如我所说，这个系统工作得非常好，有人解释了为什么会有这样的区别:“根据错误的类型，你很容易就能弄清楚该怎么做。即使您无法真正找出出错的地方，也可以找出在哪里捕获错误并创建额外的调试信息。”

NullPointerException:处理Null情况或放入断言，这样NPE就不会被抛出。如果你输入的断言只是另外两种类型中的一种。如果可能，继续调试，就像断言一开始就在那里一样。

IllegalArgumentException:你的调用站点有错误。如果传入的值来自另一个函数，请查明为什么接收到不正确的值。如果传入一个参数，则会在调用堆栈中进行错误检查，直到找到没有返回预期值的函数。

您没有按照正确的顺序调用函数。如果您正在使用其中一个参数，请检查它们并抛出IllegalArgumentException描述该问题。然后，您可以在堆栈上传播腮部，直到找到问题。

不管怎样，他的观点是你只能把IllegalArgumentAssertions复制到堆栈上。您无法将illegalstateexception或nullpointerexception传播到堆栈上，因为它们与您的函数有关。

公认的做法是使用IllegalArgumentException(字符串消息)来声明一个参数是无效的，并给出尽可能多的细节…所以说，一个参数被发现是空的，而异常是非空的，你会这样做:

if( variable == null )
    throw new IllegalArgumentException("The object 'variable' cannot be null");

你几乎没有理由隐式地使用“NullPointerException”。NullPointerException是当您试图在空引用(如toString())上执行代码时由Java虚拟机抛出的异常。

抛出一个排除空参数的异常(无论是NullPointerException还是自定义类型)使得自动化空测试更加可靠。这种自动化测试可以通过反射和一组默认值来完成，就像在Guava的NullPointerTester中一样。例如，NullPointerTester将尝试调用以下方法…

Foo(String string, List<?> list) {
  checkArgument(string.length() > 0);
  // missing null check for list!
  this.string = string;
  this.list = list;
}

...with two lists of arguments: "", null and null, ImmutableList.of(). It would test that each of these calls throws the expected NullPointerException. For this implementation, passing a null list does not produce NullPointerException. It does, however, happen to produce an IllegalArgumentException because NullPointerTester happens to use a default string of "". If NullPointerTester expects only NullPointerException for null values, it catches the bug. If it expects IllegalArgumentException, it misses it.

我完全同意你说的话。早失败，快失败。非常好的异常咒语。

抛出哪个Exception主要是个人喜好的问题。在我看来，IllegalArgumentException似乎比使用NPE更具体，因为它告诉我问题是我传递给方法的参数，而不是执行方法时可能生成的值。

我的2美分