binding.Error 和 validator.Error 行为不一致

[XIELongDragon]

当使用msg自定义错误信息时
validator的Error的行为是，如果有自定义错误信息就只返回自定义信息

// Error implements error interface.
func (e *Error) Error() string {
	if e.Msg != "" {
		return e.Msg
	}
	return "invalid parameter: " + e.FailPath
}

而binding的Error却会有其他附加信息

func (e *Error) Error() string {
	if e.Msg != "" {
		return e.ErrType + ": expr_path=" + e.FailField + ", cause=" + e.Msg
	}
	return e.ErrType + ": expr_path=" + e.FailField + ", cause=invalid"
}

建议，

1. 两者的行为应该一致。
2. 可以考虑在e.Msg支持占位符，由用户自己决定想要的错误信息格式。如

placeholders := map[string]string {"{{ ErrType }}":e.ErrType, "{{ FailField  }}":e..FailField }
msg := e.Msg

for k, v := range placeholders{
    msg = strings.Replace(msg, k, v)
}

return msg

[andeya]

两个包都提供了SetErrorFactory函数，支持自定义。其中binding的SetErrorFactory会自动设置其内部validator的error格式
https://github.com/bytedance/go-tagexpr/blob/master/binding/default.go#L35
https://github.com/bytedance/go-tagexpr/blob/master/validator/default.go#L23

[andeya]

你提的模板变量这个思路，作为默认实现也不错，我有空实现一下吧