Question

我的main函数从文件读取json，将其解组成结构，将其转换为另一种结构类型，并通过stdout吐出格式化的JSON。

我正在尝试实现goroutines和channel来为我的for循环添加并发性。

func main() {
    muvMap := map[string]string{"male": "M", "female": "F"}
    fileA, err := os.Open("serviceAfileultimate.json")
    if err != nil {
        panic(err)
    }
    defer fileA.Close()

    data := make([]byte, 10000)
    count, err := fileA.Read(data)
    if err != nil {
        panic(err)
    }

    dataBytes := data[:count]

    var servicesA ServiceA
    json.Unmarshal(dataBytes, &servicesA)
    var servicesB = make([]ServiceB, servicesA.Count)
    goChannels := make(chan ServiceB, servicesA.Count)

    for i := 0; i < servicesA.Count; i++ {
        go func() {
            reflect.ValueOf(&servicesB[i]).Elem().FieldByName("Address").SetString(Merge(&servicesA.Users[i].Location))
            reflect.ValueOf(&servicesB[i]).Elem().FieldByName("Date_Of_Birth").SetString(dateCopyTransform(servicesA.Users[i].Dob))
            reflect.ValueOf(&servicesB[i]).Elem().FieldByName("Email").SetString(servicesA.Users[i].Email)
            reflect.ValueOf(&servicesB[i]).Elem().FieldByName("Fullname").SetString(Merge(&servicesA.Users[i].Name))
            reflect.ValueOf(&servicesB[i]).Elem().FieldByName("Gender").SetString(muvMap[servicesA.Users[i].Gender])
            reflect.ValueOf(&servicesB[i]).Elem().FieldByName("Phone").SetString(servicesA.Users[i].Cell)
            reflect.ValueOf(&servicesB[i]).Elem().FieldByName("Username").SetString(servicesA.Users[i].Username)
            goChannels <- servicesB[i]
        }()
    }

    for index := range goChannels {
        json.NewEncoder(os.Stdout).Encode(index)
    }

}

它编译但返回的信息如下：

goroutine 1 [chan receive]: main.main() C://.....go.94 +0x55b.

Answer 1

您正在打印频道信息，而不是其中包含的数据。你不想要一个循环，你只想接收然后打印。

   json := <-index
   json.NewEncoder(os.Stdout).Encode(json)

现在我需要指出，代码不会阻止。如果你想继续阅读，直到完成所有工作，你需要某种锁定/协调机制。

您经常会看到像

这样的内容

for {
   select {
        case json := <-jsonChannel:
            // do stuff
        case <-abort:
            // get out of here 
   }
}

要解决这个问题。另外，您只需使用默认容量初始化频道（意味着它是缓冲频道），这很奇怪。我建议您查看有关该主题的一些教程，因为整体上您的设计需要一些工作实际上是对非并发实现的改进。最后，您可以找到图书馆来为您抽象一些这项工作，大多数人可能会建议您这样做。这是一个例子; https://github.com/lytics/squaredance

for循环上的goroutine频道

1 个答案: