Question

Today I encountered the following situation: (“pseudo code”)

class MyClass {

    public void workOnArray(Object[] data) {
        for (Object item : data) {
            workOnItem(item);
        }
    }

    public void workOnItem(Object item) {
        if (item == null) throw new NullPointerException();
    }
}

Now if the caller calls workOnArray with an array containing null items, the caller would get a NullPointerException because of workOnItem.

But I could insert an additional check in workOnArray, in other words, the problem can be detected sooner.

(Please note that this is a trivial example, in a real life application this can be much less obvious).

On the pro side I’d say an additional check could offer more high-level diagnostic information. Also failing early is always a good thing.

On the contra side I’d ask myself “If I do that, shouldn’t I also validate each and every parameter passed into the core API of my programming language and throw the exact same exceptions?”

Is there some rule of thumb when to throw exceptions early and when to just let the callee throw it?

Answer 1

In the case of a loop processing items like that, there’s one thing that would definitely make me want to pre-validate the whole array of items first; If it would be bad for some of the items to be processed before an exception was thrown, leaving any remaining items un-processed.

Barring some sort of transaction mechanism wrapping the code, I would usually want to have some assurance that the items in the collection were valid before beginning to process them.