首页   注册   登录
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
The Go Programming Language
http://golang.org/
Go Playground
Go Projects
Revel Web Framework
beego
OPPO Watch
yujianwjj
V2EX  ›  Go

go 标准库 container/heap 的一个疑问

  •  
  •   yujianwjj · 61 天前 · 992 次点击
    这是一个创建于 61 天前的主题,其中的信息可能已经有所发展或是发生改变。

    container/heap 中的 Pop 方法,从堆当中移出一个元素,如果当前堆为空,那么调用 heap.Pop() 方法就会报错。

    panic: runtime error: index out of range [0] with length 0
    

    代码中确实没有对长度为 0 时的情况进行特殊处理。

    // Pop removes and returns the minimum element (according to Less) from the heap.
    // The complexity is O(log n) where n = h.Len().
    // Pop is equivalent to Remove(h, 0).
    func Pop(h Interface) interface{} {
    	n := h.Len() - 1
    	h.Swap(0, n)
    	down(h, 0, n)
    	return h.Pop()
    }
    

    我的疑问是为什么对这个情况不做处理?

    2 条回复    2020-03-26 01:51:07 +08:00
    rrfeng
        1
    rrfeng   61 天前
    容器有没有元素肯定要自己判断呀,不然给你返回啥?

    1. pop 返回 nil (???
    2. pop 返回元素+error (???
    你觉得哪种好?
    123444a
        2
    123444a   61 天前 via Android
    楼主欢迎你加入中国人改造 go 社区,我们一起修改这个接口吧,返回值加入 error 即可,就为了跟英语世界不一样改它
    关于   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   1344 人在线   最高记录 5168   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 29ms · UTC 18:31 · PVG 02:31 · LAX 11:31 · JFK 14:31
    ♥ Do have faith in what you're doing.