graphlms

Ревью: рабочая реализация слияния каналов

Код-ревьюMiddle10 минпроверка — Спринт10-t041

Функция fanIn объединяет переменное число каналов int в один. Разберите и прокомментируйте решение: корректно ли оно, от чего зависит корректность, что бы вы улучшили?

Как на собесе: 10 минут, ищи всё — гонки, утечки, ошибки, безопасность, стиль. Ссылайся на строки.

review.gogo
1func fanIn(ins ...chan int) chan int {2	out := make(chan int)3	var grp sync.WaitGroup4	for _, c := range ins {5		grp.Add(1)6 7		go func() {8			defer grp.Done()9 10			for v := range c {11				out <- v12			}13		}()14	}15 16	go func() {17		grp.Wait()18		close(out)19	}()20 21	return out22}
⌘ Enter — проверить