Go的50坑：新Golang开发者要注意的陷阱、技巧和常见错误[2]

时间 2019-11-17

标签 golang 开发者注意陷阱技巧常见错误栏目 Go 繁體版

原文原文链接

初级篇
- 开大括号不能放在单独的一行
- 未使用的变量
- 未使用的Imports
- 简式的变量声明仅能够在函数内部使用
- 使用简式声明重复声明变量
- 偶然的变量隐藏Accidental Variable Shadowing
- 不使用显式类型，没法使用“nil”来初始化变量
- 使用“nil” Slices and Maps
- Map的容量
- 字符串不会为“nil”
- Array函数的参数
- 在Slice和Array使用“range”语句时的出现的不但愿获得的值
- Slices和Arrays是一维的
- 访问不存在的Map Keys
- Strings没法修改
- String和Byte Slice之间的转换
- String和索引操做
- 字符串不老是UTF8文本
- 字符串的长度
- 在多行的Slice、Array和Map语句中遗漏逗号
- log.Fatal和log.Panic不只仅是Log
- 内建的数据结构操做不是同步的
- String在“range”语句中的迭代值
- 对Map使用“for range”语句迭代
- "switch"声明中的失效行为
- 自增和自减
- 按位NOT操做
- 操做优先级的差别
- 未导出的结构体不会被编码
- 有活动的Goroutines下的应用退出
- 向无缓存的Channel发送消息，只要目标接收者准备好就会当即返回
- 向已关闭的Channel发送会引发Panic
- 使用"nil" Channels
- 传值方法的接收者没法修改原有的值
进阶篇
- 关闭HTTP的响应
- 关闭HTTP的链接
- 比较Structs, Arrays, Slices, and Maps
- 从Panic中恢复
- 在Slice, Array, and Map "range"语句中更新引用元素的值
- 在Slice中"隐藏"数据
- Slice的数据“毁坏”
- "走味的"Slices
- 类型声明和方法
- 从"for switch"和"for select"代码块中跳出
- "for"声明中的迭代变量和闭包
- Defer函数调用参数的求值
- 被Defer的函数调用执行
- 失败的类型断言
- 阻塞的Goroutine和资源泄露
高级篇
- 使用指针接收方法的值的实例
- 更新Map的值
- "nil" Interfaces和"nil" Interfaces的值
- 栈和堆变量
- GOMAXPROCS, 并发, 和并行
- 读写操做的重排顺序
- 优先调度

进阶篇

关闭HTTP的响应

level: intermediate

当你使用标准http库发起请求时，你获得一个http的响应变量。若是你不读取响应主体，你依旧须要关闭它。注意对于空的响应你也必定要这么作。对于新的Go开发者而言，这个很容易就会忘掉。golang

一些新的Go开发者确实尝试关闭响应主体，但他们在错误的地方作。express

package main

import (  
    "fmt"
    "net/http"
    "io/ioutil"
)

func main() {  
    resp, err := http.Get("https://api.ipify.org?format=json")
    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。json

最多见的关闭响应主体的方法是在http响应的错误检查后调用 defer。api

package main

import (  
    "fmt"
    "net/http"
    "io/ioutil"
)

func main() {  
    resp, err := http.Get("https://api.ipify.org?format=json")
    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))
}

大多数状况下，当你的http响应失败时， resp变量将为 nil，而 err变量将是 non-nil。然而，当你获得一个重定向的错误时，两个变量都将是 non-nil。这意味着你最后依然会内存泄露。数组

经过在http响应错误处理中添加一个关闭 non-nil响应主体的的调用来修复这个问题。另外一个方法是使用一个 defer调用来关闭全部失败和成功的请求的响应主体。缓存

package main

import (  
    "fmt"
    "net/http"
    "io/ioutil"
)

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的链接可能会关闭，而没法被重用。这个小技巧应该会写在Go 1.5的文档中。服务器

若是http链接的重用对你的应用很重要，你可能须要在响应处理逻辑的后面添加像下面的代码：网络

_, err = io.Copy(ioutil.Discard, resp.Body)

若是你不当即读取整个响应将是必要的，这可能在你处理json API响应时会发生：数据结构

json.NewDecoder(resp.Body).Decode(&data)

关闭HTTP的链接

level: intermediate

一些HTTP服务器保持会保持一段时间的网络链接（根据HTTP 1.1的说明和服务器端的“keep-alive”配置）。默认状况下，标准http库只在目标HTTP服务器要求关闭时才会关闭网络链接。这意味着你的应用在某些条件下消耗完sockets/file的描述符。闭包

你能够经过设置请求变量中的 Close域的值为 true，来让http库在请求完成时关闭链接。

另外一个选项是添加一个 Connection的请求头，并设置为 close。目标HTTP服务器应该也会响应一个 Connection: close的头。当http库看到这个响应头时，它也将会关闭链接。

package main

import (  
    "fmt"
    "net/http"
    "io/ioutil"
)

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"
    "net/http"
    "io/ioutil"
)

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服务器发送一两个请求，那么在引用收到响应后马上关闭网络链接是一个好主意。增长打开文件的限制数可能也是个好主意。固然，正确的选择源自于应用。

比较Structs, Arrays, Slices, and Maps

level: intermediate

若是结构体中的各个元素均可以用你可使用等号来比较的话，那就可使用相号, ==，来比较结构体变量。

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
}

若是结构体中的元素没法比较，那使用等号将致使编译错误。注意数组仅在它们的数据元素可比较的状况下才能够比较。

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)
}

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
}

除了很慢（这个可能会也可能不会影响你的应用）， 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
}

DeepEqual()不会认为空的slice与“nil”的slice相等。这个行为与你使用 bytes.Equal()函数的行为不一样。 bytes.Equal()认为“nil”和空的slice是相等的。

package main

import (  
    "fmt"
    "bytes"
)

func main() {  
    var b1 []byte = nil
    b2 := []byte{}
    fmt.Println("b1 == b2:",bytes.Equal(b1, b2)) //prints: b1 == b2: true
}

DeepEqual()在比较slice时并不老是完美的。

package main

import (  
    "fmt"
    "reflect"
    "encoding/json"
)

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)
}

若是你的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()）。

从Panic中恢复

level: intermediate

recover()函数能够用于获取/拦截panic。仅当在一个defer函数中被完成时，调用 recover()将会完成这个小技巧。

Incorrect:

ackage main

import "fmt"

func main() {  
    recover() //doesn't do anything
    panic("not good")
    recover() //won't be executed :)
    fmt.Println("ok")
}

Works:

package main

import "fmt"

func main() {  
    defer func() {
        fmt.Println("recovered:",recover())
    }()

    panic("not good")
}

recover()的调用仅当它在defer函数中被直接调用时才有效。

Fails:

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")
}

在Slice, Array, and Map "range"语句中更新引用元素的值

level: intermediate

在“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]
}

若是你须要更新原有集合中的数据，使用索引操做符来得到数据。

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]
}

若是你的集合保存的是指针，那规则会稍有不一样。若是要更新原有记录指向的数据，你依然须要使用索引操做，但你可使用 for range语句中的第二个值来更新存储在目标位置的数据。

package main

import "fmt"

func main() {  
    data := []*struct{num int} {{1},{2},{3}}

    for _,v := range data {
        v.num *= 10
    }

    fmt.Println(data[0],data[1],data[2]) //prints &{10} &{20} &{30}
}

在Slice中"隐藏"数据

level: intermediate

当你从新划分一个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>
}

为了不这个陷阱，你须要从临时的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>
}

Slice的数据“毁坏”

level: intermediate

好比说你须要从新一个路径（在slice中保存）。你经过修改第一个文件夹的名字，而后把名字合并来建立新的路劲，来从新划分指向各个文件夹的路径。

package main

import (  
    "fmt"
    "bytes"
)

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 (not ok)

    fmt.Println("new path =>",string(path))
}

结果与你想的不同。与"AAAAsuffix/BBBBBBBBB"相反，你将会获得"AAAAsuffix/uffixBBBB"。这个状况的发生是由于两个文件夹的slice都潜在的引用了同一个原始的路径slice。这意味着原始路径也被修改了。根据你的应用，这也许会是个问题。

经过分配新的slice并拷贝须要的数据，你能够修复这个问题。另外一个选择是使用完整的slice表达式。

package main

