Browse Source

alibaba 规范调整

hubin 7 years ago
parent
commit
855f57de71
29 changed files with 165 additions and 110 deletions
  1. 3 1
      src/main/java/com/baomidou/mybatisplus/MybatisDefaultParameterHandler.java
  2. 1 0
      src/main/java/com/baomidou/mybatisplus/MybatisMapperAnnotationBuilder.java
  3. 3 1
      src/main/java/com/baomidou/mybatisplus/MybatisMapperRegistry.java
  4. 32 34
      src/main/java/com/baomidou/mybatisplus/MybatisSqlSessionTemplate.java
  5. 2 1
      src/main/java/com/baomidou/mybatisplus/generator/AutoGenerator.java
  6. 1 0
      src/main/java/com/baomidou/mybatisplus/generator/config/converts/MySqlTypeConvert.java
  7. 1 0
      src/main/java/com/baomidou/mybatisplus/generator/config/converts/OracleTypeConvert.java
  8. 1 0
      src/main/java/com/baomidou/mybatisplus/generator/config/converts/PostgreSqlTypeConvert.java
  9. 1 0
      src/main/java/com/baomidou/mybatisplus/generator/config/converts/SqlServerTypeConvert.java
  10. 9 5
      src/main/java/com/baomidou/mybatisplus/mapper/AutoSqlInjector.java
  11. 1 0
      src/main/java/com/baomidou/mybatisplus/mapper/EntityWrapper.java
  12. 2 0
      src/main/java/com/baomidou/mybatisplus/mapper/LogicSqlInjector.java
  13. 53 26
      src/main/java/com/baomidou/mybatisplus/mapper/Wrapper.java
  14. 6 0
      src/main/java/com/baomidou/mybatisplus/plugins/CachePaginationInterceptor.java
  15. 3 0
      src/main/java/com/baomidou/mybatisplus/plugins/PerformanceInterceptor.java
  16. 5 2
      src/main/java/com/baomidou/mybatisplus/plugins/SqlExplainInterceptor.java
  17. 1 0
      src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/DB2Dialect.java
  18. 1 0
      src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/H2Dialect.java
  19. 1 0
      src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/HSQLDialect.java
  20. 1 0
      src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/MySqlDialect.java
  21. 1 0
      src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/OracleDialect.java
  22. 1 0
      src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/PostgreDialect.java
  23. 1 0
      src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/SQLServer2005Dialect.java
  24. 1 0
      src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/SQLServerDialect.java
  25. 1 0
      src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/SQLiteDialect.java
  26. 2 0
      src/main/java/com/baomidou/mybatisplus/spring/MybatisMapperRefresh.java
  27. 22 36
      src/main/java/com/baomidou/mybatisplus/toolkit/StringEscape.java
  28. 4 4
      src/main/java/com/baomidou/mybatisplus/toolkit/StringUtils.java
  29. 4 0
      src/main/java/com/baomidou/mybatisplus/toolkit/SystemClock.java

+ 3 - 1
src/main/java/com/baomidou/mybatisplus/MybatisDefaultParameterHandler.java

