Use plain channel send or receive SCC-S1000
Anti-pattern
Major
2 months ago
—
4 years old
should use a simple channel send/receive instead of select with a single case
63 var events []*Event
64
65 for len(events) != 2 {
66 select {67 case event := <-eventsChannel:
68 log.Info(event)
69 events = append(events, event)
Description
Select statements with a single case can be replaced with a simple send or
receive. If you intend to handle the case when no value is received from
the channel, add a default case to make the select statement non-blocking.
Bad practice
select {
case x := <-ch:
fmt.Println(x)
}
Recommended
x := <-ch
fmt.Println(x)