Use plain channel send or receive SCC-S1000
Anti-pattern
Major
2 years ago
—
2 years old
should use for range instead of for { select {} }
252 c.socket.Close()
253 }()
254
255 for {256 select {
257 case message, ok := <-c.send:
258 if !ok {
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)