Как побороть дублирование кода?

Question

[Владимир Грабко]

Есть хелперы для роботы с глобальной картой (работать с ней возможно только через них). Вот делал но код дублируется чуть менее чем везде. Как можно исправить это дело?

func Get(id string) (Sessid, bool) {
	rw := &sync.RWMutex{}
	rw.RLock()
	defer rw.RUnlock()
	val, ok := stSessid[id].Storage
	return val, ok
}

func Set(id string, value Sessid) error {
	_, dublicate := ramGet(id)
	if dublicate {
		return errors.New("Такой ключ уже есть")
	}
	r := &sync.Mutex{}
	r.Lock()
	defer r.Unlock()
	stSessid[id].Storage = value
}

func Del(id string) error {
	_, dublicate := ramGet(id)
	if !dublicate {
		return errors.New("Ключа нету")
	}
	r := &sync.Mutex{}
	r.Lock()
	defer r.Unlock()
	delete(stSessid, id)
}
func Up(id string, value Sessid) error {
	_, dublicate := ramGet(id)
	if !dublicate {
		return errors.New("Ключа нету")
	}
	r := &sync.Mutex{}
	r.Lock()
	defer r.Unlock()
	stSessid[id].Storage = value
}

Answer 1

[Александр Павлюк]

Небольшая копипаста, это нормально для go.
У вас тут другая страшная проблема, вы создаете мьютексы как локальные переменные. В результате они бесполезны здесь.

Comments

[Владимир Грабко]

Ну глобальной их же не делать (ведь несколько потоков будут её читать и всё). Как тогда поступить?

[Александр Павлюк]

Владимир Грабко: именно глобальной, смысл мьютекса в том, чтобы иметь к нему доступ из нескольких потоков, он потокобезопасен сам по себе. Когда делаем lock на него из одного потока, другие потоки при попытке сделать lock, будут ждать, пока тот первый поток не сделает unlock. Заблокировать мьютекс может одновременно только один поток!

[Владимир Грабко]

Александр Павлюк спс

[#алгоптимизируй #отботизируй]

Владимир Грабко: Александр Павлюк: да, с мьютексами жесть.....

[#алгоптимизируй #отботизируй]

Дублирование можно слегка порезать через вызов функторов.

[Oleg Shevelev]

Этот вопрос несколько месяцев назад у Владимир Грабко всплывал с мьютексами, тогда тоже клялся что всё понял и тут нате опять...

[Владимир Грабко]

Oleg Shevelev: там не так было. И там я к стати и понял что нужно их делать локальными .)) До него я мьютексы юзал глобальные))

[Oleg Shevelev]

Владимир Грабко: что бы точно понимать что материал усвоился - напишите то как это работает. В развёрнутом виде, можно даже на хабр:)

[Владимир Грабко]

Oleg Shevelev: о нет))) У меня супер сила - сливать карму )))