Unused method receiver RVV-B0013
Bug risk
Major
2 years ago
—
3 years old
Method receiver 's' is not referenced in method's body, consider removing or renaming it as _
52 return s.client.Ping(context.Background()).Err()
53}
54
55func (s redis) GetMetadata() core.Metadata { 56 return meta
57}
58Method receiver 'r' is not referenced in method's body, consider removing or renaming it as _
108 return exists == 1, nil
109 }
110}
111func (r *redis) GetCustomAppRoutes() []*core.Route {112 return []*core.Route{}
113}
114Method receiver 'm' is not referenced in method's body, consider removing or renaming it as _
45 return nil
46}
47
48func (m *memory) GetMetadata() core.Metadata { 49 return meta
50}
51Method receiver 'm' is not referenced in method's body, consider removing or renaming it as _
49 return meta
50}
51
52func (m *memory) GetCustomAppRoutes() []*core.Route { 53 return []*core.Route{}
54}
55Method receiver 'u' is not referenced in method's body, consider removing or renaming it as _
34 return nil
35}
36
37func (u urls) GetMetadata() core.Metadata {38 return meta
39}
40
Description
Methods with unused receivers can be a symptom of unfinished refactoring or a bug. To keep the same method signature, omit the receiver name or '_' as it is unused.
Bad practice
func (f *Unix) Name() string {
return "unix"
}
func (_ *Unix) Name() string {
return "unix"
}
Recommended
func (*Unix) Name() string {
return "unix"
}