如何动态添加struct属性以避免冗余代码？ [关闭]

Posted 2021-04-15

嗨，我基本上都在努力避免代码中的冗余。

这是我的结构：

type Person struct {
    Name   string
    Text   string
    Status string
}

这是我的Func：

func ReworkElementsFromClient(w http.ResponseWriter, r *http.Request){
    var persons []Person
    err := json.NewDecoder(r.Body).Decode(&persons)
    if err != nil {
        http.Error(w, err.Error(), http.StatusBadRequest)
        return
    }

    if len(persons) == 0 {
        err = json.NewEncoder(w).Encode(persons)
        if err != nil {
            http.Error(w, err.Error(), http.StatusInternalServerError)
        }
        return
    }

    persons = append([]Person{persons[0]}, persons...)
    persons = persons[:len(persons)-1]

//Trying to avoid redundancy here 
    persons[0].Name = CreateRandomChars(CreateRandomNumbers(8, 16))
    persons[0].Text = CreateRandomChars(CreateRandomNumbers(8, 16))
    persons[0].Status = CreateRandomChars(CreateRandomNumbers(8, 16))

    err = json.NewEncoder(w).Encode(persons)
        if err != nil {
            http.Error(w, err.Error(), http.StatusInternalServerError)
        }
    return
    }

如何避免在Person[0]调用每个属性来随机化它们？

答案

在不触及反射的情况下，以下代码更清晰，更短，但与您的代码相同，并且可以轻松扩展到新字段：

c := func() string { return CreateRandomChars(CreateRandomNumbers(8, 16)) }

p := persons[0]
p.Name = c()
p.Text = c()
p.Status = c()

persons = append([]Person{p}, persons[:len(persons)-1]...)

虽然我真的不明白为什么你要复制第一个切片元素然后删掉最后一个切片元素。更合乎逻辑的是改变第一个元素而不是删除最后一个元素。

实用程序采用string指针的略微修改版本可能如下所示：

c := func(s *string) { *s = CreateRandomChars(CreateRandomNumbers(8, 16)) }

p := persons[0]
c(&p.Name)
c(&p.Text)
c(&p.Status)

persons = append([]Person{p}, persons[:len(persons)-1]...)

如果字段数增加，也可以进一步简化此指针版本，因为您可以使用for循环传递它们：

c := func(s *string) { *s = CreateRandomChars(CreateRandomNumbers(8, 16)) }

p := persons[0]
for _, s := range []*string{&p.Name, &p.Text, &p.Status} {
    c(s)
}

persons = append([]Person{p}, persons[:len(persons)-1]...)

现在我们只提到c()函数变量一次，我们甚至可以完全忽略它：

p := persons[0]
for _, s := range []*string{&p.Name, &p.Text, &p.Status} {
    *s = CreateRandomChars(CreateRandomNumbers(8, 16))
}

persons = append([]Person{p}, persons[:len(persons)-1]...)