split RuleForbiddenSyntax into 5 rules.

9e94b63a · liipx · 39559094 · 9e94b63a · 9e94b63a · 9e94b63a
6 changed file
--- a/advisor/heuristic.go
+++ b/advisor/heuristic.go
@@ -1768,6 +1768,81 @@ func (q *Query4Audit) RuleSumNPE() Rule {
 	return rule
 }

+// RuleForbiddenTrigger FUN.007
+func (q *Query4Audit) RuleForbiddenTrigger() Rule {
+	var rule = q.RuleOK()
+
+	// 由于vitess对某些语法的支持不完善，使得如创建临时表等语句无法通过语法检查
+	// 所以这里使用正则对触发器、临时表、存储过程等进行匹配
+	// 但是目前支持的也不是非常全面，有待完善匹配规则
+	// TODO TiDB 目前还不支持触发器、存储过程、自定义函数、外键
+
+	forbidden := []*regexp.Regexp{
+		regexp.MustCompile(`(?i)CREATE\s+TRIGGER\s+`),
+	}
+
+	for _, reg := range forbidden {
+		if reg.MatchString(q.Query) {
+			rule = HeuristicRules["FUN.007"]
+			if position := reg.FindIndex([]byte(q.Query)); len(position) > 0 {
+				rule.Position = position[0]
+			}
+			break
+		}
+	}
+	return rule
+}
+
+// RuleForbiddenProcedure FUN.008
+func (q *Query4Audit) RuleForbiddenProcedure() Rule {
+	var rule = q.RuleOK()
+
+	// 由于vitess对某些语法的支持不完善，使得如创建临时表等语句无法通过语法检查
+	// 所以这里使用正则对触发器、临时表、存储过程等进行匹配
+	// 但是目前支持的也不是非常全面，有待完善匹配规则
+	// TODO TiDB 目前还不支持触发器、存储过程、自定义函数、外键
+
+	forbidden := []*regexp.Regexp{
+		regexp.MustCompile(`(?i)CREATE\s+PROCEDURE\s+`),
+	}
+
+	for _, reg := range forbidden {
+		if reg.MatchString(q.Query) {
+			rule = HeuristicRules["FUN.008"]
+			if position := reg.FindIndex([]byte(q.Query)); len(position) > 0 {
+				rule.Position = position[0]
+			}
+			break
+		}
+	}
+	return rule
+}
+
+// RuleForbiddenFunction FUN.009
+func (q *Query4Audit) RuleForbiddenFunction() Rule {
+	var rule = q.RuleOK()
+
+	// 由于vitess对某些语法的支持不完善，使得如创建临时表等语句无法通过语法检查
+	// 所以这里使用正则对触发器、临时表、存储过程等进行匹配
+	// 但是目前支持的也不是非常全面，有待完善匹配规则
+	// TODO TiDB 目前还不支持触发器、存储过程、自定义函数、外键
+
+	forbidden := []*regexp.Regexp{
+		regexp.MustCompile(`(?i)CREATE\s+FUNCTION\s+`),
+	}
+
+	for _, reg := range forbidden {
+		if reg.MatchString(q.Query) {
+			rule = HeuristicRules["FUN.009"]
+			if position := reg.FindIndex([]byte(q.Query)); len(position) > 0 {
+				rule.Position = position[0]
+			}
+			break
+		}
+	}
+	return rule
+}
+
 // RulePatternMatchingUsage ARG.007
 func (q *Query4Audit) RulePatternMatchingUsage() Rule {
 	var rule = q.RuleOK()
@@ -1932,39 +2007,6 @@ func (idxAdv *IndexAdvisor) RuleUpdatePrimaryKey() Rule {
 	return rule
 }

-// RuleForbiddenSyntax CLA.017
-func (q *Query4Audit) RuleForbiddenSyntax() Rule {
-	var rule = q.RuleOK()
-
-	// 由于vitess对某些语法的支持不完善，使得如创建临时表等语句无法通过语法检查
-	// 所以这里使用正则对触发器、临时表、存储过程等进行匹配
-	// 但是目前支持的也不是非常全面，有待完善匹配规则
-	// TODO TiDB 目前还不支持触发器、存储过程、自定义函数、外键
-
-	forbidden := []*regexp.Regexp{
-		regexp.MustCompile(`(?i)CREATE\s+TRIGGER\s+`),
-
-		regexp.MustCompile(`(?i)CREATE\s+TEMPORARY\s+TABLE\s+`),
-
-		regexp.MustCompile(`(?i)CREATE\s+VIEW\s+`),
-		regexp.MustCompile(`(?i)REPLACE\s+VIEW\s+`),
-
-		regexp.MustCompile(`(?i)CREATE\s+PROCEDURE\s+`),
-		regexp.MustCompile(`(?i)CREATE\s+FUNCTION\s+`),
-	}
-
-	for _, reg := range forbidden {
-		if reg.MatchString(q.Query) {
-			rule = HeuristicRules["CLA.017"]
-			if position := reg.FindIndex([]byte(q.Query)); len(position) > 0 {
-				rule.Position = position[0]
-			}
-			break
-		}
-	}
-	return rule
-}
-
 // RuleNestedSubQueries JOI.006
 func (q *Query4Audit) RuleNestedSubQueries() Rule {
 	var rule = q.RuleOK()
@@ -2811,6 +2853,57 @@ func (q *Query4Audit) RuleTableCharsetCheck() Rule {
 	return rule
 }

+// RuleForbiddenView TBL.006
+func (q *Query4Audit) RuleForbiddenView() Rule {
+	var rule = q.RuleOK()
+
+	// 由于vitess对某些语法的支持不完善，使得如创建临时表等语句无法通过语法检查
+	// 所以这里使用正则对触发器、临时表、存储过程等进行匹配
+	// 但是目前支持的也不是非常全面，有待完善匹配规则
+	// TODO TiDB 目前还不支持触发器、存储过程、自定义函数、外键
+
+	forbidden := []*regexp.Regexp{
+		regexp.MustCompile(`(?i)CREATE\s+VIEW\s+`),
+		regexp.MustCompile(`(?i)REPLACE\s+VIEW\s+`),
+	}
+
+	for _, reg := range forbidden {
+		if reg.MatchString(q.Query) {
+			rule = HeuristicRules["TBL.006"]
+			if position := reg.FindIndex([]byte(q.Query)); len(position) > 0 {
+				rule.Position = position[0]
+			}
+			break
+		}
+	}
+	return rule
+}
+
+// RuleForbiddenTempTable TBL.007
+func (q *Query4Audit) RuleForbiddenTempTable() Rule {
+	var rule = q.RuleOK()
+
+	// 由于vitess对某些语法的支持不完善，使得如创建临时表等语句无法通过语法检查
+	// 所以这里使用正则对触发器、临时表、存储过程等进行匹配
+	// 但是目前支持的也不是非常全面，有待完善匹配规则
+	// TODO TiDB 目前还不支持触发器、存储过程、自定义函数、外键
+
+	forbidden := []*regexp.Regexp{
+		regexp.MustCompile(`(?i)CREATE\s+TEMPORARY\s+TABLE\s+`),
+	}
+
+	for _, reg := range forbidden {
+		if reg.MatchString(q.Query) {
+			rule = HeuristicRules["TBL.007"]
+			if position := reg.FindIndex([]byte(q.Query)); len(position) > 0 {
+				rule.Position = position[0]
+			}
+			break
+		}
+	}
+	return rule
+}
+
 // RuleBlobDefaultValue COL.015
 func (q *Query4Audit) RuleBlobDefaultValue() Rule {
 	var rule = q.RuleOK()

--- a/advisor/heuristic_test.go
+++ b/advisor/heuristic_test.go
@@ -1637,20 +1637,86 @@ func TestRuleHavingClause(t *testing.T) {
 	common.Log.Debug("Exiting function: %s", common.GetFunctionName())
 }

-// CLA.017
-func TestRuleForbiddenSyntax(t *testing.T) {
+// FUN.007
+func TestRuleForbiddenTrigger(t *testing.T) {
+	common.Log.Debug("Entering function: %s", common.GetFunctionName())
+	sqls := []string{
+		`CREATE TRIGGER t1 AFTER INSERT ON work FOR EACH ROW INSERT INTO time VALUES(NOW());`,
+	}
+	for _, sql := range sqls {
+		q, _ := NewQuery4Audit(sql)
+		rule := q.RuleForbiddenTrigger()
+		if rule.Item != "FUN.007" {
+			t.Error("Rule not match:", rule.Item, "Expect : FUN.007")
+		}
+
+	}
+	common.Log.Debug("Exiting function: %s", common.GetFunctionName())
+}
+
+// FUN.008
+func TestRuleForbiddenProcedure(t *testing.T) {
+	common.Log.Debug("Entering function: %s", common.GetFunctionName())
+	sqls := []string{
+		`CREATE PROCEDURE simpleproc (OUT param1 INT)`,
+	}
+	for _, sql := range sqls {
+		q, _ := NewQuery4Audit(sql)
+		rule := q.RuleForbiddenProcedure()
+		if rule.Item != "FUN.008" {
+			t.Error("Rule not match:", rule.Item, "Expect : FUN.008")
+		}
+
+	}
+	common.Log.Debug("Exiting function: %s", common.GetFunctionName())
+}
+
+// FUN.009
+func TestRuleForbiddenFunction(t *testing.T) {
+	common.Log.Debug("Entering function: %s", common.GetFunctionName())
+	sqls := []string{
+		`CREATE FUNCTION hello (s CHAR(20));`,
+	}
+	for _, sql := range sqls {
+		q, _ := NewQuery4Audit(sql)
+		rule := q.RuleForbiddenFunction()
+		if rule.Item != "FUN.009" {
+			t.Error("Rule not match:", rule.Item, "Expect : FUN.009")
+		}
+
+	}
+	common.Log.Debug("Exiting function: %s", common.GetFunctionName())
+}
+
+// TBL.006
+func TestRuleForbiddenView(t *testing.T) {
 	common.Log.Debug("Entering function: %s", common.GetFunctionName())
 	sqls := []string{
 		`create view v_today (today) AS SELECT CURRENT_DATE;`,
 		`CREATE VIEW v (col) AS SELECT 'abc';`,
-		`CREATE FUNCTION hello (s CHAR(20));`,
-		`CREATE PROCEDURE simpleproc (OUT param1 INT)`,
 	}
 	for _, sql := range sqls {
 		q, _ := NewQuery4Audit(sql)
-		rule := q.RuleForbiddenSyntax()
-		if rule.Item != "CLA.017" {
-			t.Error("Rule not match:", rule.Item, "Expect : CLA.017")
+		rule := q.RuleForbiddenView()
+		if rule.Item != "TBL.006" {
+			t.Error("Rule not match:", rule.Item, "Expect : TBL.006")
+		}
+
+	}
+	common.Log.Debug("Exiting function: %s", common.GetFunctionName())
+}
+
+// TBL.007
+func TestRuleForbiddenTempTable(t *testing.T) {
+	common.Log.Debug("Entering function: %s", common.GetFunctionName())
+	sqls := []string{
+		"CREATE TEMPORARY TABLE `work` (`time` time DEFAULT NULL) ENGINE=InnoDB;",
+	}
+	for _, sql := range sqls {
+		q, _ := NewQuery4Audit(sql)
+		rule := q.RuleForbiddenTempTable()
+		if rule.Item != "TBL.007" {
+			t.Error("Rule not match:", rule.Item, "Expect : TBL.007")
 		}

 	}

--- a/advisor/rules.go
+++ b/advisor/rules.go
@@ -416,14 +416,6 @@ func init() {
 			Case:     "update tbl set col=1",
 			Func:     (*Query4Audit).RuleOK, // 该建议在indexAdvisor中给
 		},
-		"CLA.017": {
-			Item:     "CLA.017",
-			Severity: "L2",
-			Summary:  "不建议使用存储过程、视图、触发器、临时表等",
-			Content:  `这些功能的使用在一定程度上会使得程序难以调试和拓展，更没有移植性，且会极大的增加出现 BUG 的概率。`,
-			Case:     "CREATE VIEW v_today (today) AS SELECT CURRENT_DATE;",
-			Func:     (*Query4Audit).RuleForbiddenSyntax,
-		},
 		"COL.001": {
 			Item:     "COL.001",
 			Severity: "L1",
@@ -627,6 +619,30 @@ func init() {
 			Case:     "SELECT SUM(COL) FROM tbl;",
 			Func:     (*Query4Audit).RuleSumNPE,
 		},
+		"FUN.007": {
+			Item:     "FUN.007",
+			Severity: "L1",
+			Summary:  "不建议使用触发器",
+			Content:  ``,
+			Case:     "",
+			Func:     (*Query4Audit).RuleForbiddenTrigger,
+		},
+		"FUN.008": {
+			Item:     "FUN.008",
+			Severity: "L1",
+			Summary:  "不建议使用存储过程",
+			Content:  ``,
+			Case:     "",
+			Func:     (*Query4Audit).RuleForbiddenProcedure,
+		},
+		"FUN.009": {
+			Item:     "FUN.009",
+			Severity: "L1",
+			Summary:  "不建议使用自定义函数",
+			Content:  ``,
+			Case:     "",
+			Func:     (*Query4Audit).RuleForbiddenFunction,
+		},
 		"GRP.001": {
 			Item:     "GRP.001",
 			Severity: "L2",
@@ -1065,6 +1081,22 @@ func init() {
 			Case:     "CREATE TABLE tbl (a int) DEFAULT CHARSET = latin1;",
 			Func:     (*Query4Audit).RuleTableCharsetCheck,
 		},
+		"TBL.006": {
+			Item:     "TBL.006",
+			Severity: "L1",
+			Summary:  "不建议使用视图",
+			Content:  ``,
+			Case:     "",
+			Func:     (*Query4Audit).RuleForbiddenView,
+		},
+		"TBL.007": {
+			Item:     "TBL.007",
+			Severity: "L1",
+			Summary:  "不建议使用临时表",
+			Content:  ``,
+			Case:     "",
+			Func:     (*Query4Audit).RuleForbiddenTempTable,
+		},
 	}
 }


--- a/advisor/testdata/TestListHeuristicRules.golden
+++ b/advisor/testdata/TestListHeuristicRules.golden
@@ -342,16 +342,6 @@ update tbl set col=1
 ```sql
 update tbl set col=1
 ```
-## 不建议使用存储过程、视图、触发器、临时表等
-
-* **Item**:CLA.017
-* **Severity**:L2
-* **Content**:这些功能的使用在一定程度上会使得程序难以调试和拓展，更没有移植性，且会极大的增加出现 BUG 的概率。
-* **Case**:
-
-```sql
-CREATE VIEW v_today (today) AS SELECT CURRENT_DATE;
-```
 ## 不建议使用 SELECT \* 类型查询

 * **Item**:COL.001
@@ -601,6 +591,36 @@ SELECT COUNT(1) FROM tbl;

 ```sql
 SELECT SUM(COL) FROM tbl;
+```
+## 不建议使用触发器
+
+* **Item**:FUN.007
+* **Severity**:L1
+* **Content**:
+* **Case**:
+
+```sql
+
+```
+## 不建议使用存储过程
+
+* **Item**:FUN.008
+* **Severity**:L1
+* **Content**:
+* **Case**:
+
+```sql
+
+```
+## 不建议使用自定义函数
+
+* **Item**:FUN.009
+* **Severity**:L1
+* **Content**:
+* **Case**:
+
+```sql
+
 ```
 ## 不建议对等值查询列使用 GROUP BY

@@ -1142,3 +1162,23 @@ CREATE TABLE tbl (a int) AUTO_INCREMENT = 10;
 ```sql
 CREATE TABLE tbl (a int) DEFAULT CHARSET = latin1;
 ```
+## 不建议使用视图
+
+* **Item**:TBL.006
+* **Severity**:L1
+* **Content**:
+* **Case**:
+
+```sql
+
+```
+## 不建议使用临时表
+
+* **Item**:TBL.007
+* **Severity**:L1
+* **Content**:
+* **Case**:
+
+```sql
+
+```
--- a/advisor/testdata/TestMergeConflictHeuristicRules.golden
+++ b/advisor/testdata/TestMergeConflictHeuristicRules.golden
@@ -31,7 +31,6 @@ advisor.Rule{Item:"CLA.013", Severity:"L3", Summary:"不建议使用 HAVING 子
 advisor.Rule{Item:"CLA.014", Severity:"L2", Summary:"删除全表时建议使用 TRUNCATE 替代 DELETE", Content:"删除全表时建议使用 TRUNCATE 替代 DELETE", Case:"delete from tbl", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
 advisor.Rule{Item:"CLA.015", Severity:"L4", Summary:"UPDATE 未指定 WHERE 条件", Content:"UPDATE 不指定 WHERE 条件一般是致命的，请您三思后行", Case:"update tbl set col=1", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
 advisor.Rule{Item:"CLA.016", Severity:"L2", Summary:"不要 UPDATE 主键", Content:"主键是数据表中记录的唯一标识符，不建议频繁更新主键列，这将影响元数据统计信息进而影响正常的查询。", Case:"update tbl set col=1", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
-advisor.Rule{Item:"CLA.017", Severity:"L2", Summary:"不建议使用存储过程、视图、触发器、临时表等", Content:"这些功能的使用在一定程度上会使得程序难以调试和拓展，更没有移植性，且会极大的增加出现 BUG 的概率。", Case:"CREATE VIEW v_today (today) AS SELECT CURRENT_DATE;", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
 advisor.Rule{Item:"COL.001", Severity:"L1", Summary:"不建议使用 SELECT * 类型查询", Content:"当表结构变更时，使用 * 通配符选择所有列将导致查询的含义和行为会发生更改，可能导致查询返回更多的数据。", Case:"select * from tbl where id=1", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
 advisor.Rule{Item:"COL.002", Severity:"L2", Summary:"INSERT/REPLACE 未指定列名", Content:"当表结构发生变更，如果 INSERT 或 REPLACE 请求不明确指定列名，请求的结果将会与预想的不同; 建议使用 “INSERT INTO tbl(col1，col2)VALUES ...” 代替。", Case:"insert into tbl values(1,'name')", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
 advisor.Rule{Item:"COL.003", Severity:"L2", Summary:"建议修改自增 ID 为无符号类型", Content:"建议修改自增 ID 为无符号类型", Case:"create table test(`id` int(11) NOT NULL AUTO_INCREMENT)", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
@@ -57,6 +56,9 @@ advisor.Rule{Item:"FUN.003", Severity:"L3", Summary:"使用了合并为可空列
 advisor.Rule{Item:"FUN.004", Severity:"L4", Summary:"不建议使用 SYSDATE() 函数", Content:"SYSDATE() 函数可能导致主从数据不一致，请使用 NOW() 函数替代 SYSDATE()。", Case:"SELECT SYSDATE();", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
 advisor.Rule{Item:"FUN.005", Severity:"L1", Summary:"不建议使用 COUNT(col) 或 COUNT(常量)", Content:"不要使用 COUNT(col) 或 COUNT(常量) 来替代 COUNT(*), COUNT(*) 是 SQL92 定义的标准统计行数的方法，跟数据无关，跟 NULL 和非 NULL 也无关。", Case:"SELECT COUNT(1) FROM tbl;", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
 advisor.Rule{Item:"FUN.006", Severity:"L1", Summary:"使用 SUM(COL) 时需注意 NPE 问题", Content:"当某一列的值全是 NULL 时，COUNT(COL) 的返回结果为0,但 SUM(COL) 的返回结果为 NULL，因此使用 SUM() 时需注意 NPE 问题。可以使用如下方式来避免 SUM 的 NPE 问题: SELECT IF(ISNULL(SUM(COL)), 0, SUM(COL)) FROM tbl", Case:"SELECT SUM(COL) FROM tbl;", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
+advisor.Rule{Item:"FUN.007", Severity:"L1", Summary:"不建议使用触发器", Content:"", Case:"", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
+advisor.Rule{Item:"FUN.008", Severity:"L1", Summary:"不建议使用存储过程", Content:"", Case:"", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
+advisor.Rule{Item:"FUN.009", Severity:"L1", Summary:"不建议使用自定义函数", Content:"", Case:"", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
 advisor.Rule{Item:"GRP.001", Severity:"L2", Summary:"不建议对等值查询列使用 GROUP BY", Content:"GROUP BY 中的列在前面的 WHERE 条件中使用了等值查询，对这样的列进行 GROUP BY 意义不大。", Case:"select film_id, title from film where release_year='2006' group by release_year", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
 advisor.Rule{Item:"JOI.001", Severity:"L2", Summary:"JOIN 语句混用逗号和 ANSI 模式", Content:"表连接的时候混用逗号和 ANSI JOIN 不便于人类理解，并且MySQL不同版本的表连接行为和优先级均有所不同，当 MySQL 版本变化后可能会引入错误。", Case:"select c1,c2,c3 from t1,t2 join t3 on t1.c1=t2.c1,t1.c3=t3,c1 where id>1000", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
 advisor.Rule{Item:"JOI.002", Severity:"L4", Summary:"同一张表被连接两次", Content:"相同的表在 FROM 子句中至少出现两次，可以简化为对该表的单次访问。", Case:"select tb1.col from (tb1, tb2) join tb2 on tb1.id=tb.id where tb1.id=1", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
@@ -108,3 +110,5 @@ advisor.Rule{Item:"TBL.002", Severity:"L4", Summary:"请为表选择合适的存
 advisor.Rule{Item:"TBL.003", Severity:"L8", Summary:"以DUAL命名的表在数据库中有特殊含义", Content:"DUAL表为虚拟表，不需要创建即可使用，也不建议服务以DUAL命名表。", Case:"create table dual(id int, primary key (id));", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
 advisor.Rule{Item:"TBL.004", Severity:"L2", Summary:"表的初始AUTO_INCREMENT值不为0", Content:"AUTO_INCREMENT不为0会导致数据空洞。", Case:"CREATE TABLE tbl (a int) AUTO_INCREMENT = 10;", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
 advisor.Rule{Item:"TBL.005", Severity:"L4", Summary:"请使用推荐的字符集", Content:"表字符集只允许设置为utf8,utf8mb4", Case:"CREATE TABLE tbl (a int) DEFAULT CHARSET = latin1;", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
+advisor.Rule{Item:"TBL.006", Severity:"L1", Summary:"不建议使用视图", Content:"", Case:"", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
+advisor.Rule{Item:"TBL.007", Severity:"L1", Summary:"不建议使用临时表", Content:"", Case:"", Position:0, Func:func(*advisor.Query4Audit) advisor.Rule {...}}
--- a/doc/heuristic.md
+++ b/doc/heuristic.md
@@ -342,16 +342,6 @@ update tbl set col=1
 ```sql
 update tbl set col=1
 ```
-## 不建议使用存储过程、视图、触发器、临时表等
-
-* **Item**:CLA.017
-* **Severity**:L2
-* **Content**:这些功能的使用在一定程度上会使得程序难以调试和拓展，更没有移植性，且会极大的增加出现 BUG 的概率。
-* **Case**:
-
-```sql
-CREATE VIEW v_today (today) AS SELECT CURRENT_DATE;
-```
 ## 不建议使用 SELECT \* 类型查询

 * **Item**:COL.001
@@ -601,6 +591,36 @@ SELECT COUNT(1) FROM tbl;

 ```sql
 SELECT SUM(COL) FROM tbl;
+```
+## 不建议使用触发器
+
+* **Item**:FUN.007
+* **Severity**:L1
+* **Content**:
+* **Case**:
+
+```sql
+
+```
+## 不建议使用存储过程
+
+* **Item**:FUN.008
+* **Severity**:L1
+* **Content**:
+* **Case**:
+
+```sql
+
+```
+## 不建议使用自定义函数
+
+* **Item**:FUN.009
+* **Severity**:L1
+* **Content**:
+* **Case**:
+
+```sql
+
 ```
 ## 不建议对等值查询列使用 GROUP BY

@@ -1142,3 +1162,23 @@ CREATE TABLE tbl (a int) AUTO_INCREMENT = 10;
 ```sql
 CREATE TABLE tbl (a int) DEFAULT CHARSET = latin1;
 ```
+## 不建议使用视图
+
+* **Item**:TBL.006
+* **Severity**:L1
+* **Content**:
+* **Case**:
+
+```sql
+
+```
+## 不建议使用临时表
+
+* **Item**:TBL.007
+* **Severity**:L1
+* **Content**:
+* **Case**:
+
+```sql
+
+```