標準的な Go コードを作成する

コードの一部を直接見てみましょう

type MyselfError struct{}

func (m *MyselfError) Error() string {
 return "实现 error 接口的 Error 方法"
}

func someWork() *MyselfError {
 return nil
}

func main() {
 var err error
 err = someWork()
 fmt.Println(err == nil)
}

// output: false

この例の出力を見て驚くかもしれません?

これは、Go では 2 つの nil の比較が等しくない可能性があるためです。記事 Go 言語の型の比較 # では、次のように述べました。 ==<span style="font-size: 15px;"></span> インターフェイスの比較は、型と値が等しい場合にのみ true になります。

type error interface {
    Error() string
}

someWork<span style="font-size: 15px;"></span>関数によって返されたエラーは、タイプ MyselfError および値 nil のエラー インターフェイスであり、明らかに要件を満たしていません。 : 型と値が両方とも nil の場合のみ、インターフェース型の nil 判定が true になります。

在 Go 中，不要通过<span style="font-size: 15px;">err == nil</span>来做逻辑判断条件。这不光是由于使用它会产生潜在的 bug，这样的代码交于测试童鞋，他们可能也会喷你，你知道是为什么吗？

我们可以把代码分为主干代码和分支代码，主干代码代表正常逻辑，分支代码记录异常case。两者最简单的区分方法就是：在一个函数中，主干代码与最左侧只隔一个 tab 距离，超过一个 tab 距离的为分支代码。

在处理错误返回的函数中，我们应该先做错误异常的处理，错误处理的逻辑属于分支代码，而正常逻辑则应在主干代码上。

错误示例

func bar() {
 var err error
 err = foo()
 if err == nil {
  // 程序正常的代码逻辑
 } else {
  switch err.(type) {
  case err1:
   // 做错误处理1
  case err2:
   // 做错误处理2
  default:
   // 做通用错误处理
  }
 }
}

现在你能知道测试童鞋为什么喷你吗？

有一个词叫做测试覆盖率，它代表测试用例走过的代码行数。如果你将<span style="font-size: 15px;">err==nil</span>的判断前置，那这段代码就对于测试不友好。

在测试过程中，有时我们很难人为构造错误的发生，那么很可能测试用例只会走<span style="font-size: 15px;">err==nil</span>下面的代码逻辑。

规范示例

func main() {
 var err error
 err = foo()
 if err != nil {
  switch err.(type) {
  case err1:
   // 做错误处理1
  case err2:
   // 做错误处理2
  default:
   // 做通用错误处理
  }
 }
 // 程序正常的代码逻辑
}

这样的代码规范，让我们在初次接手新项目，或者 code review 其他人的代码时，能够通过阅读主干代码而快速理解地代码业务逻辑，而不至于陷入琐碎的 case 处理中。

总结

今天的文章虽然很短，但是希望能给大家带来启示。

在 Go 中 err == nil 不需要判断，而该判断异常 case，正常逻辑置于主干，异常代码置于分支。

開発チーム内で適切なコーディング標準セットを確立すると、コードの読みやすさと共同作業の効率が向上します。同様の仕様をまだ持っていない場合は、Go コード レビュー コメントと uber-go/guide を参照して完全な仕様を作成してください。