Индекс строки вне диапазона при работе с String Builder - PullRequest
0 голосов
/ 30 октября 2019

Итак, я получаю сообщение об ошибке такого типа:

java.lang.StringIndexOutOfBoundsException: индекс строки вне диапазона: 447

Каждый раз, когда я запускаю этот код:

String text = "What is Lorem Ipsum? Lorem Ipsum is simply dummy text of the printing and typesetting industry. Lorem Ipsum has been the industry's standard dummy text ever since the 1500s, when an unknown printer took a galley of type and scrambled it to make a type specimen book. It has survived not only five centuries, but also the leap into electronic typesetting, remaining essentially unchanged. It was popularised in the 1960s with the release of Letraset sheets containing Lorem Ipsum passages, and more recently with desktop publishing software like Aldus PageMaker including versions of Lorem Ipsum.";
    StringBuilder textSb = new StringBuilder(text);
    int textLength = textSb.length();
    System.out.println(textLength);
    for(int i = 3; i< textLength; i += 4){
        if (textSb.charAt(i) == ' '){
            textSb.deleteCharAt(i+1);
            i--;
        }else{
            textSb.deleteCharAt(i);
            i--;
        }

ОК. Итак, что он хочет сделать. Я пытаюсь каждый раз удалять цикл четвертого символа, и если это пробел, я удаляю второй символ после пробела. Я попытался "i--", потому что текст становится меньше с одним символом за раз, что будет означать, что мой индекс не будет работать, поскольку я продолжаю удалять эти символы.

Основная проблема заключается в этой ошибке. Как можно 447 не быть в пределах досягаемости, учитывая, что в моей строке около 597 символов.

Ответы [ 4 ]

0 голосов
/ 30 октября 2019
  1. Вам нужно проверить, если i+1 меньше textLength (нужно переназначать после каждого оператора удаления, как показано в 2) или textSb.length(), прежде чем делать textSb.deleteCharAt(i+1);.

    Вы можете добавить проверку в существующее условие if:

         if (textSb.charAt(i) == ' ' && i+1<textLength)
    

    ИЛИ

         if (textSb.charAt(i) == ' ' && i+1<textSb.length())
    

Вам также необходимо переназначить значение textLength при удалении символа, упомянутого Эллиоттом.

  • Добавить textLength = textSb.length(); после каждого оператора удаления.

ИЛИ

  • изменить for loop на for(int i = 3; i< textSb.length(); i += 4)
0 голосов
/ 30 октября 2019

После изменения длины исходной строки (textSb), но с использованием исходной длины строки, неудивительно, что вы получили StringIndexOutOfBoundsException.

Посмотрите этот пример: https://github.com/lombocska/stackoverflow/blob/master/src/main/java/string_index_out_of_range/StringBuilderTestClass.java#L35

for (int i = 3; i < textSb.length(); i += 4) {
  ...
}

The answer is that you should use the modified text length in the for cycle instead of the unmodified one. :)

0 голосов
/ 30 октября 2019

textSb.deleteCharAt (я);виновникНе следует удалять из того же StringBuilder, а создать его копию, а затем выполнять операции с ней.

0 голосов
/ 30 октября 2019

477 - потому что вы удалили остальные индексы. просто проверьте перед удалением индекса как

if (textSb.charAt(i) == ' ' && (i+1<textLength)){
            textSb.deleteCharAt(i+1);
            i--;
}
...