commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Adrian Crum <adrian.c...@sandglass-software.com>
Subject Re: [lang] NullArgumentException usage
Date Thu, 07 Jul 2011 17:14:55 GMT
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-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message