import (  
    "fmt"
    "bytes"
)

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 (ok now)

    fmt.Println("new path =>",string(path))
}

完整的slice表达式中的额外参数能够控制新的slice的容量。如今在那个slice后添加元素将会触发一个新的buffer分配，而不是覆盖第二个slice中的数据。

"走味的"Slices

level: intermediate

多个slice能够引用同一个数据。好比，当你从一个已有的slice建立一个新的slice时，这就会发生。若是你的应用功能须要这种行为，那么你将须要关注下“走味的”slice。

在某些状况下，在一个slice中添加新的数据，在原有数组没法保持更多新的数据时，将致使分配一个新的数组。而如今其余的slice还指向老的数组（和老的数据）。

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]
}

类型声明和方法

level: intermediate

当你经过把一个现有（非interface）的类型定义为一个新的类型时，新的类型不会继承现有类型的方法。

Fails:

package main

import "sync"

type myMutex sync.Mutex

func main() {  
    var mtx myMutex
    mtx.Lock() //error
    mtx.Unlock() //error  
}

Compile Errors:

/tmp/sandbox106401185/main.go:9: mtx.Lock undefined (type myMutex has no field or method Lock) /tmp/sandbox106401185/main.go:10: mtx.Unlock undefined (type myMutex has no field or method Unlock)

若是你确实须要原有类型的方法，你能够定义一个新的struct类型，用匿名方式把原有类型嵌入其中。

Works:

package main

import "sync"

type myLocker struct {  
    sync.Mutex
}

func main() {  
    var lock myLocker
    lock.Lock() //ok
    lock.Unlock() //ok
}

interface类型的声明也会保留它们的方法集合。

Works:

package main

import "sync"

type myLocker sync.Locker

func main() {  
    var lock myLocker = new(sync.Mutex)
    lock.Lock() //ok
    lock.Unlock() //ok
}

从"for switch"和"for select"代码块中跳出

level: intermediate

没有标签的“break”声明只能从内部的switch/select代码块中跳出来。若是没法使用“return”声明的话，那就为外部循环定义一个标签是另外一个好的选择。

package main

import "fmt"

func main() {  
    loop:
        for {
            switch {
            case true:
                fmt.Println("breaking out...")
                break loop
            }
        }

    fmt.Println("out!")
}

"goto"声明也能够完成这个功能。。。

"for"声明中的迭代变量和闭包

level: intermediate

这在Go中是个很常见的技巧。 for语句中的迭代变量在每次迭代时被从新使用。这就意味着你在 for循环中建立的闭包（即函数字面量）将会引用同一个变量（而在那些goroutine开始执行时就会获得那个变量的值）。

Incorrect:

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
}

最简单的解决方法（不须要修改goroutine）是，在 for循环代码块内把当前迭代的变量值保存到一个局部变量中。

Works:

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
}

另外一个解决方法是把当前的迭代变量做为匿名goroutine的参数。

Works:

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
}

下面这个陷阱稍微复杂一些的版本。

Incorrect:

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
}

Works:

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
}

在运行这段代码时你认为会看到什么结果？（缘由是什么？）

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)
}

Defer函数调用参数的求值

level: intermediate

被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)
}

被Defer的函数调用执行

level: intermediate

被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...
    }
}

解决这个问题的一个方法是把代码块写成一个函数。

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语句 :-)

失败的类型断言

level: intermediate

失败的类型断言返回断言声明中使用的目标类型的“零值”。这在与隐藏变量混合时，会发生未知状况。

Incorrect:

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")
    }
}

Works:

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)
    }
}

阻塞的Goroutine和资源泄露

level: intermediate

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新手而言，可能会直接使用代码，而不去思考它可能有问题。

高级篇

使用指针接收方法的值的实例

level: advanced

只要值是可取址的，那在这个值上调用指针接收方法是没问题的。换句话说，在某些状况下，你不须要在有一个接收值的方法版本。

然而并非全部的变量是可取址的。Map的元素就不是。经过interface引用的变量也不是。

package main

import "fmt"

type data struct {  
    name string
}

func (p *data) print() {  
    fmt.Println("name:",p.name)
}

type printer interface {  
    print()
}

func main() {  
    d1 := data{"one"}
    d1.print() //ok

    var in printer = data{"two"} //error
    in.print()

    m := map[string]data {"x":data{"three"}}
    m["x"].print() //error
}

Compile Errors:

/tmp/sandbox017696142/main.go:21: cannot use data literal (type data) as type printer in assignment: data does not implement printer (print method has pointer receiver)
/tmp/sandbox017696142/main.go:25: cannot call pointer method on m["x"] /tmp/sandbox017696142/main.go:25: cannot take the address of m["x"]

更新Map的值

level: advanced

若是你有一个struct值的map，你没法更新单个的struct值。

Fails:

package main

type data struct {  
    name string
}

func main() {  
    m := map[string]data {"x":{"one"}}
    m["x"].name = "two" //error
}

Compile Error:

/tmp/sandbox380452744/main.go:9: cannot assign to m["x"].name

这个操做无效是由于map元素是没法取址的。

而让Go新手更加困惑的是slice元素是能够取址的。

package main

import "fmt"

type data struct {  
    name string
}

func main() {  
    s := []data {{"one"}}
    s[0].name = "two" //ok
    fmt.Println(s)    //prints: [{two}]
}

注意在不久以前，使用编译器之一（gccgo）是能够更新map的元素值的，但这一行为很快就被修复了 :-)它也被认为是Go 1.3的潜在特性。在那时还不是要急需支持的，但依旧在todo list中。

第一个有效的方法是使用一个临时变量。

package main

import "fmt"

type data struct {  
    name string
}

func main() {  
    m := map[string]data {"x":{"one"}}
    r := m["x"]
    r.name = "two"
    m["x"] = r
    fmt.Printf("%v",m) //prints: map[x:{two}]
}

另外一个有效的方法是使用指针的map。

package main

import "fmt"

type data struct {  
    name string
}

func main() {  
    m := map[string]*data {"x":{"one"}}
    m["x"].name = "two" //ok
    fmt.Println(m["x"]) //prints: &{two}
}

顺便说下，当你运行下面的代码时会发生什么？

package main

type data struct {  
    name string
}

func main() {  
    m := map[string]*data {"x":{"one"}}
    m["z"].name = "what?" //???
}

"nil" Interfaces和"nil" Interfaces的值

level: advanced

这在Go中是第二最多见的技巧，由于interface虽然看起来像指针，但并非指针。interface变量仅在类型和值为“nil”时才为“nil”。

interface的类型和值会根据用于建立对应interface变量的类型和值的变化而变化。当你检查一个interface变量是否等于“nil”时，这就会致使未预期的行为。

package main

import "fmt"

func main() {  
    var data *byte
    var in interface{}

    fmt.Println(data,data == nil) //prints: <nil> true
    fmt.Println(in,in == nil)     //prints: <nil> true

    in = data
    fmt.Println(in,in == nil)     //prints: <nil> false
    //'data' is 'nil', but 'in' is not 'nil'
}

当你的函数返回interface时，当心这个陷阱。

Incorrect:

package main

import "fmt"

func main() {  
    doit := func(arg int) interface{} {
        var result *struct{} = nil

        if(arg > 0) {
            result = &struct{}{}
        }

        return result
    }

    if res := doit(-1); res != nil {
        fmt.Println("good result:",res) //prints: good result: <nil>
        //'res' is not 'nil', but its value is 'nil'
    }
}

Works:

package main

import "fmt"

func main() {  
    doit := func(arg int) interface{} {
        var result *struct{} = nil

        if(arg > 0) {
            result = &struct{}{}
        } else {
            return nil //return an explicit 'nil'
        }

        return result
    }

    if res := doit(-1); res != nil {
        fmt.Println("good result:",res)
    } else {
        fmt.Println("bad result (res is nil)") //here as expected
    }
}

栈和堆变量

level: advanced

你并不老是知道变量是分配到栈仍是堆上。在C++中，使用 new建立的变量老是在堆上。在Go中，即便是使用 new()或者 make()函数来分配，变量的位置仍是由编译器决定。编译器根据变量的大小和“泄露分析”的结果来决定其位置。这也意味着在局部变量上返回引用是没问题的，而这在C或者C++这样的语言中是不行的。

