注册 登录  
 加关注
   显示下一条  |  关闭
温馨提示!由于新浪微博认证机制调整,您的新浪微博帐号绑定已过期,请重新绑定!立即重新绑定新浪微博》  |  关闭

snoopyxdy的博客

https://github.com/DoubleSpout

 
 
 

日志

 
 

golang几个注意点(转载)  

2016-09-18 12:01:33|  分类: golang |  标签: |举报 |字号 订阅

  下载LOFTER 我的照片书  |


译者注:今天看到这个文章,感觉非常有用,我们平常写代码中也是经常遇到,所以翻译一下,让大家平常写Go代码的时候引起关注,同时在翻译的过程中也加入了一些自己的理解注释。


Go中有三种最容易犯的错误,我将以最常见的方式来展现而不是最容易理解的方式.

这三种错误都是在Kubernetes的代码中出现过的,这个之前在code review的时候最长见到

循环中的变量逃逸出循环的.

下面这些代码会输出什么呢?大家先预测一下然后再往下翻滚看答案

func print(pi *int) { 
fmt.Println(*pi)
}
for i := 0; i < 10; i++ {
   defer fmt.Println(i)
   defer func(){ fmt.Println(i) }()
   defer func(i int){ fmt.Println(i) }(i)
   defer print(&i)
   go fmt.Println(i)
   go func(){ fmt.Println(i) }() }

答案是:

func print(pi *int) { 
fmt.Println(*pi)
}
for i := 0; i < 10; i++ {
   defer fmt.Println(i) // 预期的结果; 打印 9 ... 0 defer func(){ fmt.Println(i) }() // 错误的结果; 打印10个"10" defer func(i int){ fmt.Println(i) }(i) // 正确的预期结果 defer print(&i) // 错误的结果; 打印10个"10" go fmt.Println(i) // 正确; 但是打印 0-9之间的顺序不保证 go func(){ fmt.Println(i) }() // 错误; 完全不可预知.
}
for key, value := range myMap {  
   
// Same for key & value as i!
}


每个人期望这些值和循环内的值相同,但是Go重用了每次循环的变量. 这就意味着你绝对不要让key,value,i这些值的地址逃逸出循环。使用匿名函数 de style="margin: 0px; padding: 0px; max-width: 100%; box-sizing: border-box; word-wrap: break-word !important;" >func() { /* do something with i */ }de> (a "closure") 是一个微妙的解决地址逃逸问题的方法,因为在Go里面函数调用,这个参数是值copy。

Nil interface 如果有nil 指针

type Cat interface {  
   Meow() }
type Tabby struct {}
func (*Tabby) Meow() {
fmt.Println("meow")
}
func GetACat() Cat {
   var myTabby *Tabby = nil // Oops, we forgot to set myTabby to a real value return myTabby }
func TestGetACat(t *testing.T) {
   if GetACat() == nil { t.Errorf("Forgot to return a real cat!") } }

大家猜猜会怎么样? 上面的这个测试讲永远不会检测到空指针。这是因为interface作为一个指针的容器来使用了,因此GetACat可以返回一个指针或者空指针. 所以最好不要像GetACat这样来写代码,让你和你的小伙伴们都感到开心。而且这种错误在err里面也是经常见.详见 http://golang.org/doc/faq#nil_error

影子变量.

var ErrDidNotWork = errors.New("did not work")
func DoTheThing(reallyDoIt bool) (err error) {
   if reallyDoIt {
      result, err := tryTheThing()
      if err != nil || result != "it worked" { err = ErrDidNotWork } }
  return err }

上面的这个代码将永远返回nil的错误, 因为循环内的err变量是一个函数返回err变量的影子变量. 所以上面的代码解决方案是添加一个 de style="margin: 0px; padding: 0px; max-width: 100%; box-sizing: border-box; word-wrap: break-word !important;" >var result stringde> 不要在内部使用de style="margin: 0px; padding: 0px; max-width: 100%; box-sizing: border-box; word-wrap: break-word !important;" >:=de> 来什么新变量.

  评论这张
 
阅读(101)| 评论(0)
推荐 转载

历史上的今天

在LOFTER的更多文章

评论

<#--最新日志,群博日志--> <#--推荐日志--> <#--引用记录--> <#--博主推荐--> <#--随机阅读--> <#--首页推荐--> <#--历史上的今天--> <#--被推荐日志--> <#--上一篇,下一篇--> <#-- 热度 --> <#-- 网易新闻广告 --> <#--右边模块结构--> <#--评论模块结构--> <#--引用模块结构--> <#--博主发起的投票-->
 
 
 
 
 
 
 
 
 
 
 
 
 
 

页脚

网易公司版权所有 ©1997-2016