1

Как будет правильнее (и займёт меньше времени при выполнении), если на каждой итерации цикла создаётся новая строка?

1 вариант - создание объекта строки в цикле:

while (conn.hasNext()) {
    String timerStr = StringUtils.isBlank(conn.getString("time_field")) ? "time" : conn.getString("time_field"));
    report.setTimer(new TimerSystem(id, timerStr));
}

2 вариант - вынос создания строки за цикл:

String timerStr;
while (conn.hasNext()) {
    timerStr = StringUtils.isBlank(conn.getString("time_field")) ? "time" : conn.getString("time_field"));
    report.setTimer(new TimerSystem(id, timerStr));
} 

3 вариант - без объявления переменной String:

while (conn.hasNext()) {
    report.setTimer(new TimerSystem(id, StringUtils.isBlank(conn.getString("time_field")) ? "time" : conn.getString("time_field"))));
} 

Или же все эти варианты абсолютно равнозначны?

P.S. На сайте уже был похожий вопрос Как будет быстрее? java, но мне всё-таки кажется, что здесь работа со строками и примитивными типами может отличаться.

3
  • а где в примерах конкатенация строк?
    – Grundy
    22 мар 2017 в 10:42
  • @Grundy, так ведь вопрос не про конкатенацию, а про создание новой строки.
    – Ksenia
    22 мар 2017 в 10:44
  • Да, я не совсем верно понял вопрос. 1 и 3 варианты равнозначны, если переменная timerStr не используется вне цикла, то и 2 так же. Для проверки можно посмотреть генерируемый байткод
    – Grundy
    22 мар 2017 в 11:16

1 ответ 1

4

На мой взгляд, оптимизировать в приведенных вариантах нечего. Переменная timerStr - это всего лишь указатель на строку. Cтрого говоря, сама аллокация памяти и создание новой строки у вас совершенно в другом месте, это зависит от реализации функции conn.getString(). Внутри цикла вы работаете только с указателями на строки.

Я бы записал в более удобочитаемом виде и не беспокоился о производительности до того момента пока вы не поймете, что именно это место является узким и требует оптимизаций:

while (conn.hasNext()) {
    String timeField = conn.getString("time_field");
    String timerStr = StringUtils.isBlank(timeField) ? "time" : timeField;
    report.setTimer(new TimerSystem(id, timerStr));
}

Дополнительно можно прочитать про string interning.

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.