On Mon, Jul 8, 2013 at 8:22 AM, Jörg Schaible
<joerg.schai...@scalaris.com>wrote:

> Hi Hen,
>
> Henri Yandell wrote:
>
> > I don't see any value having the first two methods - replacing the '=='
> > sign is a bit too far in the 'provide simple methods' direction I think
> :)
> >
> > I think the third method is already in Lang as:
> >
> >     ArrayUtils.contains(array, null);
>
>
> Well, no, this is not, what the method does!


Correct, but to be fair, it's simple enough to use this method to implement
the desired check.  Negate the result.


> With Lang we could use now:
>
>     ObjectUtils.firstNotNull(array) != null
>

I don't see such a method as this.  If it did exist, it still wouldn't
implement the feature requested (check that no element of the array is
null).  I'm pretty sure !ArrayUtils.contains(array, null) is fine.

Matt


>
> > Having a containsNull is semantically nice, but I'm not sure it's needed.
>
> The question is here more, why the method is part of ObjectUtils, since it
> operates on an Array and I did not find it immediately (and I am probably
> not alone).
>
> Cheers,
> Jörg
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>
>

Reply via email to