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

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

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

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

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

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

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

你应该使用IllegalArgumentException (IAE)，而不是NullPointerException (NPE)，原因如下:

首先，NPE JavaDoc显式地列出了适用于NPE的情况。请注意，当不恰当地使用null时，所有这些都是由运行时抛出的。相比之下，IAE JavaDoc再清楚不过了:“抛出是为了表明一个方法被传递了一个非法或不适当的参数。”没错，就是你!

其次，当您在堆栈跟踪中看到NPE时，您会假设什么?可能有人解引用了一个null。当您看到IAE时，您假定堆栈顶部的方法调用方传递了一个非法值。同样，后一种假设是正确的，前一种假设具有误导性。

第三，由于IAE显然是为验证参数而设计的，因此必须假设它是默认的异常选择，那么为什么要选择NPE呢?当然不是针对不同的行为——你真的期望调用代码分别捕获NPE和IAE，并因此做一些不同的事情吗?您是否试图传达更具体的错误消息?但是无论如何，您都可以在异常消息文本中这样做，就像处理所有其他不正确的参数一样。

第四，其他所有不正确的参数数据都会被IAE，为什么不一致呢?为什么非法null是如此特殊，以至于它应该从所有其他类型的非法参数中获得一个单独的异常?

最后，我接受其他答案给出的论点，即Java API的某些部分以这种方式使用NPE。然而，从异常类型到命名约定，Java API与所有内容都不一致，因此我认为仅仅盲目地复制(您最喜欢的部分)Java API并不是一个足以胜过这些其他考虑因素的好理由。

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