Merge pull request #1470 from kezhenxu94/polishing-remove-if

use removeIf and list.sort to polish code

Author: kazuki43zoo

src/main/java/org/apache/ibatis/builder/MapperBuilderAssistant.java

@@ -199,12 +199,7 @@ public ResultMap addResultMap(
         }
       }
       if (declaresConstructor) {
-        Iterator<ResultMapping> extendedResultMappingsIter = extendedResultMappings.iterator();
-        while (extendedResultMappingsIter.hasNext()) {
-          if (extendedResultMappingsIter.next().getFlags().contains(ResultFlag.CONSTRUCTOR)) {
-            extendedResultMappingsIter.remove();
-          }
-        }
+        extendedResultMappings.removeIf(resultMapping -> resultMapping.getFlags().contains(ResultFlag.CONSTRUCTOR));
       }
       resultMappings.addAll(extendedResultMappings);
     }

src/main/java/org/apache/ibatis/mapping/ResultMap.java

@@ -130,7 +130,7 @@ public ResultMap build() {
               + resultMap.getType().getName() + "' by arg names " + constructorArgNames
               + ". There might be more info in debug log.");
         }
-        Collections.sort(resultMap.constructorResultMappings, (o1, o2) -> {
+        resultMap.constructorResultMappings.sort((o1, o2) -> {
           int paramIdx1 = actualArgNames.indexOf(o1.getProperty());
           int paramIdx2 = actualArgNames.indexOf(o2.getProperty());
           return paramIdx1 - paramIdx2;