Unused parameter in function RVV-B0012
Bug risk
Minor
2 years ago
—
2 years old
parameter 'args' seems to be unused, consider removing or renaming it as _
336// customLogger Customized logger used to filter logging messages
337type customLogger struct{}
338
339func (dl *customLogger) Printf(format string, args ...interface{}) {340}
341
342// newHTTPServer returns a new instance of fasthttp serverparameter 'format' seems to be unused, consider removing or renaming it as _
336// customLogger Customized logger used to filter logging messages
337type customLogger struct{}
338
339func (dl *customLogger) Printf(format string, args ...interface{}) {340}
341
342// newHTTPServer returns a new instance of fasthttp server
Description
Unused parameters in functions or methods should be replaced with _
(underscore) or removed.
Functions or methods with unused parameters can be a symptom of unfinished
refactoring or a bug. If an unused parameter is present, it should be named _
(underscore) to avoid raising this issue and better readability.
Bad practice
func abc(unused string) {
fmt.Println("Not using any passed params.")
}
Recommended
func abc(_ string) {
fmt.Println("Not using any passed params.")
}
func abc() {
fmt.Println("Not using any passed params.")
}