若是你想知道变量分配的位置，在“go build”或“go run”上传入“-m“ gc标志（即， go run -gcflags -m app.go）。

GOMAXPROCS, 并发, 和并行

level: advanced

默认状况下，Go仅使用一个执行上下文/OS线程（在当前的版本）。这个数量能够经过设置 GOMAXPROCS来提升。

一个常见的误解是， GOMAXPROCS表示了CPU的数量，Go将使用这个数量来运行goroutine。而runtime.GOMAXPROCS()函数的文档让人更加的迷茫。 GOMAXPROCS变量描述（https://golang.org/pkg/runtime/）所讨论OS线程的内容比较好。

你能够设置 GOMAXPROCS的数量大于CPU的数量。 GOMAXPROCS的最大值是256。

package main

import (  
    "fmt"
    "runtime"
)

func main() {  
    fmt.Println(runtime.GOMAXPROCS(-1)) //prints: 1
    fmt.Println(runtime.NumCPU())       //prints: 1 (on play.golang.org)
    runtime.GOMAXPROCS(20)
    fmt.Println(runtime.GOMAXPROCS(-1)) //prints: 20
    runtime.GOMAXPROCS(300)
    fmt.Println(runtime.GOMAXPROCS(-1)) //prints: 256
}

读写操做的重排顺序

level: advanced

Go可能会对某些操做进行从新排序，但它能保证在一个goroutine内的全部行为顺序是不变的。然而，它并不保证多goroutine的执行顺序。

package main

import (  
    "runtime"
    "time"
)

var _ = runtime.GOMAXPROCS(3)

var a, b int

func u1() {  
    a = 1
    b = 2
}

func u2() {  
    a = 3
    b = 4
}

func p() {  
    println(a)
    println(b)
}

func main() {  
    go u1()
    go u2()
    go p()
    time.Sleep(1 * time.Second)
}

若是你多运行几回上面的代码，你可能会发现 a和 b变量有多个不一样的组合：

a和 b最有趣的组合式是 "02"。这代表 b在 a以前更新了。

若是你须要在多goroutine内放置读写顺序的变化，你将须要使用channel，或者使用"sync"包构建合适的结构体。

优先调度

level: advanced

有可能会出现这种状况，一个无耻的goroutine阻止其余goroutine运行。当你有一个不让调度器运行的 for循环时，这就会发生。

package main

import "fmt"

func main() {  
    done := false

    go func(){
        done = true
    }()

    for !done {
    }
    fmt.Println("done!")
}

for循环并不须要是空的。只要它包含了不会触发调度执行的代码，就会发生这种问题。

调度器会在GC、“go”声明、阻塞channel操做、阻塞系统调用和lock操做后运行。它也会在非内联函数调用后执行。

package main

import "fmt"

func main() {  
    done := false

    go func(){
        done = true
    }()

    for !done {
        fmt.Println("not done!") //not inlined
    }
    fmt.Println("done!")
}

要想知道你在 for循环中调用的函数是不是内联的，你能够在“go build”或“go run”时传入“-m” gc标志（如， go build -gcflags -m）。

另外一个选择是显式的唤起调度器。你可使用“runtime”包中的 Goshed()函数。

package main

import (  
    "fmt"
    "runtime"
)

func main() {  
    done := false

    go func(){
        done = true
    }()

    for !done {
        runtime.Gosched()
    }
    fmt.Println("done!")
}

Go的50坑：新Golang开发者要注意的陷阱、技巧和常见错误[2]

目录

进阶篇

关闭HTTP的响应

关闭HTTP的链接

比较Structs, Arrays, Slices, and Maps

从Panic中恢复

在Slice, Array, and Map "range"语句中更新引用元素的值

在Slice中"隐藏"数据

Slice的数据“毁坏”

"走味的"Slices

类型声明和方法

从"for switch"和"for select"代码块中跳出

"for"声明中的迭代变量和闭包

Defer函数调用参数的求值

被Defer的函数调用执行

失败的类型断言

阻塞的Goroutine和资源泄露

高级篇

使用指针接收方法的值的实例

更新Map的值

"nil" Interfaces和"nil" Interfaces的值

栈和堆变量

GOMAXPROCS, 并发, 和并行

读写操做的重排顺序

优先调度