避免使用覆蓋註釋編寫重複代碼

Question

下面是我們公司中有人寫的類。

public class Reader1 extends PdsxAdapterBaseGemsReader { 

    @Override 
    public PdsxReader acquireReader(PdsxReadRequest pdsxReadRequest) { 
     return new PdsxUserReadOnlyGemsReader(pdsxReadRequest); 
    } 

    protected static class PdsxUserReadOnlyGemsReader extends PdsxAdapterBaseGemsReader.PdsxBaseGemsReader { 

      protected PdsxUserReadOnlyGemsReader(PdsxReadRequest pdsxReadRequest) { 
       super(pdsxReadRequest); 
      } 

      protected ClusterIdEnum getClusterId() { 
       return ClusterIdEnum.MARKEPLACE1; 
      } 

      @Override 
      public List<PdsxRecord> readData() { 
       List<UserCacheDoImpl> userDos = readUserCacheRecords(req, serverId,startLastModifiedDate, endLastModifiedDate); 
       // some code 
      } 


      private List<UserCacheDoImpl> readUserCacheRecords(PdsxReadRequest req, int serverId, Date startLastModifiedDate, Date endLastModifiedDate) { 
       List<UserCacheDoImpl> userDos = new ArrayList<UserCacheDoImpl>(); 
       BackEndIterator userIter = new BackEndIterator(getClusterId().getId(), serverId, startLastModifiedDate, endLastModifiedDate, READ_BATCH_SIZE); 
       //some code 
      } 


      private PdsxRecord createPdsxRecordFromUserDO(UserCacheDoImpl userCache) { 
       //some code 
      } 

      private void xValidateAttrKey(PdsxAttrKey readAttr) { 
       // some code 
      } 
    } 

} 

問題陳述： -

我需要編寫具有相同功能的另一個類的所有方法的名字，因爲它是。唯一不同的是這個方法 - 的

protected ClusterIdEnum getClusterId() { 
    return ClusterIdEnum.MARKEPLACE1; 
} 

它需要

protected ClusterIdEnum getClusterId() { 
    return ClusterIdEnum.MARKEPLACE2; 
} 

所以，如果我需要做的是在一個很老土的方式，我可以直接複製粘貼第一Reader類的一切並通過更改羣集ID方法來創建一個新的Reader2類。就這樣。但這不是一個好方法。

有誰能告訴我什麼是最好的方法來做同樣的事情，而不寫任何duplicate code很多次。

我被在基類中重寫getLcusterId() method想着這樣的事情，用代碼像這個 -

public class Reader2 extends Reader1 { 

    @Override 
    public PdsxReader acquireReader(PdsxReadRequest pdsxReadRequest) { 
     return new PdsxUserNewReadOnlyGemsReader(pdsxReadRequest); 
    } 

    protected static class PdsxUserNewReadOnlyGemsReader extends PdsxUserReadOnlyGemsReader { 

     protected PdsxUserNewReadOnlyGemsReader(PdsxReadRequest pdsxReadRequest) { 
      super(pdsxReadRequest); 
     } 

     @Override 
     protected ClusterIdEnum getClusterId() { 
      return ClusterIdEnum.MARKEPLACE2; 
     } 
    } 
} 

由於一切都在第一類相同的，所以我並不需要實現什麼在我的Reader2類中吧？這只是我可以覆蓋getClusterId() method。

讓我知道如果我在這裏丟失任何東西。

Answer 1

如果它是可行的修改acquireReader方法的簽名，你可以將字段添加到您的閱讀器類：

public class Reader1 extends PdsxAdapterBaseGemsReader { 

@Override 
public PdsxReader acquireReader(PdsxReadRequest pdsxReadRequest, ClusterIdEnum clusterId) { 
    return new PdsxUserReadOnlyGemsReader(pdsxReadRequest, clusterId); 
} 

protected static class PdsxUserReadOnlyGemsReader extends PdsxAdapterBaseGemsReader.PdsxBaseGemsReader { 
     private ClusterIdEnum clusterId; 

     protected PdsxUserReadOnlyGemsReader(PdsxReadRequest pdsxReadRequest, ClusterIdEnum clusterId) { 
      super(pdsxReadRequest); 
      this.clusterId = clusterId; 
     } 

     protected ClusterIdEnum getClusterId() { 
      return clusterId; 
     } 

     //... 
} 

或者，也可以在Reader1構造函數到這一點，但它可以通過創建讀取器對象時的clusterId：

public class Reader1 extends PdsxAdapterBaseGemsReader { 
     private ClusterIdEnum clusterId; 

     public Reader1 (ClusterIdEnum clusterId) { 
      this.clusterId = clusterId; 
     } 

     @Override 
     public PdsxReader acquireReader(PdsxReadRequest pdsxReadRequest) { 
      return new PdsxUserReadOnlyGemsReader(pdsxReadRequest, clusterId); 
     } 

protected static class PdsxUserReadOnlyGemsReader extends PdsxAdapterBaseGemsReader.PdsxBaseGemsReader { 
     private ClusterIdEnum clusterId; 

     protected PdsxUserReadOnlyGemsReader(PdsxReadRequest pdsxReadRequest, ClusterIdEnum clusterId) { 
      super(pdsxReadRequest); 
      this.clusterId = clusterId; 
     } 

     protected ClusterIdEnum getClusterId() { 
      return clusterId; 
     } 

     //... 
}