I use an Assert class (similar to commons Validate):

Assert.notNull(argName, arg);

http://ci.apache.org/projects/ofbiz/site/javadocs/org/ofbiz/base/util/Assert.html

The difference is it throws IllegalArgumentException.

-Adrian


On 7/7/2011 5:56 PM, Gary Gregory wrote:
Hi All:

I do like using NullArgumentException, but I find writing this over and over
tedious:

if (arg == null) {
  thrown new NullArgumentException(argName);
}
something(arg);

How about this instead:

NullArgumentException.check(arg, argName);
something(arg);

or:

something(NullArgumentException.check(arg, argName));

Depending on the style you like.

Where check is:

     public static<T>  T check(T arg, String argName) {
         if (arg == null) {
             throw new NullArgumentException(argName);
         }
         return arg;
     }

Yes, you are pushing the argName on the stack (or passing it in a register)
and that is extra work, but you do not have to use the new method then ;)

?


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to