编写地道的go代码

    xiaoxiao2021-03-25  190

    在阅读本文之前,我先推荐你阅读官方的 Effective Go文档,或者是中文翻译版: 高效Go编程,它提供了很多编写标准而高效的Go代码指导,本文不会再重复介绍这些内容。

    最地道的Go代码就是Go的标准库的代码,你有空的时候可以多看看Google的工程师是如何实现的。

    本文仅作为一个参考,如果你有好的建议和意见,欢迎添加评论。

    1 注释

    可以通过/* …… */或者// ……增加注释,//之后应该加一个空格。

    如果你想在每个文件中的头部加上注释,需要在版权注释和 Package前面加一个空行,否则版权注释会作为Package的注释。

    // Copyright 2009 The Go Authors. All rights reserved. // Use of this source code is governed by a BSD-style // license that can be found in the LICENSE file. /* Package net provides a portable interface for network I/O, including TCP/IP, UDP, domain name resolution, and Unix domain sockets. ...... */ package net ......

    注释应该用一个完整的句子,注释的第一个单词应该是要注释的指示符,以便在godoc中容易查找。

    注释应该以一个句点.结束。

    2 声明slice

    声明空的slice应该使用下面的格式:

    var t []string

    而不是这种格式:

    t := []string{}

    前者声明了一个nil slice而后者是一个长度为0的非nil的slice。

    3 关于字符串大小写

    错误字符串不应该大写。

    应该写成:

    fmt.Errorf(“failed to write data”)

    而不是写成:

    fmt.Errorf(“Failed to write data”)

    这是因为这些字符串可能和其它字符串相连接,组合后的字符串如果中间有大写字母开头的单词很突兀,除非这些首字母大写单词是固定使用的单词。

    缩写词必须保持一致,比如都大写URL或者小写url。比如HTTP、ID等。 例如 sendOAuth 或者 oauthSend 。

    常量一般声明为MaxLength,而不是以下划线分隔MAX_LENGTH或者MAXLENGTH。

    也就是Go语言一般使用 MixedCaps 或者 mixedCaps 命名的方式区分包含多个单词的名称。

    4 处理error而不是panic或者忽略

    为了编写强壮的代码,不用使用_忽略错误,而是要处理每一个错误,尽管代码写起来可能有些繁琐。

    尽量不要使用panic。

    5 尽量减少代码缩进

    6 一些名称

    有些单词可能有多种写法,在项目中应该保持一致,比如Golang采用的写法:

    // marshaling // unmarshaling // canceling // cancelation

    而不是

    // marshalling // unmarshalling // cancelling // cancellation

    包名应该用单数的形式,比如util、model,而不是utils、models。

    Receiver 的名称应该缩写,一般使用一个或者两个字符作为 Receiver 的名称,如

    func (f foo) method() { ... }

    如果方法中没有使用 receiver ,还可以省略 receiver name ,这样更清晰的表明方法中没有使用它:

    func (foo) method() { ... }

    7 package级的Error变量

    通常会把自定义的 Error 放在 package 级别中,统一进行维护:

    var ( ErrCacheMiss = errors.New("memcache: cache miss") ErrCASConflict = errors.New("memcache: compare-and-swap conflict") ErrNotStored = errors.New("memcache: item not stored") ErrServerError = errors.New("memcache: server error") ErrNoStats = errors.New("memcache: no statistics available") ErrMalformedKey = errors.New("malformed: key is too long or contains invalid characters") ErrNoServers = errors.New("memcache: no servers configured or available") )

    并且变量以Err开头。

    8 空字符串检查

    不要使用下面的方式检查空字符串:

    if len(s) == 0 { ... }

    而是使用下面的方式

    if s == "" { ... }

    下面的方法更是语法不对:

    if s == nil || s == "" { ... }

    9 非空slice检查

    不要使用下面的方式检查空的slice:

    if s != nil && len(s) > 0 { ... }

    直接比较长度即可:

    if len(s) > 0 { ... }

    同样的道理也适用 map 和 channel 。

    10 省略不必要的变量

    比如

    var whitespaceRegex, _ = regexp.Compile("\\s+")

    可以简写为

    var whitespaceRegex = regexp.MustCompile(\s+)

    有时候你看到的一些第三方的类提供了类似的方法:

    func Foo(...) (...,error) func MustFoo(...) (...)

    MustFoo 一般提供了一个不带 error 返回的类型。

    11 直接使用bool值

    对于bool类型的变量var b bool,直接使用它作为判断条件,而不是使用它和true/false进行比较

    if b { ... } if !b { ... }

    而不是

    if b == true { ... } if b == false { ... }

    12 byte/string slice相等性比较

    不要使用

    var s1 []byte var s2 []byte ... bytes.Compare(s1, s2) == 0 bytes.Compare(s1, s2) != 0

    而是:

    var s1 []byte var s2 []byte ... bytes.Equal(s1, s2) == 0 bytes.Equal(s1, s2) != 0

    13 检查是否包含子字符串

    不要使用 strings.IndexRune(s1, ‘x’) > -1 及其类似的方法 IndexAny 、Index 检查字符串包含,而是使用 strings.ContainsRune 、strings.ContainsAny 、strings.Contains 来检查。

    14 使用类型转换而不是struct字面值

    对于两个类型:

    type t1 struct { a int b int } type t2 struct { a int b int }

    可以使用类型转换将类型t1的变量转换成类型t2的变量,而不是像下面的代码进行转换

    v1 := t1{1, 2} _ = t2{v1.a, v1.b} 应该使用类型转换,因为这两个struct底层的数据结构是一致的。

    _ = t2(v1)

    15 复制slice

    不要使用下面的复制slice的方式:

    var b1, b2 []byte for i, v := range b1 { b2[i] = v } for i := range b1 { b2[i] = b1[i] }

    而是使用内建的copy函数:

    copy(b2, b1)

    16 不要在for中使用多此一举的true

    不要这样:

    for true { }

    而是要这样:

    for { }

    17 尽量缩短if

    下面的代码:

    x := true if x { return true } return false

    可以用return x代替。

    同样下面的代码也可以使用 return err 代替:

    func fn1() error { var err error if err != nil { return err } return nil } func fn1() bool{ ... b := fn() if b { ... return true } else { return false } }

    应该写成:

    func fn1() bool{ ... b := fn() if !b { return false } ... return true }

    也就是减少if的分支/缩进。

    不要这样:

    var a, b []int for _, v := range a { b = append(b, v) }

    而是要这样

    var a, b []int b = append(b, a...)

    18 简化range

    var m map[string]int for _ = range m { } for _, _ = range m { }

    可以简化为

    for range m { }

    对 slice 和 channel 也适用。

    19 正则表达式中使用 raw 字符串避免转义字符

    在使用正则表达式时,不要:

    regexp.MustCompile("\\.") regexp.Compile("\\.")

    而是直接使用 raw 字符串,可以避免大量的 \ 出现:

    regexp.MustCompile(`\.`) regexp.Compile(`\.`)

    20 简化只包含单个case的select

    select { case <-ch: }

    直接写成<-ch即可。send也一样。

    for { select { case x := <-ch: _ = x } }

    直接改成for-range即可。

    这种简化只适用包含单个case的情况。

    21 slice的索引

    有时可以忽略slice的第一个索引或者第二个索引:

    var s []int _ = s[:len(s)] _ = s[0:len(s)]

    可以写成 s[:]

    22 使用 time.Since

    下面的代码经常会用到:

    _ = time.Now().Sub(t1)

    可以简写为:

    _ = time.Since(t1)

    23 使用 strings.TrimPrefix/strings.TrimSuffix 掐头去尾

    不要自己判断字符串是否以XXX开头或者结尾,然后自己再去掉XXX,而是使用现成的 strings.TrimPrefix/strings.TrimSuffix 。

    var s1 = "a string value" var s2 = "a " var s3 string if strings.HasPrefix(s1, s2) { s3 = s1[len(s2):] }

    可以简化为

    var s1 = "a string value" var s2 = "a " var s3 = strings.TrimPrefix(s1, s2)

    24 使用工具检查你的代码

    以上的很多优化规则都可以通过工具检查,下面列出了一些有用的工具:

    go fmt go vet gosimple keyify staticcheck unused golint misspell goimports errcheck 原文链接:http://colobu.com/2017/02/07/write-idiomatic-golang-codes/

    转载请注明原文地址: https://ju.6miu.com/read-2710.html

    最新回复(0)