Hi David;

+1;

That is more meaningful!


Thanks;

--Gurkan


________________________________
From: David Blevins <[email protected]>
To: [email protected]
Sent: Thursday, September 17, 2009 2:04:11 AM
Subject: small change: isFooExist() -> hasFoo()

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