抽象クラスの子クラスでコードの重複を避ける

Question

私は、Excelテンプレートファイルを埋めてExcelエクスポートを作成する抽象的なアクションを持つ古いStrutsアプリケーションで作業しています。

各Excelエクスポートアクションは、この親クラスを拡張し、Excelテンプレートを埋め込む独自の実装を追加し、独自のテンプレートと出力ファイル名を定義します。

public abstract class ExcelExportAction extends BaseAction { 
    protected abstract String getInputFilename(); 
    protected abstract String getOutputFilename(); 
    protected abstract HSSFWorkbook modifyWorkbook(HSSFWorkbook workbook, HttpServletRequest request); 

    @Override 
    protected final ActionForward run(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) 
    throws Exception { 

     Workbook workbook = new Workbook(getInputFilename()); 
     workbook = modifyWorkbook(workbook, request); 
     addWorkBookToResponse(workbook, getOutputFilename());  

     return null; 
    } 
... 
} 

実装は次のように見ていると、コードの重複がSonarQubeで「警告し」ている場所です：

public class BudgetReportExcelAction extends ExcelExportAction { 
    private static final String INPUT_FILENAME = "Report-Budget-Template.xls"; 
    private static final String OUTPUT_FILENAME = "Report-Customerbudget.xls"; 

    @Override 
    protected String getInputFilename() { 
     return INPUT_FILENAME; 
    } 
    @Override 
    protected String getOutputFilename() { 
     return OUTPUT_FILENAME; 
    } 
    @Override 
    protected HSSFWorkbook modifyWorkbook(HSSFWorkbook workbook, HttpServletRequest request) { 
     /* modify the workbook */ 
     return workbook; 
    } 
} 

任意のアイデアをどのように重複を避けるため？

Answer 1

私がここで見ているコードには重複しているコードはほとんどありませんが、本当に "水を石から搾りたい"場合は、入力ファイル名と出力を受け入れるコンストラクタを基本クラスに追加できますそれぞれの派生クラスがその入力ファイル名と出力ファイル名を基本クラスのコンストラクタに渡すようにします。

Answer 2

ここでは無回答：例では「コードの重複」はありません。

要点：あなたの抽象クラスは、あなたの子クラスに対して特定の "レイアウト"をで駆動します。それは抽象クラスを使用してエッセンスです。

つまり、ここで心配することはありません。

よく知られた「承認済み」パターンに従っています。例えば、あなたの子クラスの@Overrideアノテーションをうまく使っています。

すてきな一日です。