Practical Advice for Go Library Authorsby Jack Lindamoodat GopherCon 2016
https://www.youtube.com/watch?v=5v2fqm_8jYI
幻灯地址:http://go-talks.appspot.com/github.com/cep21/go-talks/practical-advice-for-go-library-authors.slide#1
命名包名是将来使用过程中的一部分,所以避免重复包名和结构与函数。比如
1
var h client.Client → var h http.Client
1
context.NewContext() => context.Background()
Object Creation
golang 没有构造函数,因此创建对象一般有两种办法:
默认的0值
单独的构造函数,NewSomething()
推荐使用默认 0 值的构造方法
在默认0值的情况下,各个方法要处理好0值,比如有些东西发现是0值后,给入一个默认值。
New() 构造函数很灵活,可以做任何事情,因此对于代码阅读上不利,意味着隐藏了很多东西。
有些库使用私有 struct,公开接口的方法,authImpl struct and Auth interface,这是反模式,不推荐使用。
不推荐使用 Singleton,虽然标准库中大量使用了 Singleton 模式,但是 Jack 个人不喜欢这种模式。
使用高阶函数作为选项这种形式不推荐:NewSomething(WithThingA(), WithThingB())
日志一些日志是直接打印到标准输出去,这是非常不好的设计,因为用户如果想关根本关不了。
建议
确定一下作为库是不是真的需要打印日志,是不是应该把输出日志的工作交给调用方决定?
如果一定需要日志,那么使用回调函数方式
输出日志到一个 interface
不要假定传进来的就是标准库的 log ,有很多选择。
尊重 stdout 和 stderr
不要使用 singleton
接受 interface ,但返回的是 struct
这点和 Java 不同,Java 更倾向于所有东西都是通过 interface 操作。而 golang 不需要,golang 使用的是隐性interface。
什么时候 panic最好都不 panic。如果非要 panic,可能最合适的地方是 init 的时候,因为刚一运行就能看到挂了,比较容易处理。但即使如此,也尽量不要 panic。
检查 error问:我们是需要检查所有的 error 么?比如有些似乎不大容易出错。答:需要,特别是你说的这些不大容易出错的!!
我们用 error 代替了 exception,所以不要忽略这个东西。
处理的办法
最好的办法是 Bubble up,也就是传回调用方
但有的时候(比如 goroutine) 不适合,那就:
做日志
或者增加某个计数器
什么时候应该返回错误比较合适?
当不满足约定
当需要的答案无法得到
为了方便自己测试
为了方便库用户测试
虽然 channel 是 golang 一个处理并发很好地东西,但是并非所有场合都需要。比如标准库中就很少有在 API 中使用 channel 的。
将使用 channel 的位置向上层移动。
可以使用回调函数。
不要混合使用 mutex 和 channel
有一些库的 New() 会发起他们的 goroutine,这是不好的。
标准库使用的是 Serve() 函数。以及对应的 Close() 函数
将 goroutine 向上层推
所有的阻塞、长时间的操作,都应该可以被 cancel
由于 context.Context 很容易存储东西,所以很容易被滥用。要尽力去避免使用 Context
Singleton 和 context.Value() 是同样性质的东西,像全局变量一样,对于程序状态来说是个黑箱。
如果什么东西很难做,嗯,那就让别人去做吧
为了效率而升级
但是,正确性要比效率重要,在正确性的前提下,注意效率
不要在库中使用 /vendor (在 main 包中可以)
注意 build tag
保持干净
尽量使用所有的静态分析工具来检查代码。
https://blog.lab99.org/post/golang-2017-09-21-video-practice-advice-for-go-library-authors.html
< END >
喜欢就点个在看 or 转发个朋友圈呗
衣舞晨风