WRT #firstNonNull, I don't know why I couldn't find it before. I do now, and I agree it would seem to fit better in ArrayUtils.
Matt On Mon, Jul 8, 2013 at 10:04 AM, Jörg Schaible <joerg.schai...@scalaris.com>wrote: > Hi Matt, > > Matt Benson wrote: > > > 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. > > > http://commons.apache.org/proper/commons-lang/javadocs/api-release/org/apache/commons/lang3/ObjectUtils.html#firstNonNull(T.. > .) > > > 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. > > I've re-read the original code and you're right. > > - Jörg > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > >
