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

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

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