Checkstyle 2509 (#6525)

* fix test checkstyle

* fix testcase

* fix testcase

* fix checkstyle

Author: wenshao

core/src/main/java/com/alibaba/druid/mock/handler/MySqlMockExecuteHandlerImpl.java

@@ -38,7 +38,7 @@ public class MySqlMockExecuteHandlerImpl implements MockExecuteHandler {
     @Override
     public ResultSet executeQuery(MockStatementBase statement, String sql) throws SQLException {
         SQLStatementParser parser = new MySqlStatementParser(sql);
-        List<SQLStatement> stmtList = parser.parseStatementList(); //
+        List<SQLStatement> stmtList = parser.parseStatementList();
 
         if (stmtList.size() > 1) {
             throw new SQLException("not support multi-statment. " + sql);

core/src/main/java/com/alibaba/druid/pool/DruidDataSource.java

@@ -2868,7 +2868,7 @@ public void run() {
                     if (timeBetweenEvictionRunsMillis > 0) {
                         Thread.sleep(timeBetweenEvictionRunsMillis);
                     } else {
-                        Thread.sleep(1000); //
+                        Thread.sleep(1000);
                     }
 
                     if (Thread.interrupted()) {

core/src/main/java/com/alibaba/druid/stat/TableStat.java

@@ -444,7 +444,7 @@ public static class Column {
         private boolean having;
         private boolean join;
         private boolean primaryKey; // for ddl
-        private boolean unique; //
+        private boolean unique;
         private boolean update;
         private Map<String, Object> attributes = new HashMap<String, Object>();
         private transient String fullName;
@@ -631,7 +631,7 @@ public static enum Mode {
         DropIndex(256), //
         CreateIndex(512), //
         Replace(1024),
-        DESC(2048); //
+        DESC(2048);
 
         public final int mark;
 

core/src/main/java/com/alibaba/druid/support/monitor/dao/MonitorDaoJdbcImpl.java

@@ -813,7 +813,7 @@ public void insertAppIfNotExits(String domain, String app) throws SQLException {
     public List<MonitorApp> listApp(String domain) throws SQLException {
         List<MonitorApp> list = new ArrayList<MonitorApp>();
 
-        String sql = "select id, domain, app from druid_app " //
+        String sql = "select id, domain, app from druid_app "
                 + " where domain = ?";
         Connection conn = null;
         PreparedStatement stmt = null;
@@ -838,7 +838,7 @@ public List<MonitorApp> listApp(String domain) throws SQLException {
     }
 
     public MonitorApp findApp(String domain, String app) throws SQLException {
-        String sql = "select id, domain, app from druid_app " //
+        String sql = "select id, domain, app from druid_app "
                 + " where domain = ? and app = ?";
         Connection conn = null;
         PreparedStatement stmt = null;
@@ -876,7 +876,7 @@ private MonitorApp readApp(ResultSet rs) throws SQLException {
     public List<MonitorCluster> listCluster(String domain, String app) throws SQLException {
         List<MonitorCluster> list = new ArrayList<MonitorCluster>();
 
-        String sql = "select id, domain, app, cluster from druid_cluster " //
+        String sql = "select id, domain, app, cluster from druid_cluster "
                 + " where domain = ?";
 
         if (app != null) {
@@ -919,7 +919,7 @@ public void insertClusterIfNotExits(String domain, String app, String cluster) t
     }
 
     public MonitorCluster findCluster(String domain, String app, String cluster) throws SQLException {
-        String sql = "select id, domain, app, cluster from druid_cluster " //
+        String sql = "select id, domain, app, cluster from druid_cluster "
                 + " where domain = ? and app = ? and cluster = ?";
         Connection conn = null;
         PreparedStatement stmt = null;
@@ -960,19 +960,19 @@ public void insertOrUpdateInstance(String domain, String app, String cluster, St
                                        Date startTime, long pid) throws SQLException {
         MonitorInstance monitorInst = findInst(domain, app, cluster, host);
         if (monitorInst == null) {
-            String sql = "insert into druid_inst (domain, app, cluster, host, ip, lastActiveTime, lastPID) " //
+            String sql = "insert into druid_inst (domain, app, cluster, host, ip, lastActiveTime, lastPID) "
                     + " values (?, ?, ?, ?, ?, ?, ?)";
             JdbcUtils.execute(dataSource, sql, domain, app, cluster, host, ip, startTime, pid);
         } else {
-            String sql = "update druid_inst set ip = ?, lastActiveTime = ?, lastPID = ? " //
+            String sql = "update druid_inst set ip = ?, lastActiveTime = ?, lastPID = ? "
                     + " where domain = ? and app = ? and cluster = ? and host = ? ";
             JdbcUtils.execute(dataSource, sql, ip, startTime, pid, domain, app, cluster, host);
         }
     }
 
     public MonitorInstance findInst(String domain, String app, String cluster, String host) throws SQLException {
-        String sql = "select id, domain, app, cluster, host, ip, lastActiveTime, lastPID from druid_inst " //
-                + " where domain = ? and app = ? and cluster = ? and host = ? " //
+        String sql = "select id, domain, app, cluster, host, ip, lastActiveTime, lastPID from druid_inst "
+                + " where domain = ? and app = ? and cluster = ? and host = ? "
                 + " limit 1";
         Connection conn = null;
         PreparedStatement stmt = null;
@@ -1002,7 +1002,7 @@ public MonitorInstance findInst(String domain, String app, String cluster, Strin
     public List<MonitorInstance> listInst(String domain, String app, String cluster) throws SQLException {
         List<MonitorInstance> list = new ArrayList<MonitorInstance>();
 
-        String sql = "select id, domain, app, cluster, host, ip, lastActiveTime, lastPID from druid_inst " //
+        String sql = "select id, domain, app, cluster, host, ip, lastActiveTime, lastPID from druid_inst "
                 + "where domain = ?";
 
         if (app != null) {

core/src/main/java/com/alibaba/druid/util/JdbcUtils.java

@@ -880,7 +880,7 @@ public static String makeInsertToTableSql(String tableName, Collection<String> n
         StringBuilder sql = new StringBuilder() //
                 .append("insert into ") //
                 .append(tableName) //
-                .append("("); //
+                .append("(");
 
         int nameCount = 0;
         for (String name : names) {

core/src/test/java/com/alibaba/druid/Bug_for_alibank.java

@@ -15,10 +15,8 @@
  */
 package com.alibaba.druid;
 
-import junit.framework.TestCase;
-
 import com.alibaba.druid.support.json.JSONUtils;
-
+import junit.framework.TestCase;
 
 public class Bug_for_alibank extends TestCase {
     public void test_bug() throws Exception {

core/src/test/java/com/alibaba/druid/DBCPTest.java

@@ -15,14 +15,12 @@
  */
 package com.alibaba.druid;
 
-import java.sql.Connection;
-import java.util.concurrent.CountDownLatch;
-
+import com.alibaba.druid.mock.MockDriver;
 import junit.framework.TestCase;
-
 import org.apache.commons.dbcp.BasicDataSource;
 
-import com.alibaba.druid.mock.MockDriver;
+import java.sql.Connection;
+import java.util.concurrent.CountDownLatch;
 
 public class DBCPTest extends TestCase {
     public void test_max() throws Exception {

core/src/test/java/com/alibaba/druid/DragoonMetaInfTest.java

@@ -15,6 +15,8 @@
  */
 package com.alibaba.druid;
 
+import junit.framework.TestCase;
+
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.Reader;
@@ -23,8 +25,6 @@
 import java.util.Enumeration;
 import java.util.Properties;
 
-import junit.framework.TestCase;
-
 public class DragoonMetaInfTest extends TestCase {
     public void test_0() throws Exception {
         ClassLoader classLoader = Thread.currentThread().getContextClassLoader();

core/src/test/java/com/alibaba/druid/LockTest.java

@@ -15,11 +15,11 @@
  */
 package com.alibaba.druid;
 
+import junit.framework.TestCase;
+
 import java.util.concurrent.locks.Lock;
 import java.util.concurrent.locks.ReentrantLock;
 
-import junit.framework.TestCase;
-
 public class LockTest extends TestCase {
     public void test_0() throws Exception {
         Lock lock = new ReentrantLock();

core/src/test/java/com/alibaba/druid/TestForZY.java

@@ -18,7 +18,6 @@
 import java.util.HashMap;
 import java.util.Map;
 
-
 public class TestForZY {
     public void foo() throws Exception {
         Map<String, Map.Entry<String, Boolean>> m = new HashMap<String, Map.Entry<String, Boolean>>();