1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
|
package inamedparam
import (
"go/ast"
"golang.org/x/tools/go/analysis"
"golang.org/x/tools/go/analysis/passes/inspect"
"golang.org/x/tools/go/ast/inspector"
)
var Analyzer = &analysis.Analyzer{
Name: "inamedparam",
Doc: "reports interfaces with unnamed method parameters",
Run: run,
Requires: []*analysis.Analyzer{
inspect.Analyzer,
},
}
func run(pass *analysis.Pass) (interface{}, error) {
inspect := pass.ResultOf[inspect.Analyzer].(*inspector.Inspector)
types := []ast.Node{
&ast.InterfaceType{},
}
inspect.Preorder(types, func(n ast.Node) {
interfaceType, ok := n.(*ast.InterfaceType)
if !ok || interfaceType == nil || interfaceType.Methods == nil {
return
}
for _, method := range interfaceType.Methods.List {
interfaceFunc, ok := method.Type.(*ast.FuncType)
if !ok || interfaceFunc == nil || interfaceFunc.Params == nil {
continue
}
methodName := method.Names[0].Name
for _, param := range interfaceFunc.Params.List {
if param.Names == nil {
var builtParamType string
switch paramType := param.Type.(type) {
case *ast.SelectorExpr:
if ident := paramType.X.(*ast.Ident); ident != nil {
builtParamType += ident.Name + "."
}
builtParamType += paramType.Sel.Name
case *ast.Ident:
builtParamType = paramType.Name
}
if builtParamType != "" {
pass.Reportf(param.Pos(), "interface method %v must have named param for type %v", methodName, builtParamType)
} else {
pass.Reportf(param.Pos(), "interface method %v must have all named params", methodName)
}
}
}
}
})
return nil, nil
}
|
