Do you check the input parameters of your methods for validity? I don't. I used to, but not anymore. I just let my methods crash with a null pointer and other exceptions when parameters are not valid. This may sound illogical, but only in the beginning. I'm suggesting you use validating decorators instead.
Let's take a look at this rather typical Java example:
Pretty defensive, right? If we remove these validations, the code will be much shorter, but it will crash with rather confusing messages if
NULL is provided by the client. Moreover, if the file already exists, our
Report will silently overwrite it. Pretty dangerous, right?
Yes, we must protect ourselves, and we must be defensive.
But not this way, not by bloating the class with validations that have nothing to do with its core functionality. Instead, we should use decorators to do the validation. Here is how. First, there must be an interface
Then, a class that implements the core functionality:
And, finally, a number of decorators that will protect us:
Now, the client has the flexibility of composing a complex object from decorators that perform their specific tasks. The core object will do the reporting, while the decorators will validate parameters:
What do we achieve with this approach? First and foremost: smaller objects. And smaller objects always mean higher maintainability. Our
DefaultReport class will always remain small, no matter how many validations we may invent in the future. The more things we need to validate, the more validating decorators we will create. All of them will be small and cohesive. And we'll be able to put them together in different variations.
Besides that, this approach makes our code much more reusable, as classes perform very few operations and don't defend themselves by default. While being defensive is an important feature, we'll use validating decorators. But this will not always be the case. Sometimes validation is just too expensive in terms of time and memory, and we may want to work directly with objects that don't defend themselves.
I also decided not to use the Java Validation API anymore for the same reason. Its annotations make classes much more verbose and less cohesive. I'm using validating decorators instead.