Found empty body in an
if or else branch SCC-SA9003
Anti-pattern
Minor
6 months ago
—
4 years old
empty branch
72 api.Go(func(exit chan struct{}) {
73 defer s.Stop()
74
75 if err := s.Serve(tcpListener); err != nil { //nolint:staticcheck 76 // may be stopped - not error
77 // zapwriter.Logger("api").Fatal("failed to serve", zap.Error(err))
78 }
Description
Empty body in an if or else branch, so either remove this code or comment it
out to keep the codebase clean.
Bad practice
if x > 0 {
sum += x
} else {
// empty block
}
Recommended
if x > 0 {
sum += x
}