refactor: change validate reserved words

src/dataobj/metric.go

@@ -63,17 +63,14 @@ func (m *MetricValue) CheckValidity() (err error) {
 		return
 	}
 
-	//将保留字替换
+	//检测保留字
-	var illegal bool
+	if HasReservedWords(m.Metric) {
-	m.Metric, illegal = ReplaceReservedWords(m.Metric)
+		err = fmt.Errorf("metric:%s contains reserved words:[\\t] [\\r] [\\n] [,] [ ] [=]", m.Metric)
-	if illegal {
-		err = fmt.Errorf("Metric contains reserved word")
 		return
 	}
 
-	m.Endpoint, illegal = ReplaceReservedWords(m.Endpoint)
+	if HasReservedWords(m.Endpoint) {
-	if illegal {
+		err = fmt.Errorf("endpoint:%s contains reserved words:[\\t] [\\r] [\\n] [,] [ ] [=]", m.Endpoint)
-		err = fmt.Errorf("Endpoint contains reserved word:%s", m.Endpoint)
 		return
 	}
 
@@ -151,7 +148,7 @@ func (m *MetricValue) CheckValidity() (err error) {
 	return
 }
 
-func ReplaceReservedWords(str string) (string, bool) {
+func HasReservedWords(str string) bool {
 	if -1 == strings.IndexFunc(str,
 		func(r rune) bool {
 			return r == '\t' ||
@@ -159,27 +156,13 @@ func ReplaceReservedWords(str string) (string, bool) {
 				r == '\n' ||
 				r == ',' ||
 				r == ' ' ||
-				r == ':' ||
 				r == '='
 		}) {
 
-		return str, false
+		return false
 	}
 
-	return strings.Map(func(r rune) rune {
+	return true
-		if r == '\t' ||
-			r == '\r' ||
-			r == '\n' ||
-			r == ',' ||
-			r == ' ' ||
-			r == ':' ||
-			r == '=' {
-			return '_'
-		}
-		return r
-	}, str), true
-
-	return str, false
 }
 
 func SortedTags(tags map[string]string) string {