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

标准是抛出NullPointerException。通常不会出错的“有效Java”在第42项(第一版)、第60项(第二版)或第72项(第三版)中简要讨论了这一点。“赞成使用标准异常”:

可以说，都是错误的方法 调用归结为非法 争论或非法国家，但其他 异常通常用于 一些不合法的论点 州。如果调用方传入null 某个参数，其值为空 是被禁止的吗 NullPointerException被抛出 而不是IllegalArgumentException。”

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

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

一些集合假设使用NullPointerException而不是IllegalArgumentException拒绝null。例如，如果你比较一个包含null的集合和一个拒绝null的集合，第一个集合将调用另一个集合的containsAll，并捕获它的NullPointerException——而不是IllegalArgumentException。(我正在查看AbstractSet.equals的实现。)

You could reasonably argue that using unchecked exceptions in this way is an antipattern, that comparing collections that contain null to collections that can't contain null is a likely bug that really should produce an exception, or that putting null in a collection at all is a bad idea. Nevertheless, unless you're willing to say that equals should throw an exception in such a case, you're stuck remembering that NullPointerException is required in certain circumstances but not in others. ("IAE before NPE except after 'c'...")

类似地，构建工具可以自动插入空检查。值得注意的是，Kotlin的编译器在向Java API传递可能为空的值时执行此操作。当检查失败时，结果是NullPointerException。因此，为了给所有的Kotlin用户和Java用户提供一致的行为，您需要使用NullPointerException。

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

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