时间:2021-07-01 10:21:17 帮助过:7人阅读
public class DemoPlugin extends PreorderDetector { private BugReporter bugReporter; public DemoPlugin(BugReporter reporter){ bugReporter = reporter; } public boolean hasHashcode; public boolean hasEqual; public MethodAnnotation equals; @Override public void visit(JavaClass obj){ System.out.println("visit class!!!!!!!!!" + obj.getClassName()); hasHashcode = false; hasEqual = false; equals = null; } @Override public void visit(Method me){ if(!me.isPublic()) return; if("equals".equals(me.getName()) && "(Ljava/lang/Object;)Z".equals(me.getSignature())){ hasEqual = true; System.out.println("find equals() method"); equals = MethodAnnotation.fromVisitedMethod(this); } if("hashCode".equals(me.getName()) && "()I".equals(me.getSignature())){ hasHashcode = true; System.out.println("find hashCode() method"); } if("hashcode".equals(me.getName()) && "()I".equals(me.getSignature())){ bugReporter.reportBug(new BugInstance(this, "HASHCODE WRONG SPELL", NORMAL_PRIORITY). addClass(getDottedClassName()).addMethod(MethodAnnotation.fromVisitedMethod(this))); } } @Override public void visitAfter(JavaClass obj){ if(hasEqual && !hasHashcode){ BugInstance instance = new BugInstance(this, "EQUALS WITHOUT HASHCODE", NORMAL_PRIORITY). addClass(getDottedClassName()).addMethod(equals); bugReporter.reportBug(instance); } } }
这里我们继承了PreorderDetector,而PreorderDetector继承了PreorderVisitor,是因为BugInstance需要使用Detector进行实例,而PreorderDetector实现了Detector接口。
上面方法我们使用了上个变量hasHashcode,hasEquals和equals,前两个变量用来保存当前访问的类是否存在hashCode和equals方法,第三个变量存放的是方法注释(要在报告bug中,这里先不管)。
上述三个变量每次都在visit(JavaClass obj)方法中进行状态重置,这样才能对所有的类都进行检测。visit(Method me)方法中用来判断是否是equals、hashCode、hashcode方法,并修改相应的状态。如果是hashcode方法,则直接将该bug报告。
visitAfter(JavaClass obj)方法执行时,已经对当前类检测完毕,这样可以判断是否出现equals方法,而没有出现hashCode方法的情况。
将fandbugs.xml和massage.xml加到项目中打包,便能对我们定义的模式进行检测。
以上仅是一个自定义在类、方法、字段层面上的检测器的一般步骤,例子仅是为了说明过程,只要我们熟悉我们想要检测的模式,就能通过上述方法实现。
FindBugs检测器实现(2)
标签: