StringBuffer Append («»)

Question

Я currentlly рефакторинга приложения, использующее много этого:

StringBuffer buff1 = new StringBuffer(""); 

buff1.append("some value A"); 
buff1.append(""); 
buff1.append("some value B"); 

кодера, который сделал эти строки коды не кажется идиот, есть ли причины I не можете использовать append ("") для StringBuffer?

Answer 1

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

Answer 2

Нет, вам это не нужно append(""). Вам также не нужно инициализировать StringBuffer с помощью String. new StringBuffer(); будет работать нормально.

Answer 3

Nope.

И учитывая, что они жестко закодированных строк, было бы идентичным написать

buff1 = new StringBuffer("some value Asome value B"); 

Кстати, это немного более эффективно использовать StringBuilder, а не StringBuffer (и API идентичен).

Answer 4

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

Answer 5

Если нет никаких дополнительных Присоединяет, сделанные в StringBuffer код должен быть переписан в

String buff1 = "some value some value B"; 

Это более кратким, более удобным для чтения и безопаснее, чем:

StringBuffer buff1 = new StringBuffer(""); 

buff1.append("some value A"); 
buff1.append("some value B"); 

Я говорю «безопаснее» потому что String неизменен и StringBuffer неизменен, поэтому нет никакого риска, что String случайно изменится после построения.

---

Помимо

Это распространенное заблуждение, что "конкатенации строк в Java плохо". Хотя это правда, что вы не должны писать код, как этот

String buff1 = "foo";  
buff1 += "some value A"; 
buff1 += "some value B"; 

Это вполне приемлемо, чтобы написать такой код:

String buff1 = "foo" + "some value A" + "some value B"; 

когда конкатенация выполняется в одном операторе код будет оптимизирован :

String buff1 = "foo some value A some value B"; 

Answer 6

Возможно, он намеревался поставить buff1.append(" ");. Просто я беру.

Answer 7

buff1.append ("") не имеет значения и его следует удалить.