2 回答
TA贡献1725条经验 获得超7个赞
您的代码中有许多可以优化的重复代码。
您的代码必须至少包含以下“片段”:
从条目名称到条目文本的映射,可以存储在映射中,例如
var mappings = map[string]string {
"Foo": "Foo text",
"Bar": "Bar text",
"Baz": "Baz text",
"Bat": "Bat text",
// ... other mappings
}
以及属于一个组的键列表,可以存储在一个切片中,例如
var group1 = []string{"Foo", "Bar", "Baz", "Bat"}
一旦你定义了这些,你就可以有一个处理组的辅助方法:
func handleGroup(r *http.Request, group []string) (res string) {
for _, v := range group {
if r.FormValue(v) == "on" {
res := ", " + mappings[v]
}
}
if res == "" {
return "none\n"
}
return res[2:] + "\n"
}
就这样。在此之后,您的处理程序可以如此简单:
func checkboxHandler(w http.ResponseWriter, r *http.Request) {
// Handle group1:
res1 := handleGroup(r, group1)
// Handle group2:
res2 := handleGroup(r, group2)
}
笔记:
这不是您的要求,但该解决方案可以非常轻松地处理翻译:每个翻译都可以有自己的mappings地图,仅此而已。没有什么需要改变的。
性能也不是您关心的问题,但是以这种方式附加字符串并不是很有效。如果性能至少是一个小问题,您可以使用以下方法改进它而不会增加复杂性bytes.Buffer:
func handleGroup(r *http.Request, group []string) string {
buf := &bytes.Buffer{}
for _, v := range group {
if r.FormValue(v) == "on" {
buf.WriteString(", ")
buf.WriteString(mappings[v])
}
}
if buf.Len() == 0 {
return "none\n"
}
buf.WriteString("\n")
return string(buf.Bytes()[2:])
}
TA贡献1836条经验 获得超13个赞
这会将表单值存储到一个数组中。然后,它会将数组迭代为一个字符串,并在每个名称的末尾附加“,”。然后,如果它长于 2,它将把最后一个“,”(2 个字节)放在最后,否则,打印“无”
func(w http.ResponseWriter, r *http.Request) {
r.ParseMultipartForm(0)
arr := []string{}
if r.FormValue("Foo") {
arr = append(arr, "Foo")
}
if r.FormValue("Bar") {
arr = append(arr, "Bar")
}
if r.FormValue("Baz") {
arr = append(arr, "Baz")
}
if r.FormValue("Bat") {
arr = append(arr, "Bat")
}
out := ""
for _, title := range arr {
out += title +", "
}
if len(out) > 2 {
out := out[0: len(out)-2]
} else {
out = "None"
}
fmt.Println(out)
}
如果你想迭代,
for k, vs:= range r.Form {
for _, v:= range vs{
fmt.Println(k, v)
}
}
- 2 回答
- 0 关注
- 159 浏览
添加回答
举报