本文轉(zhuǎn)載自微信公眾號(hào)「Golang技術(shù)分享」，作者機(jī)器鈴砍菜刀。轉(zhuǎn)載本文請(qǐng)聯(lián)系Golang技術(shù)分享公眾號(hào)。

在公司進(jìn)行代碼開發(fā)，一般都會(huì)制定一套編程規(guī)范。良好的代碼規(guī)范可以改善項(xiàng)目可讀性，提高團(tuán)隊(duì)開發(fā)的合作效率。具體在 Go 語言中，我們可以借鑒 Go 官方的 Go Code Review Comments、Uber 開源的 uber-go/guide 項(xiàng)目，大家感興趣可以去學(xué)習(xí)。

本文我們聚焦于一個(gè)點(diǎn)：Go 的 error 判斷。

啟示代碼

我們直接看一段代碼

type MyselfError struct{} 
 
func (m *MyselfError) Error() string { 
 return "實(shí)現(xiàn) error 接口的 Error 方法" 
} 
 
func someWork() *MyselfError { 
 return nil 
} 
 
func main() { 
 var err error 
 err = someWork() 
 fmt.Println(err == nil) 
} 
 
// output: false 

這個(gè)例子的輸出可能會(huì)讓你感到意外?

這是由于在 Go 中，兩個(gè) nil 的比較也許并不相等。在Go 語言類型可比性一文中我們說過：對(duì)于接口 interface 而言，它的比較存在兩個(gè)維度，分別是動(dòng)態(tài)類型和動(dòng)態(tài)值。接口的==比較，只有在類型與值均相等的情況下才會(huì)為真。

type error interface { 
    Error() string 
} 

someWork函數(shù)返回的 err 它是類型為 MyselfError，值為 nil 的 error 接口，顯然不滿足要求：只有類型和值同時(shí)都為 nil 的情況下，接口類型的 nil 判斷才會(huì)為真。

主分支代碼

有了上面的鋪墊，你應(yīng)該懂我要說什么了吧?

在 Go 中，不要通過err == nil來做邏輯判斷條件。這不光是由于使用它會(huì)產(chǎn)生潛在的 bug，這樣的代碼交于測(cè)試童鞋，他們可能也會(huì)噴你，你知道是為什么嗎?

我們可以把代碼分為主干代碼和分支代碼，主干代碼代表正常邏輯，分支代碼記錄異常case。兩者最簡(jiǎn)單的區(qū)分方法就是：在一個(gè)函數(shù)中，主干代碼與最左側(cè)只隔一個(gè) tab 距離，超過一個(gè) tab 距離的為分支代碼。

在處理錯(cuò)誤返回的函數(shù)中，我們應(yīng)該先做錯(cuò)誤異常的處理，錯(cuò)誤處理的邏輯屬于分支代碼，而正常邏輯則應(yīng)在主干代碼上。

錯(cuò)誤示例

func bar() { 
 var err error 
 err = foo() 
 if err == nil { 
  // 程序正常的代碼邏輯 
 } else { 
  switch err.(type) { 
  case err1: 
   // 做錯(cuò)誤處理1 
  case err2: 
   // 做錯(cuò)誤處理2 
  default: 
   // 做通用錯(cuò)誤處理 
  } 
 } 
} 

現(xiàn)在你能知道測(cè)試童鞋為什么噴你嗎?

有一個(gè)詞叫做測(cè)試覆蓋率，它代表測(cè)試用例走過的代碼行數(shù)。如果你將err==nil的判斷前置，那這段代碼就對(duì)于測(cè)試不友好。

在測(cè)試過程中，有時(shí)我們很難人為構(gòu)造錯(cuò)誤的發(fā)生，那么很可能測(cè)試用例只會(huì)走err==nil下面的代碼邏輯。

規(guī)范示例

func main() { 
 var err error 
 err = foo() 
 if err != nil { 
  switch err.(type) { 
  case err1: 
   // 做錯(cuò)誤處理1 
  case err2: 
   // 做錯(cuò)誤處理2 
  default: 
   // 做通用錯(cuò)誤處理 
  } 
 } 
 // 程序正常的代碼邏輯 
} 

這樣的代碼規(guī)范，讓我們?cè)诔醮谓邮中马?xiàng)目，或者 code review 其他人的代碼時(shí)，能夠通過閱讀主干代碼而快速理解地代碼業(yè)務(wù)邏輯，而不至于陷入瑣碎的 case 處理中。

總結(jié)

今天的文章雖然很短，但是希望能給大家?guī)韱⑹尽?/p>

在 Go 中 err == nil 不需要判斷，而該判斷異常 case，正常邏輯置于主干，異常代碼置于分支。

在開發(fā)組內(nèi)建立起一套良好的代碼規(guī)范，會(huì)有助于提升代碼可讀性以及工作協(xié)作效率。如果你們還沒有類似的規(guī)范，那就去參考 Go Code Review Comments、 uber-go/guide 來整活一套?

參考

Go Code Review Comments：https://github.com/golang/go/wiki/CodeReviewComments

uber-go/guide：https://github.com/uber-go/guide