Использование одиночного StringBuilder - throws OutofmemeoryException

Question

как показано ниже, чтобы прочитать каждую строку и выполнить конкатенацию на основе списка listValues.

BufferedReader br = null; 
StringBuilder sb = new StringBuilder(""); 
InputStream in = new FileInputStream(new File(file)); 
br = new BufferedReader(new InputStreamReader(in), 102400); 
for (String input; (input= br.readLine()) != null;) { 
    for (int i = 0; i < listValues.size(); i++) { 
     sb.append(input.substring(1, 5)); 
    } 
    map.put(sb.toString(), someOtherValue); 
    sb.delete(0, sb.length()); 
} 

Использование одного и того же StringBuilder для каждой итерации путем установки удаления содержимого каждый раз. Все еще бросает

Exception in thread "main" java.lang.OutOfMemoryError: Java heap space 
at java.lang.String.substring(Unknown Source) 

Какая ошибка я сделал?

Редактировать: Я исправился согласно Предложение Bathsheba. Но теперь бросает,

Exception in thread "main" java.lang.OutOfMemoryError: Java heap space 
at java.util.Arrays.copyOfRange(Unknown Source) 
at java.lang.String.<init>(Unknown Source) 
at java.io.BufferedReader.readLine(Unknown Source) 
at java.io.BufferedReader.readLine(Unknown Source) 

на for (String input; (input= br.readLine()) != null;) Что сейчас проблема?

Answer 1

Использование substring в плотной петле не рекомендуется, так как оно создаст много строк, которые не могут быть собраны в мусор.

Решение в вашем случае будет использовать charAt и добавить char S к экземпляру StringBuilder:

for (int j = 1; j <= 5; ++j){ /*ToDo - check the loop bounds*/ 
    sb.append(input.charAt(j)); /*StringBuilder has an overload for `char` insertion*/ 
}