Merge pull request #3138 from dukbong/Refactoring

Use isEmpty() instead of length()
mybatis · Apr 14, 2024 · e9e9a29 · e9e9a29
2 parents 382cbf4 + a020b39
commit e9e9a29
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/main/java/org/apache/ibatis/executor/keygen/SelectKeyGenerator.java b/src/main/java/org/apache/ibatis/executor/keygen/SelectKeyGenerator.java
@@ -65,7 +65,7 @@ private void processGeneratedKeys(Executor executor, MappedStatement ms, Object
         // The transaction will be closed by parent executor.
         Executor keyExecutor = configuration.newExecutor(executor.getTransaction(), ExecutorType.SIMPLE);
         List<Object> values = keyExecutor.query(keyStatement, parameter, RowBounds.DEFAULT, Executor.NO_RESULT_HANDLER);
-        if (values.size() == 0) {
+        if (values.isEmpty()) {
           throw new ExecutorException("SelectKey returned no data.");
         }
         if (values.size() > 1) {