aceld / aceld/zinx

RouterSlices中Handlers的意义还存在逻辑错误

Offen
#291 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Go
Sterne
7.8k
Forks
1.3k
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

```ruby
func (r *RouterSlices) AddHandler(msgId uint32, Handlers ...ziface.RouterHandler) {
// 1. Check if the API handler method bound to the current msg already exists
if _, ok := r.Apis[msgId]; ok {
panic("repeated api , msgId = " + strconv.Itoa(int(msgId)))
}

finalSize := len(r.Handlers) + len(Handlers)
mergedHandlers := make([]ziface.RouterHandler, finalSize)
copy(mergedHandlers, r.Handlers)
copy(mergedHandlers[len(r.Handlers):], Handlers)
r.Apis[msgId] = append(r.Apis[msgId], mergedHandlers...)
}
```

```ruby
type RouterSlices struct {
Apis map[uint32][]ziface.RouterHandler
Handlers []ziface.RouterHandler
sync.RWMutex
}
```

我不理解这个结构体中Handlers的意义,我的理解应该是每一个msgid有一个自己的handler切片,但是可以添加handler,为什么要判断当前msgid是否在map中存在,此外为什么新的msgID需要加上原来的r.Handlers,还有是不是需要加一个写锁,防止并发安全的问题,望解答!!!

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.