Как исправить зависть от проекта на основе Java?

Question

После того, как я использовал «inFusion Hydrogen» для анализа моего проекта, довольно легко понять, что myQuestionFun() вызывает зависть.

public abstract class Father{ 
    protected DataModel dataModel; 
    public abstract void myQuestionFun(); 
} 
public class Child extends Father{ 

    public DataModel<TableInfo> tableList = new DataModel<TableInfo>; 
    public DataModel<TableInfo> getTableList(){ 
     return this.tableList; 
    } 

    @Override 
    public DataModel<Report> getDataModel(){ 
     return this.dataModel; 
    } 

    @Override 
    public void myQuestionFun(){ 
     List<TableInfo> tabList = new ArrayList<TableInfo>(); 
     for (int i=0; dataModel.getDataList().size(); i++) { 
      Report rep = (Report)dataModel.getDataList.get(i); 
      TableInfo tabInfo = new TableInfo(); 
      tabInfo.setId(rep.getId()); 
      tabInfo.setName(rep.getName()); 
      tabList.add(tabInfo); 
     } 
     tableList.setPage(dataModel.getPage()); 
     tableList.setSorter(dataModel.getSorter()); 
    } 
} 

Вопрос в том, что это лучший способ исправить это? Поскольку myQuestionFun() является функцией абстрактного метода родительского класса, я не могу его переместить.

Answer 1

Extract следующего кода в качестве метода от ребенка в Отец:

protected List<TableInfo> getTabList() { 
    final List<TableInfo> tabList = new ArrayList<TableInfo>(); 
    for (int i=0; dataModel.getDataList().size(); i++) { 
      Report rep = (Report) dataModel.getDataList.get(i); 
      TableInfo tabInfo = new TableInfo(); 
      tabInfo.setId(rep.getId()); 
      tabInfo.setName(rep.getName()); 
      tabList.add(tabInfo); 
    } 
    return tabList; 
} 

Переместить следующий метод Отец, если она всегда должна возвращать this.dataModel:

public DataModel<Report> getDataModel(){ 
    return this.dataModel; 
} 

Answer 2

Удалить dataModel Полевая форма Father и изменить его на интерфейс.