2020-03-10 21:08:28 +00:00

74 lines
1.4 KiB
Go

package checks
import (
"go/ast"
"go/token"
"golang.org/x/tools/go/analysis"
config "github.com/tommy-muehle/go-mnd/config"
)
const OperationCheck = "operation"
type OperationAnalyzer struct {
pass *analysis.Pass
config *config.Config
}
func NewOperationAnalyzer(pass *analysis.Pass, config *config.Config) *OperationAnalyzer {
return &OperationAnalyzer{
pass: pass,
config: config,
}
}
func (a *OperationAnalyzer) NodeFilter() []ast.Node {
return []ast.Node{
(*ast.AssignStmt)(nil),
}
}
func (a *OperationAnalyzer) Check(n ast.Node) {
stmt, ok := n.(*ast.AssignStmt)
if !ok {
return
}
for _, expr := range stmt.Rhs {
switch x := expr.(type) {
case *ast.BinaryExpr:
switch xExpr := x.X.(type) {
case *ast.BinaryExpr:
a.checkBinaryExpr(xExpr)
}
switch yExpr := x.Y.(type) {
case *ast.BinaryExpr:
a.checkBinaryExpr(yExpr)
}
a.checkBinaryExpr(x)
}
}
}
func (a *OperationAnalyzer) checkBinaryExpr(expr *ast.BinaryExpr) {
switch x := expr.X.(type) {
case *ast.BasicLit:
if a.isMagicNumber(x) {
a.pass.Reportf(x.Pos(), reportMsg, x.Value, OperationCheck)
}
}
switch y := expr.Y.(type) {
case *ast.BasicLit:
if a.isMagicNumber(y) {
a.pass.Reportf(y.Pos(), reportMsg, y.Value, OperationCheck)
}
}
}
func (a *OperationAnalyzer) isMagicNumber(l *ast.BasicLit) bool {
return (l.Kind == token.FLOAT || l.Kind == token.INT) && !a.config.IsIgnoredNumber(l.Value)
}