合规国际互联网加速 OSASE为企业客户提供高速稳定SD-WAN国际加速解决方案。 广告
1、关闭HTTP的响应 当你使用标准http库发起请求时,你得到一个http的响应变量。如果你不读取响应主体,你依旧需要关闭它。注意对于空的响应你也一定要这么做。对于新的Go开发者而言,这个很容易就会忘掉。 一些新的Go开发者确实尝试关闭响应主体,但他们在错误的地方做。 ~~~ package main import ( "fmt" "io/ioutil" "net/http" ) func main() { resp, err := http.Get("http://www.oldboygo.cn/") // resp, err := http.Get("http://www.baidu.cn/") defer resp.Body.Close() //not ok if err != nil { fmt.Println(err) return } body, err := ioutil.ReadAll(resp.Body) if err != nil { fmt.Println(err) return } fmt.Println(string(body)) } ~~~ 这段代码对于成功的请求没问题,但如果http的请求失败,resp变量可能会是nil,这将导致一个runtime panic。 ~~~ panic: runtime error: invalid memory address or nil pointer dereference [signal SIGSEGV: segmentation violation code=0x1 addr=0x40 pc=0x11eb322] ~~~ 最常见的关闭响应主体的方法是在http响应的错误检查后调用defer。 ~~~ package main import ( "fmt" "io/ioutil" "net/http" ) func main() { // resp, err := http.Get("https://api.ipify.org?format=json") resp, err := http.Get("https://www.oldboygo.com") if err != nil { fmt.Println(err) return } defer resp.Body.Close() //ok, most of the time :-) body, err := ioutil.ReadAll(resp.Body) if err != nil { fmt.Println(err) return } fmt.Println(string(body)) } ~~~ 运行结果: ~~~ Get https://www.oldboygo.com: dial tcp: lookup www.oldboygo.com: no such host ~~~ 大多数情况下,当你的http响应失败时,resp变量将为nil,而err变量将是non-nil。然而,当你得到一个重定向的错误时,两个变量都将是non-nil。这意味着你最后依然会内存泄露。 通过在http响应错误处理中添加一个关闭non-nil响应主体的的调用来修复这个问题。另一个方法是使用一个defer调用来关闭所有失败和成功的请求的响应主体。 ~~~ package main import ( "fmt" "io/ioutil" "net/http" ) func main() { resp, err := http.Get("https://api.ipify.org?format=json") if resp != nil { defer resp.Body.Close() } if err != nil { fmt.Println(err) return } body, err := ioutil.ReadAll(resp.Body) if err != nil { fmt.Println(err) return } fmt.Println(string(body)) } ~~~ resp.Body.Close()的原始实现也会读取并丢弃剩余的响应主体数据。这确保了http的链接在keepalive http连接行为开启的情况下,可以被另一个请求复用。最新的http客户端的行为是不同的。现在读取并丢弃剩余的响应数据是你的职责。如果你不这么做,http的连接可能会关闭,而无法被重用。 如果http连接的重用对你的应用很重要,你可能需要在响应处理逻辑的后面添加像下面的代码: _, err = io.Copy(ioutil.Discard, resp.Body) 如果你不立即读取整个响应将是必要的,这可能在你处理json API响应时会发生: json.NewDecoder(resp.Body).Decode(&data) 2、关闭HTTP的连接 一些HTTP服务器保持会保持一段时间的网络连接(根据HTTP 1.1的说明和服务器端的“keep-alive”配置)。默认情况下,标准http库只在目标HTTP服务器要求关闭时才会关闭网络连接。这意味着你的应用在某些条件下消耗完sockets/file的描述符。 你可以通过设置请求变量中的Close域的值为true,来让http库在请求完成时关闭连接。 另一个选项是添加一个Connection的请求头,并设置为close。目标HTTP服务器应该也会响应一个Connection: close的头。当http库看到这个响应头时,它也将会关闭连接。 ~~~ package main import ( "fmt" "io/ioutil" "net/http" ) func main() { req, err := http.NewRequest("GET", "http://golang.org", nil) if err != nil { fmt.Println(err) return } req.Close = true //or do this: //req.Header.Add("Connection", "close") resp, err := http.DefaultClient.Do(req) if resp != nil { defer resp.Body.Close() } if err != nil { fmt.Println(err) return } body, err := ioutil.ReadAll(resp.Body) if err != nil { fmt.Println(err) return } fmt.Println(len(string(body))) } ~~~ 你也可以取消http的全局连接复用。你将需要为此创建一个自定义的http传输配置。 ~~~ package main import ( "fmt" "io/ioutil" "net/http" ) func main() { tr := &http.Transport{DisableKeepAlives: true} client := &http.Client{Transport: tr} resp, err := client.Get("http://golang.org") if resp != nil { defer resp.Body.Close() } if err != nil { fmt.Println(err) return } fmt.Println(resp.StatusCode) body, err := ioutil.ReadAll(resp.Body) if err != nil { fmt.Println(err) return } fmt.Println(len(string(body))) } ~~~ 如果你向同一个HTTP服务器发送大量的请求,那么把保持网络连接的打开是没问题的。然而,如果你的应用在短时间内向大量不同的HTTP服务器发送一两个请求,那么在引用收到响应后立刻关闭网络连接是一个好主意。增加打开文件的限制数可能也是个好主意。当然,正确的选择源自于应用。 3、比较Structs, Arrays, Slices, and Maps 如果结构体中的各个元素都可以用你可以使用等号来比较的话,那就可以使用相号, == 来比较结构体变量。 ~~~ package main import "fmt" type data struct { num int fp float32 complex complex64 str string char rune yes bool events <-chan string handler interface{} ref *byte raw [10]byte } func main() { v1 := data{} v2 := data{} fmt.Println("v1 == v2:", v1 == v2) //prints: v1 == v2: true } ~~~ 运行结果: ~~~ v1 == v2: true ~~~ 如果结构体中的元素无法比较,那使用等号将导致编译错误。注意数组仅在它们的数据元素可比较的情况下才可以比较。 ~~~ package main import "fmt" type data struct { num int //ok checks [10]func() bool //not comparable doit func() bool //not comparable m map[string]string //not comparable bytes []byte //not comparable } func main() { v1 := data{} v2 := data{} fmt.Println("v1 == v2:", v1 == v2) } ~~~ 编译错误: ~~~ ./main.go:16:30: invalid operation: v1 == v2 (struct containing [10]func() bool cannot be compared) ~~~ Go确实提供了一些助手函数,用于比较那些无法使用等号比较的变量。 最常用的方法是使用reflect包中的 DeepEqual()函数。 ~~~ package main import ( "fmt" "reflect" ) type data struct { num int //ok checks [10]func() bool //not comparable doit func() bool //not comparable m map[string]string //not comparable bytes []byte //not comparable } func main() { v1 := data{} v2 := data{} fmt.Println("v1 == v2:", reflect.DeepEqual(v1, v2)) //prints: v1 == v2: true m1 := map[string]string{"one": "a", "two": "b"} m2 := map[string]string{"two": "b", "one": "a"} fmt.Println("m1 == m2:", reflect.DeepEqual(m1, m2)) //prints: m1 == m2: true s1 := []int{1, 2, 3} s2 := []int{1, 2, 3} fmt.Println("s1 == s2:", reflect.DeepEqual(s1, s2)) //prints: s1 == s2: true } ~~~ 运行结果: ~~~ v1 == v2: true m1 == m2: true s1 == s2: true ~~~ 除了很慢(这个可能会也可能不会影响你的应用),DeepEqual()也有其他自身的技巧。 ~~~ package main import ( "fmt" "reflect" ) func main() { var b1 []byte = nil b2 := []byte{} fmt.Println("b1 == b2:", reflect.DeepEqual(b1, b2)) //prints: b1 == b2: false } ~~~ 运行结果: ~~~ b1 == b2: false ~~~ DeepEqual()不会认为空的slice与“nil”的slice相等。这个行为与你使用bytes.Equal()函数的行为不同。bytes.Equal()认为“nil”和空的slice是相等的。 ~~~ package main import ( "bytes" "fmt" ) func main() { var b1 []byte = nil b2 := []byte{} fmt.Println("b1 == b2:", bytes.Equal(b1, b2)) //prints: b1 == b2: true } ~~~ 运行结果: ~~~ b1 == b2: true ~~~ DeepEqual()在比较slice时并不总是完美的。 ~~~ package main import ( "encoding/json" "fmt" "reflect" ) func main() { var str string = "one" var in interface{} = "one" fmt.Println("str == in:", str == in, reflect.DeepEqual(str, in)) //prints: str == in: true true v1 := []string{"one", "two"} v2 := []interface{}{"one", "two"} fmt.Println("v1 == v2:", reflect.DeepEqual(v1, v2)) //prints: v1 == v2: false (not ok) data := map[string]interface{}{ "code": 200, "value": []string{"one", "two"}, } encoded, _ := json.Marshal(data) var decoded map[string]interface{} json.Unmarshal(encoded, &decoded) fmt.Println("data == decoded:", reflect.DeepEqual(data, decoded)) //prints: data == decoded: false (not ok) } ~~~ 运行结果: ~~~ str == in: true true v1 == v2: false data == decoded: false ~~~ 如果你的byte slice(或者字符串)中包含文字数据,而当你要不区分大小写形式的值时(在使用==,bytes.Equal(),或者bytes.Compare()),你可能会尝试使用“bytes”和“string”包中的ToUpper()或者ToLower()函数。对于英语文本,这么做是没问题的,但对于许多其他的语言来说就不行了。这时应该使用strings.EqualFold()和bytes.EqualFold()。 如果你的byte slice中包含需要验证用户数据的隐私信息(比如,加密哈希、tokens等),不要使用reflect.DeepEqual()、bytes.Equal(),或者bytes.Compare(),因为这些函数将会让你的应用易于被定时攻击。为了避免泄露时间信息,使用'crypto/subtle'包中的函数(即,subtle.ConstantTimeCompare())。 4、从Panic中恢复 recover()函数可以用于获取/拦截panic。仅当在一个defer函数中被完成时,调用recover()将会完成这个小技巧。 错误代码: ~~~ package main import "fmt" func main() { recover() // doesn't do anything panic("not good") recover() // won't be executed fmt.Println("ok") } ~~~ 运行结果: ~~~ panic: not good ~~~ 正确代码: ~~~ package main import "fmt" func main() { defer func() { fmt.Println("recovered:", recover()) }() panic("not good") } ~~~ 运行结果: ~~~ recovered: not good ~~~ recover()的调用仅当它在defer函数中被直接调用时才有效。 错误代码: ~~~ package main import "fmt" func doRecover() { fmt.Println("recovered =>", recover()) //prints: recovered => <nil> } func main() { defer func() { doRecover() //panic is not recovered }() panic("not good") } ~~~ 运行结果: ~~~ recovered => <nil> panic: not good ~~~ 5、在Slice, Array, and Map "range"语句中更新引用元素的值 在“range”语句中生成的数据的值是真实集合元素的拷贝。它们不是原有元素的引用。 这意味着更新这些值将不会修改原来的数据。同时也意味着使用这些值的地址将不会得到原有数据的指针。 ~~~ package main import "fmt" func main() { data := []int{1, 2, 3} for _, v := range data { v *= 10 //original item is not changed } fmt.Println("data:", data) //prints data: [1 2 3] } ~~~ 运行结果: ~~~ data: [1 2 3] ~~~ 如果你需要更新原有集合中的数据,使用索引操作符来获得数据。 ~~~ package main import "fmt" func main() { data := []int{1, 2, 3} for i, _ := range data { data[i] *= 10 } fmt.Println("data:", data) //prints data: [10 20 30] } ~~~ 运行结果: ~~~ data: [10 20 30] ~~~ 如果你的集合保存的是指针,那规则会稍有不同。 如果要更新原有记录指向的数据,你依然需要使用索引操作,但你可以使用for range语句中的第二个值来更新存储在目标位置的数据。 ~~~ &{10} &{20} &{30} ~~~ 6、在Slice中"隐藏"数据 当你重新划分一个slice时,新的slice将引用原有slice的数组。如果你忘了这个行为的话,在你的应用分配大量临时的slice用于创建新的slice来引用原有数据的一小部分时,会导致难以预期的内存使用。 ~~~ package main import "fmt" func get() []byte { raw := make([]byte, 10000) fmt.Println(len(raw), cap(raw), &raw[0]) //prints: 10000 10000 <byte_addr_x> return raw[:3] } func main() { data := get() fmt.Println(len(data), cap(data), &data[0]) //prints: 3 10000 <byte_addr_x> } ~~~ 运行结果: ~~~ 10000 10000 0xc420080000 3 10000 0xc420080000 ~~~ 为了避免这个陷阱,你需要从临时的slice中拷贝数据(而不是重新划分slice)。 ~~~ package main import "fmt" func get() []byte { raw := make([]byte, 10000) fmt.Println(len(raw), cap(raw), &raw[0]) //prints: 10000 10000 <byte_addr_x> res := make([]byte, 3) copy(res, raw[:3]) return res } func main() { data := get() fmt.Println(len(data), cap(data), &data[0]) //prints: 3 3 <byte_addr_y> } ~~~ 7、Slice的数据“毁坏” 比如说你需要重新一个路径(在slice中保存)。你通过修改第一个文件夹的名字,然后把名字合并来创建新的路劲,来重新划分指向各个文件夹的路径。 ~~~ package main import ( "bytes" "fmt" ) func main() { path := []byte("AAAA/BBBBBBBBB") sepIndex := bytes.IndexByte(path, '/') dir1 := path[:sepIndex] dir2 := path[sepIndex+1:] fmt.Println("dir1 =>", string(dir1)) //prints: dir1 => AAAA fmt.Println("dir2 =>", string(dir2)) //prints: dir2 => BBBBBBBBB dir1 = append(dir1, "suffix"...) path = bytes.Join([][]byte{dir1, dir2}, []byte{'/'}) fmt.Println("dir1 =>", string(dir1)) //prints: dir1 => AAAAsuffix fmt.Println("dir2 =>", string(dir2)) //prints: dir2 => uffixBBBB fmt.Println("new path =>", string(path)) } ~~~ 运行结果: ~~~ dir1 => AAAA dir2 => BBBBBBBBB dir1 => AAAAsuffix dir2 => uffixBBBB new path => AAAAsuffix/uffixBBBB ~~~ 结果与你想的不一样。与"AAAAsuffix/BBBBBBBBB"相反,你将会得到"AAAAsuffix/uffixBBBB"。这个情况的发生是因为两个文件夹的slice都潜在的引用了同一个原始的路径slice。这意味着原始路径也被修改了。根据你的应用,这也许会是个问题。 通过分配新的slice并拷贝需要的数据,你可以修复这个问题。另一个选择是使用完整的slice表达式。 ~~~ package main import ( "bytes" "fmt" ) func main() { path := []byte("AAAA/BBBBBBBBB") sepIndex := bytes.IndexByte(path, '/') dir1 := path[:sepIndex:sepIndex] //full slice expression dir2 := path[sepIndex+1:] fmt.Println("dir1 =>", string(dir1)) //prints: dir1 => AAAA fmt.Println("dir2 =>", string(dir2)) //prints: dir2 => BBBBBBBBB dir1 = append(dir1, "suffix"...) path = bytes.Join([][]byte{dir1, dir2}, []byte{'/'}) fmt.Println("dir1 =>", string(dir1)) //prints: dir1 => AAAAsuffix fmt.Println("dir2 =>", string(dir2)) //prints: dir2 => BBBBBBBBB fmt.Println("new path =>", string(path)) } ~~~ 运行结果: ~~~ dir1 => AAAA dir2 => BBBBBBBBB dir1 => AAAAsuffix dir2 => BBBBBBBBB new path => AAAAsuffix/BBBBBBBBB ~~~ 完整的slice表达式中的额外参数可以控制新的slice的容量。现在在那个slice后添加元素将会触发一个新的buffer分配,而不是覆盖第二个slice中的数据。 8、陈旧的(Stale)Slices 多个slice可以引用同一个数据。比如,当你从一个已有的slice创建一个新的slice时,这就会发生。如果你的应用功能需要这种行为,那么你将需要关注下“走味的”slice。 在某些情况下,在一个slice中添加新的数据,在原有数组无法保持更多新的数据时,将导致分配一个新的数组。而现在其他的slice还指向老的数组(和老的数据)。 ~~~ package main import "fmt" func main() { s1 := []int{1, 2, 3} fmt.Println(len(s1), cap(s1), s1) //prints 3 3 [1 2 3] s2 := s1[1:] fmt.Println(len(s2), cap(s2), s2) //prints 2 2 [2 3] for i := range s2 { s2[i] += 20 } //still referencing the same array fmt.Println(s1) //prints [1 22 23] fmt.Println(s2) //prints [22 23] s2 = append(s2, 4) for i := range s2 { s2[i] += 10 } //s1 is now "stale" fmt.Println(s1) //prints [1 22 23] fmt.Println(s2) //prints [32 33 14] } ~~~ 运行结果: ~~~ 3 3 [1 2 3] 2 2 [2 3] [1 22 23] [22 23] [1 22 23] [32 33 14] ~~~ 9、类型声明和方法 当你通过把一个现有(非interface)的类型定义为一个新的类型时,新的类型不会继承现有类型的方法。 错误代码: ~~~ package main import "sync" type myMutex sync.Mutex func main() { var mtx myMutex mtx.Lock() mtx.Unlock() } ~~~ 编译错误: ~~~ ./main.go:9:5: mtx.Lock undefined (type myMutex has no field or method Lock) ./main.go:10:5: mtx.Unlock undefined (type myMutex has no field or method Unlock) ~~~ 如果你确实需要原有类型的方法,你可以定义一个新的struct类型,用匿名方式把原有类型嵌入其中。 正确代码: ~~~ package main import "sync" type myLocker struct { sync.Mutex } func main() { var lock myLocker lock.Lock() lock.Unlock() } ~~~ interface类型的声明也会保留它们的方法集合。 ~~~ package main import "sync" type myLocker sync.Locker func main() { var lock myLocker = new(sync.Mutex) lock.Lock() lock.Unlock() } ~~~ 10、从"for switch"和"for select"代码块中跳出 没有标签的“break”声明只能从内部的switch/select代码块中跳出来。如果无法使用“return”声明的话,那就为外部循环定义一个标签是另一个好的选择。 ~~~ package main import "fmt" func main() { loop: for { switch { case true: fmt.Println("breaking out...") break loop } } fmt.Println("out!") } ~~~ 运行结果: ~~~ breaking out... out! ~~~ "goto"声明也可以完成这个功能。。。 11、"for"声明中的迭代变量和闭包 这在Go中是个很常见的技巧。for语句中的迭代变量在每次迭代时被重新使用。这就意味着你在for循环中创建的闭包(即函数字面量)将会引用同一个变量(而在那些goroutine开始执行时就会得到那个变量的值)。 ~~~ package main import ( "fmt" "time" ) func main() { data := []string{"one", "two", "three"} for _, v := range data { go func() { fmt.Println(v) }() } time.Sleep(3 * time.Second) //goroutines print: three, three, three } ~~~ 运行结果: ~~~ three three three ~~~ 最简单的解决方法(不需要修改goroutine)是,在for循环代码块内把当前迭代的变量值保存到一个局部变量中。 ~~~ package main import ( "fmt" "time" ) func main() { data := []string{"one", "two", "three"} for _, v := range data { vcopy := v // go func() { fmt.Println(vcopy) }() } time.Sleep(3 * time.Second) //goroutines print: one, two, three } ~~~ 运行结果: ~~~ three one two ~~~ 另一个解决方法是把当前的迭代变量作为匿名goroutine的参数。 ~~~ package main import ( "fmt" "time" ) func main() { data := []string{"one", "two", "three"} for _, v := range data { go func(in string) { fmt.Println(in) }(v) } time.Sleep(3 * time.Second) //goroutines print: one, two, three } ~~~ 运行结果: ~~~ three one two ~~~ 下面这个陷阱稍微复杂一些的版本。 ~~~ package main import ( "fmt" "time" ) type field struct { name string } func (p *field) print() { fmt.Println(p.name) } func main() { data := []field{{"one"}, {"two"}, {"three"}} for _, v := range data { go v.print() } time.Sleep(3 * time.Second) //goroutines print: three, three, three } ~~~ 运行结果: ~~~ three three three ~~~ ~~~ package main import ( "fmt" "time" ) type field struct { name string } func (p *field) print() { fmt.Println(p.name) } func main() { data := []field{{"one"}, {"two"}, {"three"}} for _, v := range data { v := v go v.print() } time.Sleep(3 * time.Second) //goroutines print: one, two, three } ~~~ 运行结果: ~~~ three one two ~~~ 在运行这段代码时你认为会看到什么结果?(原因是什么?) ~~~ package main import ( "fmt" "time" ) type field struct { name string } func (p *field) print() { fmt.Println(p.name) } func main() { data := []*field{{"one"}, {"two"}, {"three"}} for _, v := range data { go v.print() } time.Sleep(3 * time.Second) } ~~~ 运行结果: ~~~ three one two ~~~ 12、Defer函数调用参数的求值 被defer的函数的参数会在defer声明时求值(而不是在函数实际执行时)。 Arguments for a deferred function call are evaluated when the defer statement is evaluated (not when the function is actually executing). ~~~ package main import "fmt" func main() { var i int = 1 defer fmt.Println("result =>", func() int { return i * 2 }()) i++ //prints: result => 2 (not ok if you expected 4) } ~~~ 运行结果: ~~~ result => 2 ~~~ 13、被Defer的函数调用执行 被defer的调用会在包含的函数的末尾执行,而不是包含代码块的末尾。对于Go新手而言,一个很常犯的错误就是无法区分被defer的代码执行规则和变量作用规则。如果你有一个长时运行的函数,而函数内有一个for循环试图在每次迭代时都defer资源清理调用,那就会出现问题。 ~~~ package main import ( "fmt" "os" "path/filepath" ) func main() { if len(os.Args) != 2 { os.Exit(-1) } start, err := os.Stat(os.Args[1]) if err != nil || !start.IsDir() { os.Exit(-1) } var targets []string filepath.Walk(os.Args[1], func(fpath string, fi os.FileInfo, err error) error { if err != nil { return err } if !fi.Mode().IsRegular() { return nil } targets = append(targets, fpath) return nil }) for _, target := range targets { f, err := os.Open(target) if err != nil { fmt.Println("bad target:", target, "error:", err) //prints error: too many open files break } defer f.Close() //will not be closed at the end of this code block //do something with the file... } } ~~~ 运行结果: ~~~ exit status 255 ~~~ 解决这个问题的一个方法是把代码块写成一个函数。 ~~~ package main import ( "fmt" "os" "path/filepath" ) func main() { if len(os.Args) != 2 { os.Exit(-1) } start, err := os.Stat(os.Args[1]) if err != nil || !start.IsDir() { os.Exit(-1) } var targets []string filepath.Walk(os.Args[1], func(fpath string, fi os.FileInfo, err error) error { if err != nil { return err } if !fi.Mode().IsRegular() { return nil } targets = append(targets, fpath) return nil }) for _, target := range targets { func() { f, err := os.Open(target) if err != nil { fmt.Println("bad target:", target, "error:", err) return } defer f.Close() //ok //do something with the file... }() } } ~~~ 另一个方法是去掉defer语句 14、失败的类型断言 失败的类型断言返回断言声明中使用的目标类型的“零值”。这在与隐藏变量混合时,会发生未知情况。 ~~~ package main import "fmt" func main() { var data interface{} = "great" if data, ok := data.(int); ok { fmt.Println("[is an int] value =>", data) } else { fmt.Println("[not an int] value =>", data) //prints: [not an int] value => 0 (not "great") } } ~~~ 运行结果: ~~~ [not an int] value => 0 ~~~ ~~~ package main import "fmt" func main() { var data interface{} = "great" if res, ok := data.(int); ok { fmt.Println("[is an int] value =>", res) } else { fmt.Println("[not an int] value =>", data) //prints: [not an int] value => great (as expected) } } ~~~ 运行结果: ~~~ [not an int] value => great ~~~ 15、阻塞的Goroutine和资源泄露 Rob Pike在2012年的Google I/O大会上所做的“Go Concurrency Patterns”的演讲上,说道过几种基础的并发模式。从一组目标中获取第一个结果就是其中之一。 ~~~ func First(query string, replicas ...Search) Result { c := make(chan Result) searchReplica := func(i int) { c <- replicas[i](query) } for i := range replicas { go searchReplica(i) } return <-c } ~~~ 这个函数在每次搜索重复时都会起一个goroutine。每个goroutine把它的搜索结果发送到结果的channel中。结果channel的第一个值被返回。 那其他goroutine的结果会怎样呢?还有那些goroutine自身呢? 在First()函数中的结果channel是没缓存的。这意味着只有第一个goroutine返回。其他的goroutine会困在尝试发送结果的过程中。这意味着,如果你有不止一个的重复时,每个调用将会泄露资源。 为了避免泄露,你需要确保所有的goroutine退出。一个不错的方法是使用一个有足够保存所有缓存结果的channel。 ~~~ func First(query string, replicas ...Search) Result { c := make(chan Result, len(replicas)) searchReplica := func(i int) { c <- replicas[i](query) } for i := range replicas { go searchReplica(i) } return <-c } ~~~ 另一个不错的解决方法是使用一个有default情况的select语句和一个保存一个缓存结果的channel。default情况保证了即使当结果channel无法收到消息的情况下,goroutine也不会堵塞。 ~~~ func First(query string, replicas ...Search) Result { c := make(chan Result, 1) searchReplica := func(i int) { select { case c <- replicas[i](query): default: } } for i := range replicas { go searchReplica(i) } return <-c } ~~~ 你也可以使用特殊的取消channel来终止workers。 ~~~ func First(query string, replicas ...Search) Result { c := make(chan Result) done := make(chan struct{}) defer close(done) searchReplica := func(i int) { select { case c <- replicas[i](query): case <-done: } } for i := range replicas { go searchReplica(i) } return <-c } ~~~ 为何在演讲中会包含这些bug?Rob Pike仅仅是不想把演示复杂化。这么作是合理的,但对于Go新手而言,可能会直接使用代码,而不去思考它可能有问题。