deleteCharAt или setLength, в каком направлении лучше удалить последний символ из StringBuilder/StringBuffer

Question

Во многих случаях нам нужно удалить последний символ StringBuilder/StringBuffer. Например, с учетом int[]{1,2,3}, для реализации метода String toString(int[] a), контактирующего с каждым элементом с разделителем запятой. Выход должен быть 1,2,3, без хвостовой запятой.

Мы можем легко написать цикл:

int[] nums = new int[]{1,2,3,4,5}; 
StringBuilder sb = new StringBuilder(); 
for (int i = 0; i < nums.length; i++) { 
    sb.append(nums[i]); 
    sb.append(","); 
} 
//here we need to remove the tailing ',' 

но всегда нам нужно удалить хвостохранилища ','. Есть два способа реализации его:

sb.deleteCharAt(sb.length() - 1); 

и

sb.setLength(sb.length() - 1); 

Какой рекомендуется? Зачем?

ПРИМЕЧАНИЕ: Я знаю, что делает Arrays.toString. Это всего лишь пример, чтобы описать мой вопрос, может быть, не совсем правильно. Это не обсуждение конкатенации строк, а лучшие практики StringBuffer/StringBuilder.

Answer 1

На самом деле, в нем очень мало и, вероятно, зависит от оборудования и других факторов.

Метод setLength() просто изменяет счетчик и перезаписывает ненужное значение в массиве с нулевым байтом.

deleteCharAt() выполняет внутреннюю копию массива, прежде чем изменять счетчик. Это звучит драматично, но копируемый массив фактически равен нулю, потому что вы удаляете последний символ.

Я бы рекомендовал для setLength(), так как он короче набирать, и я думаю, что он проясняет, что вы делаете. Если производительность является проблемой, и при измерении вы обнаружите, что это узкое место для вас, то, возможно, вы могли бы рассмотреть другой алгоритм, который не требует изменения размера (согласно ответам JB Nizet).

Answer 2

Я бы так не сделал. Вместо этого я бы добавил только конечную запятую, если элемент не является последним элементом массива. Или я хотел бы использовать Столяр гуавы (или Apache-Commons StringUtils), что делает его гораздо яснее:

String s = Joiner.on(',').join(nums); 

NB: Я просто заметил, что Столяр гуавы в не занимается примитивными массивами. Вы все равно должны получить эту идею.

Answer 3

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

for (int i = 0; i < nums.length; i++) { 
    if (i > 0) 
     sb.append(','); 
    sb.append(nums[i]); 
} 

Тогда вам не нужно беспокоиться об удалении последнего символа, потому что это уже правильно.