您的位置:首页 > 编程语言 > Go语言

[go] binary.Write 小坑一个兼论go的错误处理哲学

2018-01-23 18:16 330 查看
有如下go代码:

const (
foo = 123
)
buffer := new(bytes.Buffer)
binary.Write(buffer, binary.BigEndian, foo)
fmt.Println(buffer.Len())


输出结果是0,foo没有写入到buffer中。原因在于,binary包的Write内部调用了intDataSize函数获取foo的长度:

func intDataSize(data interface{}) int {
switch data := data.(type) {
case bool, int8, uint8, *bool, *int8, *uint8:
return 1
case []int8:
return len(data)
case []uint8:
return len(data)
case int16, uint16, *int16, *uint16:
return 2
case []int16:
return 2 * len(data)
case []uint16:
return 2 * len(data)
case int32, uint32, *int32, *uint32:
return 4
case []int32:
return 4 * len(data)
case []uint32:
return 4 * len(data)
case int64, uint64, *int64, *uint64:
return 8
case []int64:
return 8 * len(data)
case []uint64:
return 8 * len(data)
}
return 0
}


而foo的类型是int,intDataSize函数不处理int类型,因此返回0。导致没有数据被写入。

当然,这是我用错了,因为binary包的文档里面写了:

Numbers are translated by reading and writing fixed-size values.

A fixed-size value is either a fixed-size arithmetic

type (bool, int8, uint8, int16, float32, complex64, …)

or an array or struct containing only fixed-size values.

也就是说,使用binary包时,必须使用明确的长度确定的类型,可以用int32,但别用int。

没看清除文档,用错了,怨不得别人。但是binary包把错误直接吞掉的做法,似乎和go的错误处理哲学背离了啊。

intDataSize 函数对于他不能处理的类型,难道不能返回一个error么?或者粗暴些直接给个panic也行啊。这样至少能快速定位错误吧。否则对于不习惯看文档,只是看一下方法的声明就想快速使用的人,这种坑掉进去要爬出来是颇费一些工夫的。
内容来自用户分享和网络整理,不保证内容的准确性,如有侵权内容,可联系管理员处理 点击这里给我发消息
标签: