避免抽象类的子类上的代码重复

Question

我正在开发一个旧的Struts应用程序，它有一个抽象操作，通过填充Excel模板文件来创建Excel导出。

每个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

这里没有答案：示例中没有“代码重复”。

关键是：抽象类驱动子类的特定“布局”；但这是使用抽象类的本质。

换句话说：这里没什么好担心的。

您正在遵循众所周知的“认可”模式；例如，在子类中很好地使用@Override注释。

一切都好，祝你愉快！

Answer 2

我在这里看到的代码中几乎没有任何代码重复，但是如果您真的想“从石头中挤水”，您可以向基类添加一个构造函数，它接受输入文件名和输出文件名，并使每个派生类将其输入文件名和输出文件名传递给基类的构造函数。