Правильний спосіб використання StringBuilder в SQL

Question 1

Я щойно знайшов у своєму проекті таку збірку запитів sql:

return (new StringBuilder("select id1, " + " id2 " + " from " + " table")).toString();

Чи StringBuilderдосягається це своєю метою, тобто зменшенням використання пам'яті?

Я сумніваюся в цьому, оскільки в конструкторі використовується '+' (оператор concat String). Це займе таку ж пам’ять, як використання рядка, як код нижче? s Я зрозумів, це відрізняється при використанні StringBuilder.append().

return "select id1, " + " id2 " + " from " + " table";

Чи рівні обидва твердження за використанням пам'яті чи ні? Будь ласка, поясніть.

Спасибі заздалегідь!

Редагувати:

До речі, це не мій код . Знайшов у старому проекті. Крім того, запит не такий маленький, як у моєму прикладі. :)

Question 2

Мета використання StringBuilder, тобто зменшення пам'яті. Чи досягнуто це?

Ні, зовсім ні. Цей код використовується StringBuilderнеправильно. (Мені здається, ви це неправильно вказали; звичайно там немає цитат навколо id2таtable ?)

Зверніть увагу, що метою (як правило) є зменшення відтоку пам’яті, а не загальної використаної пам’яті, щоб полегшити життя збирача сміття.

Чи буде пам'ять рівною використанню рядка, як показано нижче?

Ні, це спричинить більший обсяг пам’яті, ніж просто прямий конкат, який ви цитували. (Поки / якщо JVM-оптимізатор не побачить, що явне StringBuilderв коді непотрібне, і оптимізує його, якщо може.)

Якщо автор цього коду хоче використовувати StringBuilder(є аргументи за, але також проти; див. Примітку в кінці цієї відповіді), краще зробити це належним чином (тут я припускаю, що навколо насправді немає цитат id2іtable ):

StringBuilder sb = new StringBuilder(some_appropriate_size);
sb.append("select id1, ");
sb.append(id2);
sb.append(" from ");
sb.append(table);
return sb.toString();

Зверніть увагу, що я перерахував some_appropriate_sizeу StringBuilderконструкторі, щоб він починався з достатньою місткістю для повного вмісту, який ми збираємось додати. Розмір за замовчуванням, який використовується, якщо ви не вказали жодного, - це 16 символів , що, як правило, занадто мало і призводить до StringBuilderнеобхідності робити перерозподіли, щоб зробити себе більшим (IIRC, в Sun / Oracle JDK, він подвоюється [або більше, якщо він знає, що йому потрібно більше для задоволення конкретного append] кожного разу, коли у нього закінчується місце).

Можливо , ви чули , що конкатенація буде використовувати StringBuilderпід ковдрою , якщо зібраний з компілятором Sun / Oracle. Це правда, він буде використовувати один StringBuilderдля загального виразу. Але він буде використовувати конструктор за замовчуванням, що означає, що в більшості випадків йому доведеться виконати перерозподіл. Однак читати його легше. Зверніть увагу, що це не відповідає серії об'єднань. Так, наприклад, для цього використовується одинStringBuilder :

return "prefix " + variable1 + " middle " + variable2 + " end";

Це приблизно перекладається на:

StringBuilder tmp = new StringBuilder(); // Using default 16 character size
tmp.append("prefix ");
tmp.append(variable1);
tmp.append(" middle ");
tmp.append(variable2);
tmp.append(" end");
return tmp.toString();

Отже, це нормально, хоча конструктор за замовчуванням та подальший перерозподіл (и) не є ідеальними, шанси на те, що він досить хороший - і конкатенація багато читабельніша.

Але це лише для одного виразу. StringBuilderДля цього використовуються декілька s:

String s;
s = "prefix ";
s += variable1;
s += " middle ";
s += variable2;
s += " end";
return s;

Це в кінцевому підсумку стає приблизно таким:

String s;
StringBuilder tmp;
s = "prefix ";
tmp = new StringBuilder();
tmp.append(s);
tmp.append(variable1);
s = tmp.toString();
tmp = new StringBuilder();
tmp.append(s);
tmp.append(" middle ");
s = tmp.toString();
tmp = new StringBuilder();
tmp.append(s);
tmp.append(variable2);
s = tmp.toString();
tmp = new StringBuilder();
tmp.append(s);
tmp.append(" end");
s = tmp.toString();
return s;

... що досить потворно.

Однак важливо пам’ятати, що у всіх випадках, окрім дуже рідких, це не має значення, і перевагу читання (що покращує ремонтопридатність) надає перевагу, якщо не брати до уваги певну проблему продуктивності.

Question 3

Коли у вас вже є всі "шматочки", які ви хочете додати, використовувати сенс StringBuilderвзагалі немає. Використання StringBuilder та конкатенація рядків у тому самому виклику згідно з вашим зразком коду ще гірше.

Це було б краще:

return "select id1, " + " id2 " + " from " + " table";

У цьому випадку конкатенація рядків фактично відбувається під час компіляції , тому це еквівалентно ще простішому:

return "select id1, id2 from table";

