разрабатывать общий API с использованием абстрактного метода, но при переходе к анонимному классу приходится использовать конечные переменные. Есть ли лучший способ сделать это?

Question

Я пытаюсь создать универсальный API, который будет работать поверх iText. Одна из функций этого API состоит в том, чтобы позволить пользователю разделить PDF на отдельную страницу и позволить пользователю добавлять список текста на каждую страницу PDF после разделения. Например, pdf из 20 страниц, и после запуска этого процесса у меня будет 20 из 1-page-pdf, и в первом pdf будет текст 000001, а в последнем pdf будет 000020 pdf. Поэтому для достижения этой цели я использую абстрактный метод, который позволяет разработчику писать код о том, как он хочет, чтобы текст форматировался с учетом текущего номера страницы.

public abstract class GenericText {

    /**
     * The X position of the text. (0, 0) is at the bottom left
     */
    private float x;

    /**
     * The Y position of the text. (0, 0) is at the bottom left
     */
    private float y;

    /**
     * The rotation of the text. Rotation 0, 90, 180, 270
     */
    private float rotation;

    /**
     * <code>com.itextpdf.text.pdf.BaseFont</code>. Determine the font for the text
     */
    private BaseFont font;

    /**
     * Determine the font size of the text
     */
    private float fontSize;

    /**
     * This tells whether text can only be placed first page or on every page
     */
    private ComponentPlacement placement;

    /**
     * Since the text that the user want to insert onto the Pdf might vary
     * from page to page, or from logical document to logical document, we allow
     * the user to write their own implementation of the text. To give the user enough
     * flexibility, we give them the reference to the physical page index, the logical page index. 
     * @param physcialPage The actual page number that the user current looking at
     * @param logicalPage A Pdf might contain multiples sub-documents, <code>logicalPage</code>
     * tell the user which logical sub-document the system currently looking at
     */
    public abstract String generateText(int physicalPage, int logicalPage);
    ...
}

PdfPrcessor.java: здесь происходит разделение

/**
 * This is the main process that will split the pdf into individual page, and text to each page
 */
public void splitPdf(String inputPdf, boolean isSplit, List<GenericText> textList, 
             String outputDir, String baseOutputName, String outputPdfName) throws IOException, DocumentException{
    ...
    PdfReader reader = new PdfReader(inputPdf)
    PdfContentByte cb = ... ;
    for(int physicalPageIndex=1 ; physicalPageIndex<=reader.getNumberOfPages(); physicalPageIndex ++)
       ...
       //Code to split PDF. Write each page to a separate pdf. For each pdf, insert all text inside `textList` onto the pdf
       ...
       //Insert text
       if(textList != null){
         for(GenericText textComponent : textList){
            String text = textComponent.generateText(physicalPageIndex, logicalPageIndex);
            addText(text, cb, textComponent.getFont(), textComponent.getFontSize(), textComponent.getX(), textComponent.getY(), textComponent.getRotation());
         }
       }
    }
    ...
}

Так что в моем основном классе я бы сделал это,

final String printName = printNameLookup.get(baseOutputName);
final String seq = config.getPrintJobSeq();
GenericText keyline = new GenericText(90, 640, 0, arial, 7, ComponentPlacement.FIRST_PAGE){
    @Override
    public String generateText(int physicalPage, int logicalPage) {
         return printName + seq + " " + Utils.right(String.valueOf(logicalPage), 6, '0');
    }
};
textList.add(keyline);
pdfProcess.splitPdf(inputPdfPath, true, textList, outputDir, baseOutputName, outputPdfName);

Это прекрасно работает, и я думаю, что оно очень гибкое, однако printName и seq должны быть объявлены как final, чтобы пройти внутрь generateText(int physicalPage, int logicalPage). Как мне спроектировать это так, чтобы оно не требовало поля final. interface поможет? Я использую API-интерфейс guava , и я могу сделать это

ImmutableListMultimap<String, File> groups = Multimaps.index(pdfList,
new Function<File, String>(){
    public String apply(File input){
        String[] ids = getId(input.getName());
        PackageLog pl = logProcessor.lookUp(new Long(ids[0]), ids[1]);
        String printName = printNameLookup.get(getPackageName(pl, s));
    }   
});

logProcessor и printNameLookup - это не final, мне нравится, как они проектируют, и я сейчас читаю их источники, но это займет какое-то время, кто-нибудь с экспертом в области дизайна может пролить мне свет?

Answer 1

Вы открыли шаблонный метод шаблон проектирования.В этом случае копирование значений в конечные переменные будет работать.

Answer 2

Скопируйте их в конечные переменные и используйте вместо них.

String printName = printNameLookup.get(baseOutputName);
String seq = config.getPrintJobSeq();
// use these in the anonymous class.
final String finalPrintName = printName;
final String finalSeq = seq;

или используйте массивы

final String[] printName = { printNameLookup.get(baseOutputName) };
final String[] seq = { config.getPrintJobSeq() };

// use printName[0] and seq[0] everywhere.