Ok, made that change.  Left all the beans alone.

-David

On Sep 16, 2009, at 4:04 PM, David Blevins wrote:

Hi All,

Wondering if there'd be any objections to me updating all the boolean returning methods that use isFooExist() to use something that works grammatically like hasFoo().

For example in AnnotationUtil:

public static boolean isMethodParameterAnnotationExist(Method method, Class<? extends Annotation> clazz)

Would become:

public static boolean hasMethodParameterAnnotation(Method method, Class<? extends Annotation> clazz)

And then usage of it would look like this:

 for (Method m : methods)
 {
     if (hasMethodParameterAnnotation(m, annotation))
     {
         list.add(m);
     }
 }

Which in english would read "if method 'm' has the method parameter annotation 'annotation' ...."

Thoughts?


-David



Reply via email to