Merge pull request mybatis#3139 from dukbong/Replace

Recommend using isEmpty() instead of size().

Author: hazendaz

src/main/java/org/apache/ibatis/executor/loader/ResultLoaderMap.java

@@ -70,6 +70,10 @@ public Set<String> getPropertyNames() {
   public int size() {
     return loaderMap.size();
   }
+  
+  public boolean isEmpty() {
+    return loaderMap.isEmpty();
+  }
 
   public boolean hasLoader(String property) {
     return loaderMap.containsKey(property.toUpperCase(Locale.ENGLISH));

src/main/java/org/apache/ibatis/executor/loader/javassist/JavassistProxyFactory.java

@@ -143,14 +143,14 @@ public Object invoke(Object enhanced, Method method, Method methodProxy, Object[
             original = objectFactory.create(type, constructorArgTypes, constructorArgs);
           }
           PropertyCopier.copyBeanProperties(type, enhanced, original);
-          if (lazyLoader.size() > 0) {
+          if (!lazyLoader.isEmpty()) {
             return new JavassistSerialStateHolder(original, lazyLoader.getProperties(), objectFactory,
                 constructorArgTypes, constructorArgs);
           } else {
             return original;
           }
         }
-        if (lazyLoader.size() > 0 && !FINALIZE_METHOD.equals(methodName)) {
+        if (!lazyLoader.isEmpty() && !FINALIZE_METHOD.equals(methodName)) {
           if (aggressive || lazyLoadTriggerMethods.contains(methodName)) {
             lazyLoader.loadAll();
           } else if (PropertyNamer.isSetter(methodName)) {