最近我看到Golang社区有一个非常有意思的提议,想要给Golang增加一个内置的clear()函数,来清除map、无内容的slice、指向array的指针等:proposal: spec: add clear(x) builtin, to clear map, zero content of slice, ptr-to-array #56351。在golang中清除一个map的所有元素,目前只有通过for循环+delete的方式,这确实不太优雅。
for k := range m {
delete(m, k)
}
对于map来说,for循环+delete的方式还有一个缺点,即map中的 空bucket 不会被删除,仍然会占用部分空间。具体原因可以自行搜索,本文不做详细介绍。
但其实,delete函数还存在一个小问题,而这个小问题让 delete函数的正确性 受到影响。我们看看下面这段代码:
package main
import (
"fmt"
"math"
)
func main() {
m := make(map[float64]string)
m[math.NaN()] = "Yes"
for k := range m {
delete(m, k)
}
fmt.Printf("Map has length %d, contents: %#v\n", len(m), m)
// 输出:Map has length 1, contents: map[float64]string{NaN:"Yes"}
}
感兴趣的读者可以自己跑一次试试。很明显,这里的代码运行结果与我们的预期不一致!如果map中存在以浮点数NaN为key的元素,你无法删除它。 造成这一问题的原因在于,Golang对浮点数的比较操作遵从IEEE754标准 Floating-point values are comparable and ordered, as defined by the IEEE-754 standard. 在这一标准下,NaN与任何浮点数都不相等,并且两个NaN之间也不相等。
而delete(m, k)函数需要通过对比值,删除与k相等的元素,如果m中没有与k相等的元素,delete函数什么也不做。所以,delete函数无法删除map中将NaN作为key的元素(即使你可以用range来迭代出它)
我在读到这一提案之前,并不知道NaN带来的问题,对此我也十分惊讶。增加一个内置的clear函数,固然能够提升代码效率,但更重要的是修正一个遗留的代码正确性问题,希望能在go 1.21版本中看到这一改进吧!
网友评论