请教一个 golang 问题

[ttxshang]

第一种写法

func (bus *es) PublicAll(topics []*EventTopic) {
	for _, topic := range topics {
		bus.mutex.Lock()
		h, ok := bus.handlers[topic.Topic]
		bus.mutex.Unlock()
		if !ok {
			continue
		}

		bus.client.Subscribe(topic.Topic, byte(topic.QoSValue), func(c mqtt.Client, m mqtt.Message) {
			go func(h EventHandler) {
				defer func() {
					if err := recover(); err != nil {
						fmt.Printf("goroutine panic: %+v \r\n", err)
					}

				}()
				h(topic.Topic, m)
			}(h)
		})
	}
}

第二种写法

func (bus *es) PublicAll(topics []*EventTopic) {
	for _, topic := range topics {
		bus.mutex.Lock()
		h, ok := bus.handlers[topic.Topic]
		bus.mutex.Unlock()
		if !ok {
			continue
		}

		go func(h EventHandler) {
			defer func() {
				if err := recover(); err != nil {
					fmt.Printf("goroutine panic: %+v \r\n", err)
				}

			}()
			bus.client.Subscribe(topic.Topic, byte(topic.QoSValue), func(c mqtt.Client, m mqtt.Message) {
				h(topic.Topic, m)
			})
		}(h)
	}
}

请问下上面两种写法，哪一种更好，为什么？除了以上方法还有其他更好的实现？感谢

[Trim21]

循环变量 topic 忘复制了吧？

[neoblackcap]

你这样写，bus 变量在 goroutine 里面运用，需不需要加锁啊？

[qwerqqq]

topic 的地址会变哦，你这么写有问题的，如果不考虑我觉得第二种效率可能要高一些

[yuancoder]

topic 变量只会在循环开始的时候声明一次，后面每次循环都只是修改这个变量的值。
循环里面闭包引用的 topic 都是同一个，值会被修改。

[hzzhzzdogee]

赞同 3 楼

[concernedz]

经典 for 循环 v 值变更，最后 topic 循环内需要重新声明下