Use plain channel send or receive SCC-S1000
Anti-pattern
Major
5 months ago
—
4 years old
should use for range instead of for { select {} }
120 logger.Info("start")
121 defer logger.Info("end")
122
123 for {124 select {
125 case b := <-bt.checkBuildsChan:
126 if b == nil {
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)