From dfcfffbc671633c9790256feb8e0d335b0284548 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E6=B8=A9=E5=B0=91?= Date: Fri, 18 May 2012 10:46:31 +0800 Subject: [PATCH] refactor --- .../ast/OracleHint.java => ast/SQLCommentHint.java} | 10 ++++------ .../sql/dialect/mysql/parser/MySqlExprParser.java | 4 ++-- .../sql/dialect/oracle/visitor/OracleASTVisitor.java | 2 +- .../dialect/oracle/visitor/OracleOutputVisitor.java | 4 ++-- .../com/alibaba/druid/sql/parser/SQLExprParser.java | 4 ++-- .../alibaba/druid/sql/visitor/SQLASTOutputVisitor.java | 4 ++-- .../com/alibaba/druid/sql/visitor/SQLASTVisitor.java | 6 +++--- .../druid/sql/visitor/SQLASTVisitorAdapter.java | 6 +++--- 8 files changed, 19 insertions(+), 21 deletions(-) rename src/main/java/com/alibaba/druid/sql/{dialect/oracle/ast/OracleHint.java => ast/SQLCommentHint.java} (76%) diff --git a/src/main/java/com/alibaba/druid/sql/dialect/oracle/ast/OracleHint.java b/src/main/java/com/alibaba/druid/sql/ast/SQLCommentHint.java similarity index 76% rename from src/main/java/com/alibaba/druid/sql/dialect/oracle/ast/OracleHint.java rename to src/main/java/com/alibaba/druid/sql/ast/SQLCommentHint.java index 442b8864e4..473ecdeac7 100644 --- a/src/main/java/com/alibaba/druid/sql/dialect/oracle/ast/OracleHint.java +++ b/src/main/java/com/alibaba/druid/sql/ast/SQLCommentHint.java @@ -13,23 +13,21 @@ * See the License for the specific language governing permissions and * limitations under the License. */ -package com.alibaba.druid.sql.dialect.oracle.ast; +package com.alibaba.druid.sql.ast; -import com.alibaba.druid.sql.ast.SQLHint; -import com.alibaba.druid.sql.ast.SQLObjectImpl; import com.alibaba.druid.sql.visitor.SQLASTVisitor; -public class OracleHint extends SQLObjectImpl implements SQLHint { +public class SQLCommentHint extends SQLObjectImpl implements SQLHint { private static final long serialVersionUID = 1L; private String text; - public OracleHint(){ + public SQLCommentHint(){ } - public OracleHint(String text){ + public SQLCommentHint(String text){ this.text = text; } diff --git a/src/main/java/com/alibaba/druid/sql/dialect/mysql/parser/MySqlExprParser.java b/src/main/java/com/alibaba/druid/sql/dialect/mysql/parser/MySqlExprParser.java index 1f170f2684..501a568d07 100644 --- a/src/main/java/com/alibaba/druid/sql/dialect/mysql/parser/MySqlExprParser.java +++ b/src/main/java/com/alibaba/druid/sql/dialect/mysql/parser/MySqlExprParser.java @@ -17,6 +17,7 @@ package com.alibaba.druid.sql.dialect.mysql.parser; import java.util.List; +import com.alibaba.druid.sql.ast.SQLCommentHint; import com.alibaba.druid.sql.ast.SQLExpr; import com.alibaba.druid.sql.ast.SQLHint; import com.alibaba.druid.sql.ast.SQLName; @@ -43,7 +44,6 @@ import com.alibaba.druid.sql.dialect.mysql.ast.expr.MySqlUserName; import com.alibaba.druid.sql.dialect.mysql.ast.statement.MySqlSQLColumnDefinition; import com.alibaba.druid.sql.dialect.mysql.ast.statement.MySqlSelectQueryBlock; import com.alibaba.druid.sql.dialect.mysql.ast.statement.MySqlSelectQueryBlock.Limit; -import com.alibaba.druid.sql.dialect.oracle.ast.OracleHint; import com.alibaba.druid.sql.parser.Lexer; import com.alibaba.druid.sql.parser.ParserException; import com.alibaba.druid.sql.parser.SQLExprParser; @@ -613,7 +613,7 @@ public class MySqlExprParser extends SQLExprParser { public void parseHints(List hints) { if (lexer.token() == Token.HINT) { - hints.add(new OracleHint(lexer.stringVal())); + hints.add(new SQLCommentHint(lexer.stringVal())); lexer.nextToken(); } } diff --git a/src/main/java/com/alibaba/druid/sql/dialect/oracle/visitor/OracleASTVisitor.java b/src/main/java/com/alibaba/druid/sql/dialect/oracle/visitor/OracleASTVisitor.java index 0b4b9cbc86..06ea421419 100644 --- a/src/main/java/com/alibaba/druid/sql/dialect/oracle/visitor/OracleASTVisitor.java +++ b/src/main/java/com/alibaba/druid/sql/dialect/oracle/visitor/OracleASTVisitor.java @@ -15,8 +15,8 @@ */ package com.alibaba.druid.sql.dialect.oracle.visitor; +import com.alibaba.druid.sql.ast.SQLCommentHint; import com.alibaba.druid.sql.ast.expr.SQLObjectCreateExpr; -import com.alibaba.druid.sql.dialect.oracle.ast.OracleHint; import com.alibaba.druid.sql.dialect.oracle.ast.OracleOrderBy; import com.alibaba.druid.sql.dialect.oracle.ast.clause.CycleClause; import com.alibaba.druid.sql.dialect.oracle.ast.clause.FlashbackQueryClause.AsOfFlashbackQueryClause; diff --git a/src/main/java/com/alibaba/druid/sql/dialect/oracle/visitor/OracleOutputVisitor.java b/src/main/java/com/alibaba/druid/sql/dialect/oracle/visitor/OracleOutputVisitor.java index b3f57149d8..0391cded3e 100644 --- a/src/main/java/com/alibaba/druid/sql/dialect/oracle/visitor/OracleOutputVisitor.java +++ b/src/main/java/com/alibaba/druid/sql/dialect/oracle/visitor/OracleOutputVisitor.java @@ -17,6 +17,7 @@ package com.alibaba.druid.sql.dialect.oracle.visitor; import java.util.List; +import com.alibaba.druid.sql.ast.SQLCommentHint; import com.alibaba.druid.sql.ast.SQLDataType; import com.alibaba.druid.sql.ast.SQLExpr; import com.alibaba.druid.sql.ast.SQLHint; @@ -38,7 +39,6 @@ import com.alibaba.druid.sql.ast.statement.SQLJoinTableSource.JoinType; import com.alibaba.druid.sql.ast.statement.SQLRollbackStatement; import com.alibaba.druid.sql.ast.statement.SQLSelect; import com.alibaba.druid.sql.ast.statement.SQLSelectQueryBlock; -import com.alibaba.druid.sql.dialect.oracle.ast.OracleHint; import com.alibaba.druid.sql.dialect.oracle.ast.OracleOrderBy; import com.alibaba.druid.sql.dialect.oracle.ast.clause.CycleClause; import com.alibaba.druid.sql.dialect.oracle.ast.clause.FlashbackQueryClause.AsOfFlashbackQueryClause; @@ -907,7 +907,7 @@ public class OracleOutputVisitor extends SQLASTOutputVisitor implements OracleAS } @Override - public void endVisit(OracleHint x) { + public void endVisit(SQLCommentHint x) { } diff --git a/src/main/java/com/alibaba/druid/sql/parser/SQLExprParser.java b/src/main/java/com/alibaba/druid/sql/parser/SQLExprParser.java index 6c1605953f..6f49c1df02 100644 --- a/src/main/java/com/alibaba/druid/sql/parser/SQLExprParser.java +++ b/src/main/java/com/alibaba/druid/sql/parser/SQLExprParser.java @@ -19,6 +19,7 @@ import java.math.BigInteger; import java.util.Collection; import java.util.List; +import com.alibaba.druid.sql.ast.SQLCommentHint; import com.alibaba.druid.sql.ast.SQLDataType; import com.alibaba.druid.sql.ast.SQLDataTypeImpl; import com.alibaba.druid.sql.ast.SQLExpr; @@ -63,7 +64,6 @@ import com.alibaba.druid.sql.ast.statement.SQLColumnDefinition; import com.alibaba.druid.sql.ast.statement.SQLPrimaryKey; import com.alibaba.druid.sql.ast.statement.SQLSelect; import com.alibaba.druid.sql.ast.statement.SQLSelectOrderByItem; -import com.alibaba.druid.sql.dialect.oracle.ast.OracleHint; public class SQLExprParser extends SQLParser { @@ -1201,7 +1201,7 @@ public class SQLExprParser extends SQLParser { public void parseHints(List hints) { if (lexer.token() == Token.HINT) { - hints.add(new OracleHint(lexer.stringVal())); + hints.add(new SQLCommentHint(lexer.stringVal())); lexer.nextToken(); } } diff --git a/src/main/java/com/alibaba/druid/sql/visitor/SQLASTOutputVisitor.java b/src/main/java/com/alibaba/druid/sql/visitor/SQLASTOutputVisitor.java index 56d0078022..62eff614b3 100644 --- a/src/main/java/com/alibaba/druid/sql/visitor/SQLASTOutputVisitor.java +++ b/src/main/java/com/alibaba/druid/sql/visitor/SQLASTOutputVisitor.java @@ -18,6 +18,7 @@ package com.alibaba.druid.sql.visitor; import java.io.IOException; import java.util.List; +import com.alibaba.druid.sql.ast.SQLCommentHint; import com.alibaba.druid.sql.ast.SQLDataType; import com.alibaba.druid.sql.ast.SQLExpr; import com.alibaba.druid.sql.ast.SQLObject; @@ -90,7 +91,6 @@ import com.alibaba.druid.sql.ast.statement.SQLUniqueConstraint; import com.alibaba.druid.sql.ast.statement.SQLUpdateSetItem; import com.alibaba.druid.sql.ast.statement.SQLUpdateStatement; import com.alibaba.druid.sql.ast.statement.SQLUseStatement; -import com.alibaba.druid.sql.dialect.oracle.ast.OracleHint; public class SQLASTOutputVisitor extends SQLASTVisitorAdapter { @@ -1091,7 +1091,7 @@ public class SQLASTOutputVisitor extends SQLASTVisitorAdapter { return false; } - public boolean visit(OracleHint x) { + public boolean visit(SQLCommentHint x) { print(x.getText()); return false; } diff --git a/src/main/java/com/alibaba/druid/sql/visitor/SQLASTVisitor.java b/src/main/java/com/alibaba/druid/sql/visitor/SQLASTVisitor.java index 211a656be5..e5dfec2937 100644 --- a/src/main/java/com/alibaba/druid/sql/visitor/SQLASTVisitor.java +++ b/src/main/java/com/alibaba/druid/sql/visitor/SQLASTVisitor.java @@ -15,6 +15,7 @@ */ package com.alibaba.druid.sql.visitor; +import com.alibaba.druid.sql.ast.SQLCommentHint; import com.alibaba.druid.sql.ast.SQLDataType; import com.alibaba.druid.sql.ast.SQLObject; import com.alibaba.druid.sql.ast.SQLOrderBy; @@ -84,7 +85,6 @@ import com.alibaba.druid.sql.ast.statement.SQLUniqueConstraint; import com.alibaba.druid.sql.ast.statement.SQLUpdateSetItem; import com.alibaba.druid.sql.ast.statement.SQLUpdateStatement; import com.alibaba.druid.sql.ast.statement.SQLUseStatement; -import com.alibaba.druid.sql.dialect.oracle.ast.OracleHint; public interface SQLASTVisitor { @@ -372,7 +372,7 @@ public interface SQLASTVisitor { void endVisit(SQLReleaseSavePointStatement x); - void endVisit(OracleHint x); + void endVisit(SQLCommentHint x); - boolean visit(OracleHint x); + boolean visit(SQLCommentHint x); } diff --git a/src/main/java/com/alibaba/druid/sql/visitor/SQLASTVisitorAdapter.java b/src/main/java/com/alibaba/druid/sql/visitor/SQLASTVisitorAdapter.java index 2525d8832c..e20060cc09 100644 --- a/src/main/java/com/alibaba/druid/sql/visitor/SQLASTVisitorAdapter.java +++ b/src/main/java/com/alibaba/druid/sql/visitor/SQLASTVisitorAdapter.java @@ -15,6 +15,7 @@ */ package com.alibaba.druid.sql.visitor; +import com.alibaba.druid.sql.ast.SQLCommentHint; import com.alibaba.druid.sql.ast.SQLDataType; import com.alibaba.druid.sql.ast.SQLObject; import com.alibaba.druid.sql.ast.SQLOrderBy; @@ -85,7 +86,6 @@ import com.alibaba.druid.sql.ast.statement.SQLUniqueConstraint; import com.alibaba.druid.sql.ast.statement.SQLUpdateSetItem; import com.alibaba.druid.sql.ast.statement.SQLUpdateStatement; import com.alibaba.druid.sql.ast.statement.SQLUseStatement; -import com.alibaba.druid.sql.dialect.oracle.ast.OracleHint; public class SQLASTVisitorAdapter implements SQLASTVisitor { @@ -663,12 +663,12 @@ public class SQLASTVisitorAdapter implements SQLASTVisitor { } @Override - public boolean visit(OracleHint x) { + public boolean visit(SQLCommentHint x) { return true; } @Override - public void endVisit(OracleHint x) { + public void endVisit(SQLCommentHint x) { } }