Как работать с конкурентным доступом к одной ячейке памяти?

Question

[UntitledNikname]

Маленький чат (для самообучения) на GO. В пакете "server" есть "map[string]net.Conn" где хранятся все активные каналы.
Пустой канал я удаляю из памяти сразу же как оттуда выйдет последний пользователь.
В ходе тестирования заметил что 2 разных потока.. один пытается удалить пустой канал а второй пытается записать туда новое соединение.
Не понятно как с этим правильно работать..

Comments

[Евгений Самсонов]

Под удалить подразумевается закрыть канал?

[Евгений Самсонов]

Покажите лучше код, где у вас возникает проблема

[UntitledNikname]

Евгений Самсонов, делаю прямо delete(rooms, "testRoom");

[Евгений Самсонов]

UntitledNikname, места, где работаете с map, оберните мьютексом из пакета sync

[UntitledNikname]

Евгений Самсонов, Для обучения взял прямо этот гит проект тестового tcp сервера. Проблема возникла когда в func (s *server) quit(c *client) {..} добавил проверку сколько народу осталось в чате, если ноль то удаляю через delete(...)
GIT REPO

[Евгений Самсонов]

UntitledNikname, скинул ссылку в ответе на пример с map и sync.Mutex

[UntitledNikname]

Евгений Самсонов, пробовал. Ставил

mutex.Lock()
mutex.Unlock()
....

в func (s *server) quit(c *client) {..}, в итоге принимает только 1 соединение и глохнет. Ни отключиться ни что то сделать нельзя

[Евгений Самсонов]

UntitledNikname, должно быть так

mutex.Lock()
delete(rooms, "testRoom")
mutex.Unlock()

И места, где читаете из map тоже блокируйте.

P. S. Если у вас часто чтение и редко запись, то лучше будет использовать sync.RWMutex. Тогда места где чтение блокируйте с помощью RLock, а места изменения Lock

Answer 1

[Евгений Самсонов]

Используйте мьютексы при конкурентной работе c map
https://gobyexample.com/mutexes

Answer 2

[Klars]

Mutexes
Наглядный график с кодом https://golangbot.com/mutex/
+ обязательно для ридеров и врайтеров использовать RLock and RUnlock (блокировка на чтение значения или на запись)
очень полезно