День добрый

Недавно читал про технологии azure русскую версию на их же сайте. Путано. Переключил на английский и все прояснилось 😂

имею функцию:
func snmpNew(hosts []string) {
 var hl []string
 var wg sync.WaitGroup
 wg.Add(len(hosts))
 for _, host := range hosts {
   go func(host string, hl *[]string, wg *sync.WaitGroup) {
     defer wg.Done()
     *hl = append(*hl, host)
   }(host, &hl, &wg)
 }
 wg.Wait()
 fmt.Println(hl)
 for _, i := range hl {
   classes.FileWriteString("temp_new", i)
 }
}
в функцию передаю массив из 254 элементов, дальше с помощью гоурутины создаю новый массив, и результат записываю в файл, в новый массив попадает только часть из них, а не все 254 элемента

при этом, каждый раз кол-во элементов в новом массиве разное

Вынужден ещё раз скинуть ту же ссылку
https://golangforall.com/ru/post/golang-data-handling-concurrent-programs.html

Там есть пример про слайс

Вы конкурентно пишете в слайс, паники нет, но это работает неверно.

race condition же типичный

append перепишет указатель и статус

и каждый раз будет append в какой-то случайный момент времени

собери с -race кстати, он на это будет ругаться

тут лучше использовать индексы, а не append

Кстати, норм.
Если заранее создать слайс правильной длины и писать элементы по индексам, то не будет race

ну если hosts не map[string]string

а там есть или sync.Map или можно в мьютекс обернуть

или через канал по старинке

добавил Mutex, работает.

спасибо

попробую это же на другой фунции, если возникнут проблемы отпишусь)

но зачем лишние локи в вашем кейсе?

если вы про mutex.lock, то потому что иначе часть данных теряется

если там слайс, то @pragus предложил идею про отсутствие мьютекса - каждому воркеру дать id равный ячейке в слайсе и слайс преаллокейтнуть