@@ -104,7 +104,9 @@ public class MybatisDefaultParameterHandler extends DefaultParameterHandler {
      */
     protected static Object processBatch(MappedStatement ms, Object parameterObject) {
         //检查parameterObject
-        if (null == parameterObject) return null;
+        if (null == parameterObject) {
+            return null;
+        }
         boolean isFill = false;
         // 全局配置是否配置填充器
         MetaObjectHandler metaObjectHandler = GlobalConfigUtils.getMetaObjectHandler(ms.getConfiguration());

+ 1 - 0
src/main/java/com/baomidou/mybatisplus/MybatisMapperAnnotationBuilder.java

@@ -129,6 +129,7 @@ public class MybatisMapperAnnotationBuilder extends MapperAnnotationBuilder {
         sqlProviderAnnotationTypes.add(DeleteProvider.class);
     }
 
+    @Override
     public void parse() {
         String resource = type.toString();
         if (!configuration.isResourceLoaded(resource)) {

+ 3 - 1
src/main/java/com/baomidou/mybatisplus/MybatisMapperRegistry.java

@@ -48,7 +48,7 @@ public class MybatisMapperRegistry extends MapperRegistry {
         GlobalConfigUtils.getSqlInjector(config).injectSqlRunner(config);
     }
 
-    @SuppressWarnings("unchecked")
+    @Override
     public <T> T getMapper(Class<T> type, SqlSession sqlSession) {
         final MapperProxyFactory<T> mapperProxyFactory = (MapperProxyFactory<T>) knownMappers.get(type);
         if (mapperProxyFactory == null) {
@@ -61,6 +61,7 @@ public class MybatisMapperRegistry extends MapperRegistry {
         }
     }
 
+    @Override
     public <T> boolean hasMapper(Class<T> type) {
         return knownMappers.containsKey(type);
     }
@@ -97,6 +98,7 @@ public class MybatisMapperRegistry extends MapperRegistry {
     /**
      * @since 3.2.2
      */
+    @Override
     public Collection<Class<?>> getMappers() {
         return Collections.unmodifiableCollection(knownMappers.keySet());
     }

+ 32 - 34
src/main/java/com/baomidou/mybatisplus/MybatisSqlSessionTemplate.java

@@ -118,7 +118,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public <T> T selectOne(String statement) {
         return this.sqlSessionProxy.<T>selectOne(statement);
     }
@@ -126,7 +126,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public <T> T selectOne(String statement, Object parameter) {
         return this.sqlSessionProxy.<T>selectOne(statement, parameter);
     }
@@ -134,7 +134,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public <K, V> Map<K, V> selectMap(String statement, String mapKey) {
         return this.sqlSessionProxy.<K, V>selectMap(statement, mapKey);
     }
@@ -142,7 +142,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public <K, V> Map<K, V> selectMap(String statement, Object parameter, String mapKey) {
         return this.sqlSessionProxy.<K, V>selectMap(statement, parameter, mapKey);
     }
@@ -150,7 +150,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public <K, V> Map<K, V> selectMap(String statement, Object parameter, String mapKey, RowBounds rowBounds) {
         return this.sqlSessionProxy.<K, V>selectMap(statement, parameter, mapKey, rowBounds);
     }
@@ -158,7 +158,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public <T> Cursor<T> selectCursor(String statement) {
         return this.sqlSessionProxy.selectCursor(statement);
     }
@@ -166,7 +166,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public <T> Cursor<T> selectCursor(String statement, Object parameter) {
         return this.sqlSessionProxy.selectCursor(statement, parameter);
     }
@@ -174,7 +174,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public <T> Cursor<T> selectCursor(String statement, Object parameter, RowBounds rowBounds) {
         return this.sqlSessionProxy.selectCursor(statement, parameter, rowBounds);
     }
@@ -182,7 +182,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public <E> List<E> selectList(String statement) {
         return this.sqlSessionProxy.<E>selectList(statement);
     }
@@ -190,7 +190,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public <E> List<E> selectList(String statement, Object parameter) {
         return this.sqlSessionProxy.<E>selectList(statement, parameter);
     }
@@ -198,7 +198,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public <E> List<E> selectList(String statement, Object parameter, RowBounds rowBounds) {
         return this.sqlSessionProxy.<E>selectList(statement, parameter, rowBounds);
     }
@@ -206,8 +206,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
-    @SuppressWarnings("rawtypes")
+    @Override
     public void select(String statement, ResultHandler handler) {
         this.sqlSessionProxy.select(statement, handler);
     }
@@ -215,8 +214,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
-    @SuppressWarnings("rawtypes")
+    @Override
     public void select(String statement, Object parameter, ResultHandler handler) {
         this.sqlSessionProxy.select(statement, parameter, handler);
     }
@@ -224,8 +222,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
-    @SuppressWarnings("rawtypes")
+    @Override
     public void select(String statement, Object parameter, RowBounds rowBounds, ResultHandler handler) {
         this.sqlSessionProxy.select(statement, parameter, rowBounds, handler);
     }
@@ -233,7 +230,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public int insert(String statement) {
         return this.sqlSessionProxy.insert(statement);
     }
@@ -241,7 +238,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public int insert(String statement, Object parameter) {
         return this.sqlSessionProxy.insert(statement, parameter);
     }
@@ -249,7 +246,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public int update(String statement) {
         return this.sqlSessionProxy.update(statement);
     }
@@ -257,7 +254,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public int update(String statement, Object parameter) {
         return this.sqlSessionProxy.update(statement, parameter);
     }
@@ -265,7 +262,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public int delete(String statement) {
         return this.sqlSessionProxy.delete(statement);
     }
@@ -273,7 +270,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public int delete(String statement, Object parameter) {
         return this.sqlSessionProxy.delete(statement, parameter);
     }
@@ -281,7 +278,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public <T> T getMapper(Class<T> type) {
         return getConfiguration().getMapper(type, this);
     }
@@ -289,7 +286,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public void commit() {
         throw new UnsupportedOperationException("Manual commit is not allowed over a Spring managed SqlSession");
     }
@@ -297,7 +294,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public void commit(boolean force) {
         throw new UnsupportedOperationException("Manual commit is not allowed over a Spring managed SqlSession");
     }
@@ -305,7 +302,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public void rollback() {
         throw new UnsupportedOperationException("Manual rollback is not allowed over a Spring managed SqlSession");
     }
@@ -313,7 +310,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public void rollback(boolean force) {
         throw new UnsupportedOperationException("Manual rollback is not allowed over a Spring managed SqlSession");
     }
@@ -321,7 +318,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public void close() {
         throw new UnsupportedOperationException("Manual close is not allowed over a Spring managed SqlSession");
     }
@@ -329,7 +326,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public void clearCache() {
         this.sqlSessionProxy.clearCache();
     }
@@ -337,7 +334,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public Configuration getConfiguration() {
         return this.sqlSessionFactory.getConfiguration();
     }
@@ -345,7 +342,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
     /**
      * {@inheritDoc}
      */
-
+    @Override
     public Connection getConnection() {
         return this.sqlSessionProxy.getConnection();
     }
@@ -355,7 +352,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
      *
      * @since 1.0.2
      */
-
+    @Override
     public List<BatchResult> flushStatements() {
         return this.sqlSessionProxy.flushStatements();
     }
@@ -381,7 +378,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
      * @see org.springframework.beans.factory.support.DisposableBeanAdapter#inferDestroyMethodIfNecessary
      * @see org.springframework.beans.factory.support.DisposableBeanAdapter#CLOSE_METHOD_NAME
      */
-
+    @Override
     public void destroy() throws Exception {
         // This method forces spring disposer to avoid call of
         // SqlSessionTemplate.close() which gives UnsupportedOperationException
@@ -396,6 +393,7 @@ public class MybatisSqlSessionTemplate implements SqlSession, DisposableBean {
      */
     private class SqlSessionInterceptor implements InvocationHandler {
 
+        @Override
         public Object invoke(Object proxy, Method method, Object[] args) throws Throwable {
             SqlSession sqlSession = MybatisSqlSessionTemplate.this.sqlSessionFactory
                     .openSession(MybatisSqlSessionTemplate.this.executorType);

+ 2 - 1
src/main/java/com/baomidou/mybatisplus/generator/AutoGenerator.java

@@ -238,8 +238,9 @@ public class AutoGenerator {
      * @return
      */
     private String getSuperClassName(String classPath) {
-        if (StringUtils.isEmpty(classPath))
+        if (StringUtils.isEmpty(classPath)) {
             return null;
+        }
         return classPath.substring(classPath.lastIndexOf(".") + 1);
     }
 

+ 1 - 0
src/main/java/com/baomidou/mybatisplus/generator/config/converts/MySqlTypeConvert.java

@@ -28,6 +28,7 @@ import com.baomidou.mybatisplus.generator.config.rules.DbColumnType;
  */
 public class MySqlTypeConvert implements ITypeConvert {
 
+    @Override
     public DbColumnType processTypeConvert(String fieldType) {
         String t = fieldType.toLowerCase();
         if (t.contains("char") || t.contains("text")) {

+ 1 - 0
src/main/java/com/baomidou/mybatisplus/generator/config/converts/OracleTypeConvert.java

@@ -28,6 +28,7 @@ import com.baomidou.mybatisplus.generator.config.rules.DbColumnType;
  */
 public class OracleTypeConvert implements ITypeConvert {
 
+    @Override
     public DbColumnType processTypeConvert(String fieldType) {
         String t = fieldType.toUpperCase();
         if (t.contains("CHAR")) {

+ 1 - 0
src/main/java/com/baomidou/mybatisplus/generator/config/converts/PostgreSqlTypeConvert.java

@@ -28,6 +28,7 @@ import com.baomidou.mybatisplus.generator.config.rules.DbColumnType;
  */
 public class PostgreSqlTypeConvert implements ITypeConvert {
 
+    @Override
     public DbColumnType processTypeConvert(String fieldType) {
         String t = fieldType.toLowerCase();
         if (t.contains("char") || t.contains("text")) {

+ 1 - 0
src/main/java/com/baomidou/mybatisplus/generator/config/converts/SqlServerTypeConvert.java

@@ -28,6 +28,7 @@ import com.baomidou.mybatisplus.generator.config.rules.DbColumnType;
  */
 public class SqlServerTypeConvert implements ITypeConvert {
 
+    @Override
     public DbColumnType processTypeConvert(String fieldType) {
         String t = fieldType.toLowerCase();
         if (t.contains("char") || t.contains("text") || t.contains("xml")) {

+ 9 - 5
src/main/java/com/baomidou/mybatisplus/mapper/AutoSqlInjector.java

@@ -74,6 +74,7 @@ public class AutoSqlInjector implements ISqlInjector {
      * @param builderAssistant
      * @param mapperClass
      */
+    @Override
     public void inspectInject(MapperBuilderAssistant builderAssistant, Class<?> mapperClass) {
         String className = mapperClass.toString();
         Set<String> mapperRegistryCache = GlobalConfigUtils.getMapperRegistryCache(builderAssistant.getConfiguration());
@@ -86,6 +87,7 @@ public class AutoSqlInjector implements ISqlInjector {
     /**
      * 注入单点 crudSql
      */
+    @Override
     public void inject(MapperBuilderAssistant builderAssistant, Class<?> mapperClass) {
         this.configuration = builderAssistant.getConfiguration();
         this.builderAssistant = builderAssistant;
@@ -606,9 +608,9 @@ public class AutoSqlInjector implements ISqlInjector {
                 columns.append("<choose><when test=\"ew != null and ew.sqlSelect != null\">${ew.sqlSelect}</when><otherwise>");
             }
             List<TableFieldInfo> fieldList = table.getFieldList();
-            int _size = 0;
+            int size = 0;
             if (null != fieldList) {
-                _size = fieldList.size();
+                size = fieldList.size();
             }
 
             // 主键处理
@@ -618,13 +620,13 @@ public class AutoSqlInjector implements ISqlInjector {
                 } else {
                     columns.append(sqlWordConvert(table.getKeyProperty()));
                 }
-                if (_size >= 1) {
+                if (size >= 1) {
                     // 判断其余字段是否存在
                     columns.append(",");
                 }
             }
 
-            if (_size >= 1) {
+            if (size >= 1) {
                 // 字段处理
                 int i = 0;
                 Iterator<TableFieldInfo> iterator = fieldList.iterator();
@@ -639,7 +641,7 @@ public class AutoSqlInjector implements ISqlInjector {
                         columns.append(fieldInfo.getColumn());
                         columns.append(" AS ").append(wordConvert);
                     }
-                    if (i + 1 < _size) {
+                    if (i + 1 < size) {
                         columns.append(",");
                     }
                     i++;
@@ -862,6 +864,8 @@ public class AutoSqlInjector implements ISqlInjector {
     }
 
     // --------------------------------------------------------SqlRunner------------------------------------------------------------
+
+    @Override
     public void injectSqlRunner(Configuration configuration) {
         this.configuration = configuration;
         this.languageDriver = configuration.getDefaultScriptingLanguageInstance();

+ 1 - 0
src/main/java/com/baomidou/mybatisplus/mapper/EntityWrapper.java

@@ -46,6 +46,7 @@ public class EntityWrapper<T> extends Wrapper<T> {
         this.sqlSelect = sqlSelect;
     }
 
+    @Override
     public T getEntity() {
         return entity;
     }

+ 2 - 0
src/main/java/com/baomidou/mybatisplus/mapper/LogicSqlInjector.java

@@ -112,6 +112,7 @@ public class LogicSqlInjector extends AutoSqlInjector {
      * @param modelClass
      * @param table
      */
+    @Override
     protected void injectSelectByIdSql(boolean batch, Class<?> mapperClass, Class<?> modelClass, TableInfo table) {
         if (table.isLogicDelete()) {
             SqlMethod sqlMethod = SqlMethod.LOGIC_SELECT_BY_ID;
@@ -144,6 +145,7 @@ public class LogicSqlInjector extends AutoSqlInjector {
      * @param modelClass
      * @param table
      */
+    @Override
     protected void injectUpdateByIdSql(boolean selective, Class<?> mapperClass, Class<?> modelClass, TableInfo table) {
         if (table.isLogicDelete()) {
             SqlMethod sqlMethod = selective ? SqlMethod.LOGIC_UPDATE_BY_ID : SqlMethod.LOGIC_UPDATE_ALL_COLUMN_BY_ID;

+ 53 - 26
src/main/java/com/baomidou/mybatisplus/mapper/Wrapper.java

@@ -189,6 +189,7 @@ public abstract class Wrapper<T> implements Serializable {
      */
     public abstract String getSqlSegment();
 
+    @Override
     public String toString() {
         StringBuilder sb = new StringBuilder("Wrapper<T>:");
         String sqlSegment = getSqlSegment();
@@ -245,8 +246,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> where(boolean condition, String sqlWhere, Object... params) {
-        if (condition)
+        if (condition) {
             sql.WHERE(formatSql(sqlWhere, params));
+        }
         return this;
     }
 
@@ -279,8 +281,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return
      */
     public Wrapper<T> eq(boolean condition, String column, Object params) {
-        if (condition)
+        if (condition) {
             sql.WHERE(formatSql(String.format("%s = {0}", column), params));
+        }
         return this;
     }
 
@@ -308,8 +311,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return
      */
     public Wrapper<T> ne(boolean condition, String column, Object params) {
-        if (condition)
+        if (condition) {
             sql.WHERE(formatSql(String.format("%s <> {0}", column), params));
+        }
         return this;
     }
 
@@ -377,8 +381,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return
      */
     public Wrapper<T> gt(boolean condition, String column, Object params) {
-        if (condition)
+        if (condition) {
             sql.WHERE(formatSql(String.format("%s > {0}", column), params));
+        }
         return this;
     }
 
@@ -406,8 +411,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return
      */
     public Wrapper<T> ge(boolean condition, String column, Object params) {
-        if (condition)
+        if (condition) {
             sql.WHERE(formatSql(String.format("%s >= {0}", column), params));
+        }
         return this;
     }
 
@@ -435,8 +441,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return
      */
     public Wrapper<T> lt(boolean condition, String column, Object params) {
-        if (condition)
+        if (condition) {
             sql.WHERE(formatSql(String.format("%s < {0}", column), params));
+        }
         return this;
     }
 
@@ -464,8 +471,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return
      */
     public Wrapper<T> le(boolean condition, String column, Object params) {
-        if (condition)
+        if (condition) {
             sql.WHERE(formatSql(String.format("%s <= {0}", column), params));
+        }
         return this;
     }
 
@@ -493,8 +501,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> and(boolean condition, String sqlAnd, Object... params) {
-        if (condition)
+        if (condition) {
             sql.AND().WHERE(formatSql(sqlAnd, params));
+        }
         return this;
     }
 
@@ -526,8 +535,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> andNew(boolean condition, String sqlAnd, Object... params) {
-        if (condition)
+        if (condition) {
             sql.AND_NEW().WHERE(formatSql(sqlAnd, params));
+        }
         return this;
     }
 
@@ -692,8 +702,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> groupBy(boolean condition, String columns) {
-        if (condition)
+        if (condition) {
             sql.GROUP_BY(columns);
+        }
         return this;
     }
 
@@ -726,8 +737,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return EntityWrapper<T>
      */
     public Wrapper<T> having(boolean condition, String sqlHaving, Object... params) {
-        if (condition)
+        if (condition) {
             sql.HAVING(formatSql(sqlHaving, params));
+        }
         return this;
     }
 
@@ -761,8 +773,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> orderBy(boolean condition, String columns) {
-        if (condition)
+        if (condition) {
             sql.ORDER_BY(columns);
+        }
         return this;
     }
 
@@ -823,8 +836,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> like(boolean condition, String column, String value) {
-        if (condition)
+        if (condition) {
             handerLike(column, value, SqlLike.DEFAULT, false);
+        }
         return this;
     }
 
@@ -852,8 +866,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> notLike(boolean condition, String column, String value) {
-        if (condition)
+        if (condition) {
             handerLike(column, value, SqlLike.DEFAULT, true);
+        }
         return this;
     }
 
@@ -903,8 +918,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> like(boolean condition, String column, String value, SqlLike type) {
-        if (condition)
+        if (condition) {
             handerLike(column, value, type, false);
+        }
         return this;
     }
 
@@ -934,8 +950,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> notLike(boolean condition, String column, String value, SqlLike type) {
-        if (condition)
+        if (condition) {
             handerLike(column, value, type, true);
+        }
         return this;
     }
 
@@ -963,8 +980,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> isNotNull(boolean condition, String columns) {
-        if (condition)
+        if (condition) {
             sql.IS_NOT_NULL(columns);
+        }
         return this;
     }
 
@@ -990,8 +1008,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> isNull(boolean condition, String columns) {
-        if (condition)
+        if (condition) {
             sql.IS_NULL(columns);
+        }
         return this;
     }
 
@@ -1017,8 +1036,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> exists(boolean condition, String value) {
-        if (condition)
+        if (condition) {
             sql.EXISTS(value);
+        }
         return this;
     }
 
@@ -1044,8 +1064,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> notExists(boolean condition, String value) {
-        if (condition)
+        if (condition) {
             sql.NOT_EXISTS(value);
+        }
         return this;
     }
 
@@ -1132,8 +1153,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> in(boolean condition, String column, Collection<?> value) {
-        if (condition && CollectionUtils.isNotEmpty(value))
+        if (condition && CollectionUtils.isNotEmpty(value)) {
             sql.WHERE(formatSql(inExpression(column, value, false), value.toArray()));
+        }
         return this;
     }
 
@@ -1161,8 +1183,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> notIn(boolean condition, String column, Collection<?> value) {
-        if (condition && CollectionUtils.isNotEmpty(value))
+        if (condition && CollectionUtils.isNotEmpty(value)) {
             sql.WHERE(formatSql(inExpression(column, value, true), value.toArray()));
+        }
         return this;
     }
 
@@ -1190,8 +1213,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> in(boolean condition, String column, Object[] value) {
-        if (condition && ArrayUtils.isNotEmpty(value))
+        if (condition && ArrayUtils.isNotEmpty(value)) {
             sql.WHERE(formatSql(inExpression(column, Arrays.asList(value), false), value));
+        }
         return this;
     }
 
@@ -1219,8 +1243,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> notIn(boolean condition, String column, Object... value) {
-        if (condition && ArrayUtils.isNotEmpty(value))
+        if (condition && ArrayUtils.isNotEmpty(value)) {
             sql.WHERE(formatSql(inExpression(column, Arrays.asList(value), true), value));
+        }
         return this;
     }
 
@@ -1280,8 +1305,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> between(boolean condition, String column, Object val1, Object val2) {
-        if (condition)
+        if (condition) {
             sql.WHERE(formatSql(String.format("%s BETWEEN {0} AND {1}", column), val1, val2));
+        }
         return this;
     }
 
@@ -1311,8 +1337,9 @@ public abstract class Wrapper<T> implements Serializable {
      * @return this
      */
     public Wrapper<T> notBetween(boolean condition, String column, Object val1, Object val2) {
-        if (condition)
+        if (condition) {
             sql.WHERE(formatSql(String.format("%s NOT BETWEEN {0} AND {1}", column), val1, val2));
+        }
         return this;
     }
 

+ 6 - 0
src/main/java/com/baomidou/mybatisplus/plugins/CachePaginationInterceptor.java

@@ -67,6 +67,7 @@ public class CachePaginationInterceptor extends PaginationInterceptor implements
      * Physical Pagination Interceptor for all the queries with parameter
      * {@link org.apache.ibatis.session.RowBounds}
      */
+    @Override
     public Object intercept(Invocation invocation) throws Throwable {
 
         Object target = invocation.getTarget();
@@ -97,6 +98,7 @@ public class CachePaginationInterceptor extends PaginationInterceptor implements
         return invocation.proceed();
     }
 
+    @Override
     public Object plugin(Object target) {
         if (target instanceof Executor) {
             return Plugin.wrap(target, this);
@@ -107,6 +109,7 @@ public class CachePaginationInterceptor extends PaginationInterceptor implements
         return target;
     }
 
+    @Override
     public void setProperties(Properties prop) {
         String dialectType = prop.getProperty("dialectType");
         String dialectClazz = prop.getProperty("dialectClazz");
@@ -118,16 +121,19 @@ public class CachePaginationInterceptor extends PaginationInterceptor implements
         }
     }
 
+    @Override
     public CachePaginationInterceptor setDialectType(String dialectType) {
         this.dialectType = dialectType;
         return this;
     }
 
+    @Override
     public CachePaginationInterceptor setSqlParser(ISqlParser sqlParser) {
         this.sqlParser = sqlParser;
         return this;
     }
 
+    @Override
     public CachePaginationInterceptor setOverflowCurrent(boolean overflowCurrent) {
         this.overflowCurrent = overflowCurrent;
         return this;

+ 3 - 0
src/main/java/com/baomidou/mybatisplus/plugins/PerformanceInterceptor.java

@@ -73,6 +73,7 @@ public class PerformanceInterceptor implements Interceptor {
     private Method oracleGetOriginalSqlMethod;
     private Method druidGetSQLMethod;
 
+    @Override
     public Object intercept(Invocation invocation) throws Throwable {
         Statement statement;
         Object firstArg = invocation.getArgs()[0];
@@ -163,6 +164,7 @@ public class PerformanceInterceptor implements Interceptor {
         return result;
     }
 
+    @Override
     public Object plugin(Object target) {
         if (target instanceof StatementHandler) {
             return Plugin.wrap(target, this);
@@ -170,6 +172,7 @@ public class PerformanceInterceptor implements Interceptor {
         return target;
     }
 
+    @Override
     public void setProperties(Properties prop) {
         String maxTime = prop.getProperty("maxTime");
         String format = prop.getProperty("format");

+ 5 - 2
src/main/java/com/baomidou/mybatisplus/plugins/SqlExplainInterceptor.java

@@ -62,6 +62,7 @@ public class SqlExplainInterceptor implements Interceptor {
      */
     private boolean stopProceed = false;
 
+    @Override
     public Object intercept(Invocation invocation) throws Throwable {
         /**
          * 处理 DELETE UPDATE 语句
@@ -110,8 +111,8 @@ public class SqlExplainInterceptor implements Interceptor {
                 SqlCommandType.SELECT);
         builder.resultMaps(mappedStatement.getResultMaps()).resultSetType(mappedStatement.getResultSetType())
                 .statementType(mappedStatement.getStatementType());
-        MappedStatement query_statement = builder.build();
-        DefaultParameterHandler handler = new DefaultParameterHandler(query_statement, parameter, boundSql);
+        MappedStatement queryStatement = builder.build();
+        DefaultParameterHandler handler = new DefaultParameterHandler(queryStatement, parameter, boundSql);
         try (PreparedStatement stmt = connection.prepareStatement(sqlExplain)) {
             handler.setParameters(stmt);
             try (ResultSet rs = stmt.executeQuery()) {
@@ -131,6 +132,7 @@ public class SqlExplainInterceptor implements Interceptor {
         }
     }
 
+    @Override
     public Object plugin(Object target) {
         if (target instanceof Executor) {
             return Plugin.wrap(target, this);
@@ -138,6 +140,7 @@ public class SqlExplainInterceptor implements Interceptor {
         return target;
     }
 
+    @Override
     public void setProperties(Properties prop) {
         String stopProceed = prop.getProperty("stopProceed");
         if (StringUtils.isNotEmpty(stopProceed)) {

+ 1 - 0
src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/DB2Dialect.java

@@ -43,6 +43,7 @@ public class DB2Dialect implements IDialect {
         return originalSql.toLowerCase().contains("select distinct");
     }
 
+    @Override
     public String buildPaginationSql(String originalSql, int offset, int limit) {
         int startOfSelect = originalSql.toLowerCase().indexOf("select");
         StringBuilder pagingSelect = new StringBuilder(originalSql.length() + 100)

+ 1 - 0
src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/H2Dialect.java

@@ -29,6 +29,7 @@ public class H2Dialect implements IDialect {
 
     public static final H2Dialect INSTANCE = new H2Dialect();
 
+    @Override
     public String buildPaginationSql(String originalSql, int offset, int limit) {
         StringBuilder sql = new StringBuilder(originalSql);
         sql.append(" limit ").append(limit);

+ 1 - 0
src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/HSQLDialect.java

@@ -29,6 +29,7 @@ public class HSQLDialect implements IDialect {
 
     public static final HSQLDialect INSTANCE = new HSQLDialect();
 
+    @Override
     public String buildPaginationSql(String originalSql, int offset, int limit) {
         StringBuilder sql = new StringBuilder(originalSql);
         sql.append(" limit ").append(offset).append(",").append(limit);

+ 1 - 0
src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/MySqlDialect.java

@@ -29,6 +29,7 @@ public class MySqlDialect implements IDialect {
 
     public static final MySqlDialect INSTANCE = new MySqlDialect();
 
+    @Override
     public String buildPaginationSql(String originalSql, int offset, int limit) {
         StringBuilder sql = new StringBuilder(originalSql);
         sql.append(" LIMIT ").append(offset).append(",").append(limit);

+ 1 - 0
src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/OracleDialect.java

@@ -29,6 +29,7 @@ public class OracleDialect implements IDialect {
 
     public static final OracleDialect INSTANCE = new OracleDialect();
 
+    @Override
     public String buildPaginationSql(String originalSql, int offset, int limit) {
         StringBuilder sql = new StringBuilder();
         sql.append("SELECT * FROM ( SELECT TMP.*, ROWNUM ROW_ID FROM ( ");

+ 1 - 0
src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/PostgreDialect.java

@@ -29,6 +29,7 @@ public class PostgreDialect implements IDialect {
 
     public static final PostgreDialect INSTANCE = new PostgreDialect();
 
+    @Override
     public String buildPaginationSql(String originalSql, int offset, int limit) {
         StringBuilder sql = new StringBuilder(originalSql);
         sql.append(" limit ").append(limit).append(" offset ").append(offset);

+ 1 - 0
src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/SQLServer2005Dialect.java

@@ -40,6 +40,7 @@ public class SQLServer2005Dialect implements IDialect {
         }
     }
 
+    @Override
     public String buildPaginationSql(String originalSql, int offset, int limit) {
         StringBuilder pagingBuilder = new StringBuilder();
         String orderby = getOrderByPart(originalSql);

+ 1 - 0
src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/SQLServerDialect.java

@@ -29,6 +29,7 @@ public class SQLServerDialect implements IDialect {
 
     public static final SQLServerDialect INSTANCE = new SQLServerDialect();
 
+    @Override
     public String buildPaginationSql(String originalSql, int offset, int limit) {
         StringBuilder sql = new StringBuilder(originalSql);
         sql.append(" OFFSET ").append(offset).append(" ROWS FETCH NEXT ");

+ 1 - 0
src/main/java/com/baomidou/mybatisplus/plugins/pagination/dialects/SQLiteDialect.java

@@ -29,6 +29,7 @@ public class SQLiteDialect implements IDialect {
 
     public static final SQLiteDialect INSTANCE = new SQLiteDialect();
 
+    @Override
     public String buildPaginationSql(String originalSql, int offset, int limit) {
         StringBuilder sql = new StringBuilder(originalSql);
         sql.append(" limit ").append(limit).append(" offset ").append(offset);

+ 2 - 0
src/main/java/com/baomidou/mybatisplus/spring/MybatisMapperRefresh.java

@@ -137,6 +137,7 @@ public class MybatisMapperRefresh implements Runnable {
         this.run();
     }
 
+    @Override
     public void run() {
         final GlobalConfiguration globalConfig = GlobalConfigUtils.getGlobalConfig(configuration);
         /*
@@ -147,6 +148,7 @@ public class MybatisMapperRefresh implements Runnable {
             final MybatisMapperRefresh runnable = this;
             new Thread(new Runnable() {
 
+                @Override
                 public void run() {
                     if (fileSet == null) {
                         fileSet = new HashSet<>();

+ 22 - 36
src/main/java/com/baomidou/mybatisplus/toolkit/StringEscape.java

@@ -11,56 +11,50 @@ package com.baomidou.mybatisplus.toolkit;
 public class StringEscape {
 
     /**
+     * <p>
      * 字符串是否需要转义
+     * </p>
      *
      * @param str
      * @param len
      * @return
      */
     private static boolean isEscapeNeededForString(String str, int len) {
-
         boolean needsHexEscape = false;
-
         for (int i = 0; i < len; ++i) {
             char c = str.charAt(i);
-
             switch (c) {
-                case 0: /* Must be escaped for 'mysql' */
-
+                /* Must be escaped for 'mysql' */
+                case 0:
                     needsHexEscape = true;
                     break;
-
-                case '\n': /* Must be escaped for logs */
+                /* Must be escaped for logs */
+                case '\n':
                     needsHexEscape = true;
-
                     break;
-
                 case '\r':
                     needsHexEscape = true;
                     break;
-
                 case '\\':
                     needsHexEscape = true;
-
                     break;
-
                 case '\'':
                     needsHexEscape = true;
-
                     break;
-
-                case '"': /* Better safe than sorry */
+                /* Better safe than sorry */
+                case '"':
                     needsHexEscape = true;
-
                     break;
-
-                case '\032': /* This gives problems on Win32 */
+                /* This gives problems on Win32 */
+                case '\032':
                     needsHexEscape = true;
                     break;
+                default:
+                    break;
             }
-
             if (needsHexEscape) {
-                break; // no need to scan more
+                // no need to scan more
+                break;
             }
         }
         return needsHexEscape;
@@ -73,34 +67,29 @@ public class StringEscape {
      * @return
      */
     public static String escapeString(String escapeStr) {
-
         if (escapeStr.matches("\'(.+)\'")) {
             escapeStr = escapeStr.substring(1, escapeStr.length() - 1);
         }
-
         String parameterAsString = escapeStr;
         int stringLength = escapeStr.length();
         if (isEscapeNeededForString(escapeStr, stringLength)) {
-
             StringBuilder buf = new StringBuilder((int) (escapeStr.length() * 1.1));
-
             //
             // Note: buf.append(char) is _faster_ than appending in blocks,
             // because the block append requires a System.arraycopy().... go
             // figure...
             //
-
             for (int i = 0; i < stringLength; ++i) {
                 char c = escapeStr.charAt(i);
-
                 switch (c) {
-                    case 0: /* Must be escaped for 'mysql' */
+                    /* Must be escaped for 'mysql' */
+                    case 0:
                         buf.append('\\');
                         buf.append('0');
 
                         break;
-
-                    case '\n': /* Must be escaped for logs */
+                    /* Must be escaped for logs */
+                    case '\n':
                         buf.append('\\');
                         buf.append('n');
 
@@ -123,24 +112,21 @@ public class StringEscape {
                         buf.append('\'');
 
                         break;
-
-                    case '"': /* Better safe than sorry */
+                    /* Better safe than sorry */
+                    case '"':
                         buf.append('\\');
                         buf.append('"');
 
                         break;
-
-                    case '\032': /* This gives problems on Win32 */
+                    /* This gives problems on Win32 */
+                    case '\032':
                         buf.append('\\');
                         buf.append('Z');
-
                         break;
-
                     default:
                         buf.append(c);
                 }
             }
-
             parameterAsString = buf.toString();
         }
         return "\'" + parameterAsString + "\'";

+ 4 - 4
src/main/java/com/baomidou/mybatisplus/toolkit/StringUtils.java

@@ -259,13 +259,13 @@ public class StringUtils {
     public static String quotaMarkList(Collection<?> coll) {
         StringBuilder sqlBuild = new StringBuilder();
         sqlBuild.append("(");
-        int _size = coll.size();
+        int size = coll.size();
         int i = 0;
         Iterator<?> iterator = coll.iterator();
         while (iterator.hasNext()) {
             String tempVal = StringUtils.quotaMark(iterator.next());
             sqlBuild.append(tempVal);
-            if (i + 1 < _size) {
+            if (i + 1 < size) {
                 sqlBuild.append(",");
             }
             i++;
@@ -715,7 +715,7 @@ public class StringUtils {
     }
 
     private static String wordsAndHyphenAndCamelToConstantCase(String input) {
-        boolean _betweenUpperCases = false;
+        boolean betweenUpperCases = false;
         boolean containsLowerCase = containsLowerCase(input);
 
         StringBuilder buf = new StringBuilder();
@@ -729,7 +729,7 @@ public class StringUtils {
             boolean previousIsWhitespace = Character.isWhitespace(previousChar);
             boolean lastOneIsNotUnderscore = (buf.length() > 0) && (buf.charAt(buf.length() - 1) != '_');
             boolean isNotUnderscore = c != '_';
-            if ((lastOneIsNotUnderscore) && ((isUpperCaseAndPreviousIsLowerCase) || (previousIsWhitespace) || ((_betweenUpperCases)
+            if ((lastOneIsNotUnderscore) && ((isUpperCaseAndPreviousIsLowerCase) || (previousIsWhitespace) || ((betweenUpperCases)
                     && (containsLowerCase) && (isUpperCaseAndPreviousIsUpperCase)))) {
                 buf.append("_");
             } else if (((separatorAfterDigit) && (Character.isDigit(previousChar))

+ 4 - 0
src/main/java/com/baomidou/mybatisplus/toolkit/SystemClock.java

@@ -64,6 +64,8 @@ public class SystemClock {
 
     private void scheduleClockUpdating() {
         ScheduledExecutorService scheduler = Executors.newSingleThreadScheduledExecutor(new ThreadFactory() {
+
+            @Override
             public Thread newThread(Runnable runnable) {
                 Thread thread = new Thread(runnable, "System Clock");
                 thread.setDaemon(true);
@@ -71,6 +73,8 @@ public class SystemClock {
             }
         });
         scheduler.scheduleAtFixedRate(new Runnable() {
+
+            @Override
             public void run() {
                 now.set(System.currentTimeMillis());
             }