1 package net.sourceforge.pmd.lang.java.rule;
2
3 import java.util.List;
4
5 import net.sourceforge.pmd.lang.ast.Node;
6 import net.sourceforge.pmd.lang.java.ast.ASTEqualityExpression;
7 import net.sourceforge.pmd.lang.java.ast.ASTLiteral;
8 import net.sourceforge.pmd.lang.java.ast.ASTPrimitiveType;
9 import net.sourceforge.pmd.lang.java.ast.ASTRelationalExpression;
10 import net.sourceforge.pmd.lang.java.ast.ASTVariableDeclaratorId;
11 import net.sourceforge.pmd.lang.java.symboltable.NameOccurrence;
12
13
14
15
16
17
18
19
20 public abstract class AbstractInefficientZeroCheck extends AbstractJavaRule {
21
22 public abstract boolean appliesToClassName(String name);
23
24 public abstract boolean isTargetMethod(NameOccurrence occ);
25
26 public Object visit(ASTVariableDeclaratorId node, Object data) {
27 Node nameNode = node.getTypeNameNode();
28 if (nameNode instanceof ASTPrimitiveType) {
29 return data;
30 }
31 if (!appliesToClassName(node.getNameDeclaration().getTypeImage())) {
32 return data;
33 }
34
35 List<NameOccurrence> declars = node.getUsages();
36 for (NameOccurrence occ: declars) {
37 if (!isTargetMethod(occ)) {
38 continue;
39 }
40 Node expr = occ.getLocation().jjtGetParent().jjtGetParent().jjtGetParent();
41 if ((expr instanceof ASTEqualityExpression ||
42 (expr instanceof ASTRelationalExpression && ">".equals(expr.getImage())))
43 && isCompareZero(expr)) {
44 addViolation(data, occ.getLocation());
45 }
46 }
47 return data;
48 }
49
50
51
52
53
54
55
56 private boolean isCompareZero(Node equality) {
57 return checkComparison(equality, 0) || checkComparison(equality, 1);
58
59 }
60
61
62
63
64
65
66
67
68
69
70 private boolean checkComparison(Node equality, int i) {
71 Node target = equality.jjtGetChild(i).jjtGetChild(0);
72 if (target.jjtGetNumChildren() == 0) {
73 return false;
74 }
75 target = target.jjtGetChild(0);
76 return target instanceof ASTLiteral && "0".equals(target.getImage());
77 }
78
79 }