【问题标题】:Queue of structs in GoGo中的结构队列
【发布时间】:2017-04-13 14:43:20
【问题描述】:

我一直在学习围棋,我正在用它做一个 BFS 拼图。我决定的队列实现是:

//define the queue for our BFS algo as frontier.
type frontier struct {
    queue []*graphNode
}

func (F *frontier) buildFrontier() {
    F.queue = make([]*graphNode, 0, 1000)
}

func (F *frontier) pop() (g *graphNode) {
    if F.isEmpty() {
        panic("pop on empty queue!")
    }
    temp := F.queue[0]
    F.queue = F.queue[1:]
    return temp
}

func (F *frontier) push(g *graphNode) {
    F.queue = append(F.queue, g)
}

func (F *frontier) isEmpty() bool {
    return len(F.queue) == 0
}

我有两个问题:

  1. 这是一个好的实现吗? go 中关于队列的文档很少,通常有一些关于向量的旧帖子,并且列表似乎有太多开销(这对我来说并不重要,但我正在尝试以最好的方式做到这一点) .

  2. 为什么对对象(在本例中为结构指针 F *frontier)的调用必须是指针?似乎语法的工作方式应该是指针的默认值,而不是显式的(即为什么你不想在这些实例中使用指针?)

修改后的循环版本:

//define the queue for our BFS algo as frontier.
type frontier struct {
    queue                []*graphNode
    size, head, capacity int
}

func (F *frontier) buildFrontier() {
    F.capacity = 1
    F.queue = make([]*graphNode, F.capacity)
    F.size = 0
    F.head = 0
}

func (F *frontier) pop() (g *graphNode) {
    if F.isEmpty() {
        panic("pop on empty queue!")
    }
    F.size = (F.size - 1)
    temp := F.queue[F.head]
    F.head = (F.head + 1) % F.capacity
    return temp
}

func (F *frontier) push(g *graphNode) {
    if F.isFull() {
        newSlice := make([]*graphNode, F.capacity*2)
        copy(newSlice, F.queue)
        F.queue = newSlice
        F.capacity *= 2
    }
    F.queue[(F.head+F.size)%F.capacity] = g
    F.size = (F.size + 1)
}

func (F *frontier) isEmpty() bool {
    return F.size == 0
}
func (F *frontier) isFull() bool {
    return F.size == F.capacity
}

【问题讨论】:

标签: go struct queue


【解决方案1】:
  1. 看起来您的实现可以工作,但最终效率会有点低。一开始,queue 是一个容量为 1000 的切片。这意味着底层数组可以容纳 1000 个元素。每次调用pop 时,都会将队列的开头向下移动一个元素,从而将容量减少1。最终,当您调用push 时,容量将不足以容纳新元素,即使队列中可能没有很多(或任何)元素。这将导致在调用append 时分配一个新数组。无论pushpop 调用的模式是什么,数组都将被重复重新分配,即使它永远不会容纳接近1000 个元素。我建议为此使用一个链表,可以是 list 包中的链表,也可以是您自己设计的链表。

  2. 如果函数将修改接收者的值,或者如果接收者对象在其他地方使用并且必须共享值,则接收者需要是指针。否则,它不需要是指针。如果值很大,您可能希望接收器成为指针,以便不必复制它。 (接收器的行为与任何其他函数参数一样,适用相同的注意事项。)

【讨论】:

  • 谢谢安迪,这正是我希望有人能指出的那种错误,现在对我来说很明显......我已经修改了答案以包含一个自我扩展的循环实现,你会吗好心人也提供有关它的想法吗?
  • push 的工作方式看起来不正确。我认为新元素的索引应该是(F.head+F.size)%F.capacity。我只是快速浏览了一下,所以可能还有其他问题。这对于队列来说似乎过于复杂。链表会简单得多。
  • 你是对的关于推送,再次在上面修复。它通过了我的单元测试,没有足够的删除来测试环绕。谢谢。避免重新分配很复杂,虽然这里的上限为 1,但其目的是使其更大以避免以内存为代价的副本(不是问题的限制因素)。
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 2017-06-08
  • 1970-01-01
  • 2021-06-24
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2017-09-08
相关资源
最近更新 更多