iText7.1.3 добавление SVG в PdfDocument - PullRequest
Новая
       13

iText7.1.3 добавление SVG в PdfDocument

0 голосов
/ 04 октября 2018

Использование iText 7.1.3 и попытка добавить файл SVG в PdfDocument дает вывод, где тексты с длиной 1 не отображаются .Я нашел, где это может быть проблемой.Я прошу членов команды iText проверить это. 

try {
      SvgConverter.drawOnCanvas(svgUrl.openStream(), pdfCanvas_, imageLlx, imageLly);
} catch (IOException e) {
      e.printStackTrace();
}

Вызовы отладчика: 

processText:255, DefaultSvgProcessor (com.itextpdf.svg.processors.impl)
visit:212, DefaultSvgProcessor (com.itextpdf.svg.processors.impl)
visit:204, DefaultSvgProcessor (com.itextpdf.svg.processors.impl)
visit:204, DefaultSvgProcessor (com.itextpdf.svg.processors.impl)
executeDepthFirstTraversal:153, DefaultSvgProcessor (com.itextpdf.svg.processors.impl)
process:106, DefaultSvgProcessor (com.itextpdf.svg.processors.impl)
process:768, SvgConverter (com.itextpdf.svg.converter)
convertToXObject:555, SvgConverter (com.itextpdf.svg.converter)
convertToXObject:590, SvgConverter (com.itextpdf.svg.converter)
drawOnCanvas:380, SvgConverter (com.itextpdf.svg.converter)

В функции processText , в строкедля Обрезать конечные пробелы 

trimmedText = SvgTextUtil.trimTrailingWhitespace("A");

для trimmedText = A (length = 1) возвращает пустую строку 

/**
* Process the text contained in the text-node
*
* @param textNode node containing text to process
*/
private void processText(ITextNode textNode) {
    ISvgNodeRenderer parentRenderer = this.processorState.top();

    if (parentRenderer instanceof TextSvgNodeRenderer) {
        // when svg is parsed by jsoup it leaves all whitespace in text element as is. Meaning that
        // tab/space indented xml files will retain their tabs and spaces.
        // The following regex replaces all whitespace with a single space.
        //TODO(RND-906) evaluate regex and trim methods
        String trimmedText = textNode.wholeText().replaceAll("\\s+", " ");
        //Trim leading whitespace
        trimmedText = SvgTextUtil.trimLeadingWhitespace(trimmedText);
        //Trim trailing whitespace
        trimmedText = SvgTextUtil.trimTrailingWhitespace(trimmedText);
        parentRenderer.setAttribute(SvgConstants.Attributes.TEXT_CONTENT, trimmedText);
    }
}

1 Ответ

0 голосов
/ 04 октября 2018

В trimTrailingWhitespace вы действительно указали ошибку 

public static String trimTrailingWhitespace(String toTrim) {
    if(toTrim == null){
        return "";
    }
    int end = toTrim.length();
    if (end > 0) {
        int current = end - 1;
        while (current > 0) {
            char currentChar = toTrim.charAt(current);
            if (Character.isWhitespace(currentChar) && !(currentChar == '\n' || currentChar == '\r')) {
                //if the character is whitespace and not a newline, increase current
                current--;
            } else {
                break;
            }
        }
        if(current == 0){
            return "";
        }else {
            return toTrim.substring(0, current + 1);
        }
    }else{
        return toTrim;
    }
}

Как уже отмечалось в комментарии //if the character is whitespace and not a newline, increase current, за которым следует current--;, этот метод является копией trimLeadingWhitespace (гдестрока после идентичного комментария действительно увеличивает current), измененная для работы на другом конце параметра String.К сожалению, однако, модификация была неправильной: если строка имеет непробельный символ в позиции 0 и после этого только пробелы, она ошибочно считается пустой.

Исправление будет заменять 

while (current > 0)

по 

while (current >= 0)

и 

if(current == 0)

по 

if(current < 0)

С этим исправьте 

if (end > 0) {
    [...]
}else{
    return toTrim;
}

рамку вокруг [...] кроме того, становится ненужным.И цикл while мог бы быть более компактно сформулирован как цикл for, например, так: 

public static String trimTrailingWhitespace(String toTrim) {
    if (toTrim == null) {
        return "";
    }
    int current = toTrim.length() - 1;
    for ( ; current >= 0; current--) {
        char currentChar = toTrim.charAt(current);
        if (!(Character.isWhitespace(currentChar) && !(currentChar == '\n' || currentChar == '\r'))) {
            break;
        }
    }
    return current < 0 ? "" : toTrim.substring(0, current + 1);
}
...