Re: [Avalon4:Fortress] Eating Crow (was Changes to RoleManager)

On Thursday, December 12, 2002, at 02:08  PM, Berin Loritsch wrote:
> 1) We should *never* explicitly throw a NullPointerException.
>    If an argument is required, throw an IllegalArgumentException
>    instead.  The meaning of the error is much more clear.

I'd actually prefer it the other way.. Why?

Throw a NullPointerException when an argument that is expected is null.

Use IllegalArgumentException when there is an argument supplied, but it 
is invalid for other reasons.

-pete