如果您选择抛出NPE，并且在方法中使用参数，显式检查null可能是多余的和昂贵的。我想VM已经为你做了。

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

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

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

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

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

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

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

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

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

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

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

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

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

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

我倾向于遵循JDK库的设计，特别是集合和并发(Joshua Bloch, Doug Lea，这些人知道如何设计可靠的api)。不管怎样，JDK中的许多api都会主动抛出NullPointerException。

例如，Javadoc For Map。containsKey状态:

如果键为空，则@抛出NullPointerException 不允许空键(可选)。

举办自己的NPE是完全合理的。约定是在异常消息中包含为空的参数名。

模式是这样的:

public void someMethod(Object mustNotBeNull) {  
    if (mustNotBeNull == null) {  
        throw new NullPointerException("mustNotBeNull must not be null");  
    }  
}

无论您做什么，都不要允许设置一个错误的值，并在其他代码尝试使用它时抛出异常。这使得调试成为一场噩梦。你应该始终遵循“快速失败”的原则。