Éviter la duplication de code sur les classes enfant d'une classe abstraite

Question

Je travaille sur une ancienne application Struts qui a une action abstraite pour créer des exports Excel en remplissant des fichiers de modèle Excel.

Chaque action d'exportation Excel étend cette classe parente et ajoute sa propre implémentation pour remplir le modèle Excel et définit son propre modèle et son propre nom de fichier de sortie.

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; 
    } 
... 
} 

Implémentations sont à la recherche comme celui-ci et c'est là la duplication de code est "mis en garde" par 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; 
    } 
} 

Toutes les idées comment éviter la duplication?

Answer 1

Il y a que je vois ici à peine une duplication de code dans le code, mais si vous voulez vraiment « presser l'eau d'une pierre », vous pouvez ajouter un constructeur à la classe de base, qui accepte l'entrée-nom de fichier et de sortie -filename, et fait passer chaque classe dérivée son nom de fichier d'entrée et son nom de fichier de sortie au constructeur de la classe de base.

Answer 2

Une non-réponse ici: il n'y a pas de "duplication de code" dans vos exemples.

Le point est: votre classe abstraite entraîne une certaine "disposition" pour votre classe enfant; mais c'est l'essence d'utiliser une classe abstraite. En d'autres termes, il n'y a rien à craindre ici.

Vous suivez des modèles «approuvés» bien connus; par exemple en utilisant joliment l'annotation @Override dans votre classe enfant.

Tout va bien, passez une bonne journée!