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

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

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

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

我想从其他非法参数中挑出Null参数，所以我从IAE派生了一个名为NullArgumentException的异常。甚至不需要读取异常消息，我就知道一个空参数被传递到一个方法中，并且通过读取消息，我找到了哪个参数为空。我仍然用IAE处理程序捕获NullArgumentException，但在我的日志中，我可以快速看到差异。

我倾向于遵循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");  
    }  
}

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

一般来说，开发人员不应该抛出NullPointerException异常。当代码试图解引用值为null的变量时，运行时将引发此异常。因此，如果你的方法想要显式禁止null，而不是恰好有一个空值引发一个NullPointerException，你应该抛出一个IllegalArgumentException。

在这种情况下，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.