在 FindBugs 檢測器實現（1） 中提到，FindBugs 主要有5類檢測器，這篇日志介紹下FindBugs在類、方法、字段結構上的檢測器實現。前面提到基于棧和簡單的字節碼模式要繼承OpcodeStackDetector類，并實現sawOpcode方法用來檢測每一個字節碼。

一般在寫檢測器之前，我們應該有一個自己想要檢測的代碼模式，但作為學習，這里使用一些簡單的模式作為例子：

1. 重寫了equals函數，卻沒有重寫hashCode函數。
2. 經hashCode函數拼寫為hashcode。

FindBugs在類、方法、字段層面的檢測器實現要繼承PreorderDetector類，簡單介紹些該類：

?

PreorderVisitor

PreorderVisitor 繼承了 BetterVisitor，并在 BetterVisitor的基礎之上，添加了大量的方法用來從當前訪問的類、方法或字段中獲取信息，另外該類還提供了一些visitAfter方法，用來實現訪問上的次序，在下面的例子中，我們會看到它的作用。像下面的方法 PreorderVisitor 中存在太多，這里就不一一列出，可能有些方法不能直接明了的知道其作用，大多可能是我們還不懂java的一些功能的緣故：

      
        public
      
      
         FieldDescriptor getFieldDescriptor()


      
      
        public
      
      
         MethodDescriptor getMethodDescriptor()


      
      
        public
      
      
         ClassDescriptor getClassDescriptor()


      
      
        public
      
      
         @SlashedClassName  String getClassName()


      
      
        public
      
      
        void
      
      
         visitAfter(Code obj)


      
      
        public
      
      
        void
      
       visitAfter(JavaClass obj)
    

?

來個例子

        
          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）