Не знакомы с Доктриной, но проверьте одну точку: проверьте возможные исключения в __construct () / __ destruct (), они могут приводить к фатальным ошибкам.
Другие указали на метод deleteCharAt
, но вот еще один альтернативный подход:
String prefix = "";
for (String serverId : serverIds) {
sb.append(prefix);
prefix = ",";
sb.append(serverId);
}
В качестве альтернативы, используйте класс Joiner
из Гуава :)
Начиная с Java 8, StringJoiner
является частью стандарта JRE.
Мне лично нравится добавлять символ возврата (или более для более длинного «разделителя») в конце:
for(String serverId : serverIds) {
sb.append(serverId);
sb.append(",");
}
sb.append('\b');
Обратите внимание, что у него есть проблемы:
\b
отображается в зависимости от среды, length()
содержимого String
может отличаться от длины «видимых» символов Когда \b
выглядит Хорошо, и длина не имеет значения, например, для входа в консоль, мне это кажется достаточно хорошим.
Вот еще одно решение:
for(String serverId : serverIds) {
sb.append(",");
sb.append(serverId);
}
String resultingString = "";
if ( sb.length() > 1 ) {
resultingString = sb.substring(1);
}
Я делаю что-то вроде ниже:
StringBuilder stringBuilder = new StringBuilder();
for (int i = 0; i < value.length; i++) {
stringBuilder.append(values[i]);
if (value.length-1) {
stringBuilder.append(", ");
}
}
Еще одна альтернатива:
public String join(Collection<String> collection, String seperator) {
if (collection.isEmpty()) return "";
Iterator<String> iter = collection.iterator();
StringBuilder sb = new StringBuilder(iter.next());
while (iter.hasNext()) {
sb.append(seperator);
sb.append(iter.next());
}
return sb.toString();
}
StringBuilder sb = new StringBuilder();
sb.append("abcdef");
sb.deleteCharAt(sb.length() - 1);
assertEquals("abcde",sb.toString());
// true
В качестве альтернативы,
StringBuilder result = new StringBuilder();
for(String string : collection) {
result.append(string);
result.append(',');
}
return result.substring(0, result.length() - 1) ;
Вы можете попробовать использовать класс Joiner вместо удаления последнего символа из вашего сгенерированного текста;
List<String> textList = new ArrayList<>();
textList.add("text1");
textList.add("text2");
textList.add("text3");
Joiner joiner = Joiner.on(",").useForNull("null");
String output = joiner.join(textList);
//output : "text1,text2,text3"
Еще одна альтернатива
for(String serverId : serverIds) {
sb.append(",");
sb.append(serverId);
}
sb.deleteCharAt(0);
Чтобы избежать повторного ввода (повлиять на производительность) prefix
, используйте TextUtils.isEmpty:
String prefix = "";
for (String item : list) {
sb.append(prefix);
if (TextUtils.isEmpty(prefix))
prefix = ",";
sb.append(item);
}
С Java-8
вы можете использовать статический метод класса String
,
String#join(CharSequence delimiter,Iterable<? extends CharSequence> elements)
.
public class Test {
public static void main(String[] args) {
List<String> names = new ArrayList<>();
names.add("James");
names.add("Harry");
names.add("Roy");
System.out.println(String.join(",", names));
}
}
ВЫХОД
James,Harry,Roy
Начиная с Java 8, класс String имеет статический метод join
. Первый аргумент - это строка, которую вы хотите видеть между каждой парой строк, а второй - Iterable<CharSequence>
(оба являются интерфейсами, поэтому работает что-то вроде List<String>
. Поэтому вы можете просто сделать это:
String.join(",", serverIds);
Также в Java 8 вы можете использовать новый класс StringJoiner
для сценариев, в которых вы хотите начать конструирование строки до того, как у вас будет полный список элементов для ее вставки.
В этом случае
sb.setLength(sb.length() - 1);
предпочтительнее, так как он просто присваивает последнее значение '\0'
, тогда как удаление последнего символа делает System.arraycopy
Другое простое решение:
sb.setLength(sb.length() - 1);
Более сложное решение:
Приведенное выше решение предполагает, что sb.length() > 0
... то есть существует «последний символ» удалять. Если вы не можете сделать это предположение и / или не можете справиться с исключением, которое возникнет, если это предположение неверно, сначала проверьте длину StringBuilder; например,
// Readable version
if (sb.length() > 0) {
sb.setLength(sb.length() - 1);
}
или
// Concise but harder-to-read version of the above.
sb.setLength(Math.max(sb.length() - 1, 0));
Просто получите позицию последнего вхождения персонажа.
for(String serverId : serverIds) {
sb.append(serverId);
sb.append(",");
}
sb.deleteCharAt(sb.lastIndexOf(","));
Поскольку lastIndexOf
будет выполнять обратный поиск, и вы знаете, что он найдет с первой попытки, производительность здесь не будет проблемой.
РЕДАКТИРОВАТЬ
Поскольку я продолжаю поднимать свой ответ (спасибо, ребята