Golang 最佳实践
目录
命名
避免重复
- 函数或方法名中可以省略如下字段:
- 输入输出类型
- 方法接收器类型
- 输入输出是否为指针
- 包的名字
- 方法接收器的名字
- 参数传递的变量名称
- 返回值类型的名称和类型
命名惯例
- 返回一个对象,一般以对象名命名(避免加Get)
- 动作一般会加动作前缀
- 如果涉及到不同的类型,会把类型放到函数末尾
- 函数主版本,则可以省略类型
测试助手包
- 原包名 + test
- 简单测试,桩直接使用Stub 即可
- 多个测试桩,按照测试行为命名
- 多个类型,多个测试桩,需要明确,类型+Stub,方法要明确
- 测试中局部变量,命名要能区分出真实、测试类型
遮蔽
- 避免作用域遮蔽原有的上下文变量;而在作用域外又重新使用的情况
- 不使用和标准库的包名相同的变量
Util 包
- 不使用 util、helper、common 类似的包名
- 考虑调用时如何使用
包的大小
- 文件应该足够内聚,以便维护者可以知道哪个文件包含了什么功能
- 文件应该足够小,以便一旦有了它,就很容易找到
导入
proto,stub文件导入
- 包重命名为 pb,
- 若多个包,加前缀
导入顺序:
- 标准库导入;
- 普通项目导入;
- pb等导入;
- 副作用导入;
- 分组导入
错误处理
错误消费端:
- 诊断信息,返回给人类;
- 维护者使用;
- 终端用户解释
错误定义:
- 是否需要结构;
- 考虑添加你拥有但调用者、被调用者没有的信息
使用errgroup;
错误值定义:
- 哨兵模式,直接判等;
- errors.Is() 方式包含error;
- 结构化error,提供status字段
错误信息,不冗余;
- 适当添加额外的信息使错误信息更有用;
- 一般使用 fmt.Errorf(“xxx %w”) %w 会包装参数中的err;需要提供细节的err 一般会这么使用;
- 链式,一次只能wrap一个err
- 屏蔽细节,使用 fmt.Errorf(“xxx %v”)
日志中输出错误
- 敏感数据问题
- 尽量少使用log.Error
程序初始化
- 初始化错误,应该传播到main,调用 log.Exit 退出,解释如何修复
程序检查和panic;
- 尽量少用panic,倾向于返回错误,而不是终止程序;
- panic的传导可能会影响上下文状态;
何时用panic,
- 对api的误用,报panic;
- 调用链中有一个对应的recover,确保panic 不跨越包。
- 命令后解析之前,不调用日志函数
文档
参数与配置:
- 不是每个参数都需要文档
只描述易出错和不明显的字段和参数
context
- 惯例:取消操作,返回ctx.Err(),不需要注释;
- 例外情况进行注释
是否支持并发的注释
清理,标记是否有后续的清理
godoc 格式化
- 段落之间空行
- 测试文件包含可运行代码
- 缩进行增加两个空格,格式化代码
- 可运行的代码,更好的方式是提供Example Case,而非注释
- 一行以大写字母开始,除括号和逗号外不含标点符号,后面是另一个段落,这样的行将被格式化为标题
经常使用的 err != nil ,如果判断 err==nil 增加注释强调
变量申明
初始化,非零值初始化,尽量使用 :=
非指针零值
- 用申明而不是赋值
- 锁不可复制的结构体字段,使用值类类型,用零值初始化,指针传递
- 复合类型,函数返回,最好返回指针
复合字面值
- 已知的参数,可以用复合字面值声明值
尺寸提示
- 备注,提示容量内容。比如slice,map等
- 性能敏感型尤为注意
channel
- 尽可能指向channel方向,防止误操作
函数参数列表
- 减少参数的数量,抽离出多个函数,或者使用一个未导出的实现
- 参数过多,可以使用一个struct作为入参,或者variadic 技术
不定长参数 variadic
复杂的命令行交互界面
- cobra
- subcommands
测试
把测试留给Test 函数
- 测试case 正交
- 数据类似,使用表驱动的测试
- 在Test函数中内联逻辑