Використання new StringBuilder().append("select id1, ").append(" id2 ")....toString()насправді заважає продуктивності в цьому випадку, оскільки змушує конкатенацію виконуватися під час виконання , а не під час компіляції час . На жаль

Якщо реальний код будує запит SQL, включаючи значення у запит, то це ще одне окреме проблема, яка полягає в тому, що ви повинні використовувати параметризовані запити, вказуючи значення в параметрах, а не в SQL.

У мене є стаття про String/StringBuffer яку я писав деякий час тому - раніше StringBuilderз’явився. Принципи стосуються StringBuilderоднаково, хоча.

Question 4

[[Тут є кілька хороших відповідей, але я вважаю, що їм все ще бракує трохи інформації. ]]]

return (new StringBuilder("select id1, " + " id2 " + " from " + " table"))
     .toString();

Отже, як ви зазначаєте, приклад, який ви подаєте, є спрощеним, але давайте все одно проаналізуємо його. Тут відбувається те, що компілятор насправді виконує +роботу тут, оскільки "select id1, " + " id2 " + " from " + " table"це всі константи. Отже, це перетворюється на:

return new StringBuilder("select id1,  id2  from  table").toString();

У цьому випадку, очевидно, немає сенсу використовувати StringBuilder. Ви також можете зробити:

// the compiler combines these constant strings
return "select id1, " + " id2 " + " from " + " table";

Однак, навіть якщо ви додавали будь-які поля або інші неконстанти, тоді компілятор використовував би внутрішній StringBuilder - вам не потрібно його визначати:

// an internal StringBuilder is used here
return "select id1, " + fieldName + " from " + tableName;

Під обкладинками це перетворюється на код, який приблизно еквівалентний:

StringBuilder sb = new StringBuilder("select id1, ");
sb.append(fieldName).append(" from ").append(tableName);
return sb.toString();

Дійсно, єдиний час, який вам потрібно використовувати StringBuilder безпосередньо, це коли у вас є умовний код. Наприклад, код, який виглядає так, відчайдушно потребує StringBuilder:

// 1 StringBuilder used in this line
String query = "select id1, " + fieldName + " from " + tableName;
if (where != null) {
   // another StringBuilder used here
   query += ' ' + where;
}

У +першому рядку використовується один StringBuilderекземпляр. Потім +=використовує інший StringBuilderекземпляр. Більш ефективно зробити:

// choose a good starting size to lower chances of reallocation
StringBuilder sb = new StringBuilder(64);
sb.append("select id1, ").append(fieldName).append(" from ").append(tableName);
// conditional code
if (where != null) {
   sb.append(' ').append(where);
}
return sb.toString();

Інший раз, коли я використовую a StringBuilder, я будую рядок із ряду викликів методів. Тоді я можу створити методи, які приймають StringBuilderаргумент:

private void addWhere(StringBuilder sb) {
   if (where != null) {
      sb.append(' ').append(where);
   }
}

Коли ви використовуєте a StringBuilder, ви повинні стежити за будь-яким використанням +одночасно:

sb.append("select " + fieldName);

Це +призведе до створення іншого внутрішнього StringBuilder. Звичайно, це повинно бути:

sb.append("select ").append(fieldName);

Нарешті, як зазначає @TJrowder, ви завжди повинні здогадуватися про розмір StringBuilder. Це дозволить заощадити на кількості char[]об’єктів, створених при збільшенні розміру внутрішнього буфера.

Question 5

Ви правильно вважаєте, що мета використання конструктора рядків не досягнута, принаймні не в повній мірі.

Однак, коли компілятор бачить вираз, "select id1, " + " id2 " + " from " + " table"він видає код, який фактично створює файлStringBuilder закулісне і додає до нього, тож кінцевий результат не так вже й поганий.

Але, звичайно, кожен, хто дивиться на цей код, повинен думати, що він є відсталим.

Question 6

У коді, який ви розмістили, не було б переваг, оскільки ви неправильно використовуєте StringBuilder. Ви створюєте однаковий рядок в обох випадках. Використовуючи StringBuilder, ви можете уникнути +операцій над рядками, використовуючи appendметод. Вам слід використовувати це таким чином:

return new StringBuilder("select id1, ").append(" id2 ").append(" from ").append(" table").toString();

У Java тип String - це незмінна послідовність символів, тому при додаванні двох рядків VM створює нове значення String із з’єднаними обома операндами.

StringBuilder забезпечує змінну послідовність символів, яку ви можете використовувати для зв’язку різних значень або змінних, не створюючи нових об’єктів String, і тому іноді це може бути ефективніше, ніж робота зі рядками

Це надає деякі корисні функції, як зміна вмісту послідовності символів, переданої як параметр всередині іншого методу, чого не можна зробити за допомогою рядків.

private void addWhereClause(StringBuilder sql, String column, String value) {
   //WARNING: only as an example, never append directly a value to a SQL String, or you'll be exposed to SQL Injection
   sql.append(" where ").append(column).append(" = ").append(value);
}

Більше інформації на веб-сторінці http://docs.oracle.com/javase/tutorial/java/data/buffers.html

Question 7

Ви можете також використовувати MessageFormat занадто