Целое число выдает исключение NullPointer при попытке изменить его - PullRequest
Новая
       9

Целое число выдает исключение NullPointer при попытке изменить его

0 голосов
/ 06 апреля 2019

Я пытаюсь создать PDF на основе данных, переданных через объект, используя Apache PDFBox.Поскольку существует переменное количество данных, я использую переменную с именем volY, чтобы отслеживать позицию y для записи информации.если volY больше 700, я закрываю поток контента, в который пишу, генерирую новую страницу и новый поток контента и начинаю запись на новую страницу.Методы, которые я использую для записи в pdf, возвращают целое число, которое представляет высоту строки, которую я добавляю к volY.По какой-то причине, когда я пытаюсь получить доступ к volY при переборе элементов, я получаю исключение нулевого указателя.

Вот мой код для создания PDF: 

public void generateSection(PDPage startingPage, PDPageContentStream cs, List<TestSection> sections) throws IOException {
    /*
     * TODO
     * verify list sequence integrity and reorder if not valid.
     */

    int volY = 350;
    PDPage page = startingPage;
    PDPageContentStream vcs = cs;
    // Iterate through sections
    for(int i = 0; i < sections.size(); i++) {
        if(volY > 700) {

            vcs.close();
            page = createPage();
            vcs = new PDPageContentStream(pd, page);
            volY = 50;
        }
        if(sections.get(i).isUrgent())
            cs.setNonStrokingColor(URGENT);

        drawString(sections.get(i).getType().getName(), 60, volY, vcs, page, 18);
        cs.setNonStrokingColor(REGULAR_TEXT);

        drawLine(vcs, 60, flipY(page, volY+8), 560);
        volY += 30;
        // Iterate through Items in section
        TestSection s = sections.get(i);
        for(int y = 0; y < s.getElements().size(); y++ ) {
            TestReportElement re = s.getElements().get(y);
            TestSubSection subSection = (TestSubSection)re;

            volY++;
            drawHeader(re.getTitle(), "a", volY, page, vcs);

            for(int z = 0; z < subSection.getItems().size(); z++) {
                //volY doesn't exist here for some reason?  At the very least it's not modifiable.


                if(vcs == null) {
                    System.err.println("VCS IS NULL");
                    System.exit(3);
                }
                TestInspectionItem ti = subSection.getItems().get(z);
                vcs.setNonStrokingColor(BOLD_TEXT);
                System.out.println(volY);
                drawMultipleStrings(ti.getPrompt(), volY, vcs, page, z);
                vcs.setNonStrokingColor(REGULAR_TEXT);
                for(int z1 = 0; z1 < ti.getResponses().size(); z1++) {
                    if (volY > 700) {

                        vcs.close();
                        page = createPage();
                        vcs = new PDPageContentStream(pd, page);
                        volY = 50;
                    }
                    String text = ti.getResponses().get(z1);
                    drawMultipleStrings(text, volY+15, vcs, page, z1);
                }
                if (volY > 700) {

                    vcs.close();
                    page = createPage();
                    vcs = new PDPageContentStream(pd, page);
                    volY = 50;
                }
            }

            if (volY > 700) {

                vcs.close();
                page = createPage();
                vcs = new PDPageContentStream(pd, page);
                volY = 50;
            }

        }
        // Add 70 to account for a new section.
        volY += 70;
    }
    vcs.close();
}

Вот код, который яЯ использую для рисования нескольких строк: 

private int drawMultipleStrings(String text, int y, PDPageContentStream cs, PDPage page, int index) {
    // Page is 900 units wide
    // Assume font size is 13
    int strSize = text.length();
    int height = 0;
    String textVal = text;
    List<String> allText = new ArrayList<>();
    int xVal = index % 2 == 0 ? 60 : 300;
    if(strSize > 40) {
        while(textVal.length() > 40) {
            for (int i = 40; i > 0; i--) {
                if (textVal.charAt(i) == ' ') {
                    allText.add(textVal.substring(0, i));
                    textVal = textVal.substring(i);
                    break;
                }
            }
        }
        allText.add(textVal);
        for(int ind = 0; ind < allText.size(); ind++) {
            String s = allText.get(ind);
            if(s.charAt(0) == ' ') {
                s = s.substring(1);
                drawString(s, xVal, y+(13*ind), cs, page, 13);
            } else {
                // This should only trigger on the first iteration.
                drawString(s, xVal, y+(13*ind), cs, page, 13);
            }
            height += 13;
        }
        // Allows items to be displayed in 2 columns based on the index
        return index % 2 == 0 ? 0 : height + 32;
    } else {
        drawString(text, index % 2 == 0 ? 60: 300, y, cs, page, 13);
        return 13;
    }
}

Этот код работает правильно, но если я изменю drawMultipleStrings(ti.getPrompt(), volY, vcs, page, z); на volY += drawMultipleStrings(ti.getPrompt(), volY, vcs, page, z); Выдает это исключение: 

java.lang.NullPointerException
    at org.apache.pdfbox.pdmodel.PDPageContentStream.writeOperand(PDPageContentStream.java:2429)
    at org.apache.pdfbox.pdmodel.PDPageContentStream.setNonStrokingColor(PDPageContentStream.java:1316)
    at org.apache.pdfbox.pdmodel.PDPageContentStream.setNonStrokingColor(PDPageContentStream.java:1348)
    at compliancego.report.PdfService.generateSection(PdfService.java:206)
    at compliancego.report.PdfService.generateHeader(PdfService.java:176)
    at compliancego.report.PdfService.<init>(PdfService.java:97)
    at compliancego.report.PdfService.main(PdfService.java:73)

Сначала яЯ подумал, что это потому, что некоторые данные не существуют для записи, но они прекрасно работают, если я не обновляю volY.Тогда я подумал, что проблема заключается в том, что PDPageContentStream не создается при создании новой страницы, но поток существует.

Заранее спасибо за любую помощь!

1 Ответ

1 голос
/ 18 июня 2019

Сначала вы создаете локальную переменную потока содержимого страницы, которую инициализируете полученным потоком содержимого страницы в качестве параметра: 

PDPageContentStream vcs = cs;

При изменении страницы вы закрываете поток содержимого текущей страницы в vcs, а затемустановите vcs для нового потока на новой странице: 

if (volY > 700) {

    vcs.close();
    page = createPage();
    vcs = new PDPageContentStream(pd, page);
    volY = 50;
}

Но в двух строках кода в цикле над sections вы рисуете cs вместо vcs:

cs.setNonStrokingColor(URGENT);
...
cs.setNonStrokingColor(REGULAR_TEXT);

Во время первого изменения страницы вы закрыли vcs, который указывал на тот же поток, что и cs.Таким образом, эти инструкции по настройке цвета нарисованы на замкнутом потоке cs, что приводит к наблюдаемому NullPointerException.

Чтобы исправить это, используйте здесь vcs вместо cs.

Причина, по которой это произошло только после того, как вы изменили 

drawMultipleStrings(ti.getPrompt(), volY, vcs, page, z);

на 

volY += drawMultipleStrings(ti.getPrompt(), volY, vcs, page, z);

, скорее всего, заключается в том, что до этого изменения volYникогда не становился достаточно большим, чтобы вызвать изменение страницы, но после этого это произошло.

...