Question

When I code, I often ask myself the same question :

Do I have to verify all arguments are not null ? So, in each method, I will have something like that :

if (arg1 == null) 
{
    throw FooException("...");
}

if (arg2 == null) 
{
    throw FooException("...");
}

If not, in which case is preferable ?

What’s the best practices ?

Answer 1

As always, it depends.

If you’re writing an API to be used by other teams / organizations, such defensive programming with precondition checks on public functions can really help your users; when using an external library, a meaningful error message like ‘argument passed to foo() should not be null’ is way better than NullPointerException thrown from some inner class.

Outside of API, though, I think such checks clutter the code too much. Thrown NullPointerExceptions are usually pretty easy to trace with debugger anyway. In languages that support them, you can consider using assertions – their syntax is usually less cumbersome, and you can turn them off on production so the checks won’t degrade performance.