Unused parameter in function RVV-B0012
Bug risk
10 days ago
—
8 months old
parameter 'c' seems to be unused, consider removing or renaming it as _
63 return nil
64}
65
66func helpers(c *stdapi.Context) template.FuncMap { 67 return template.FuncMap{
68 "env": func(s string) string {
69 return os.Getenv(s)parameter 'c' seems to be unused, consider removing or renaming it as _
557 return c.RenderOK()
558}
559
560func (s *Server) Initialize(c *stdapi.Context) error { 561 return stdapi.Errorf(404, "not available via api")
562}
563parameter 'c' seems to be unused, consider removing or renaming it as _
1231 return c.RenderOK()
1232}
1233
1234func (s *Server) Start(c *stdapi.Context) error {1235 return stdapi.Errorf(404, "not available via api")
1236}
1237parameter 'c' seems to be unused, consider removing or renaming it as _
1252 return c.RenderJSON(v)
1253}
1254
1255func (s *Server) SystemInstall(c *stdapi.Context) error {1256 return stdapi.Errorf(404, "not available via api")
1257}
1258parameter 'c' seems to be unused, consider removing or renaming it as _
1503 return c.RenderJSON(v)
1504}
1505
1506func (s *Server) SystemUninstall(c *stdapi.Context) error {1507 return stdapi.Errorf(404, "not available via api")
1508}
1509
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.")
}