Usunąć ostatni znak StringBuilder?

Gdy musisz przejrzeć kolekcję i utworzyć ciąg danych oddzielonych separatorem, zawsze kończy się to dodatkowym separatorem na końcu, np.

for (String serverId : serverIds) {
  sb.append(serverId);
   sb.append(",");
}

Daje coś w rodzaju: serverId_1, serverId_2, serverId_3,

Chciałbym usunąć ostatni znak w StringBuilder (bez konwersji, ponieważ nadal potrzebuję go po tej pętli).

Inni wskazali tę deleteCharAtmetodę, ale oto inne alternatywne podejście:

String prefix = "";
for (String serverId : serverIds) {
  sb.append(prefix);
  prefix = ",";
  sb.append(serverId);
}

Alternatywnie skorzystaj z Joinerklasy z Guawy :)

Począwszy od Java 8, StringJoinerjest częścią standardowego środowiska JRE.

Innym prostym rozwiązaniem jest:

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

Bardziej skomplikowane rozwiązanie:

Powyższe rozwiązanie zakłada, że sb.length() > 0... tzn. Istnieje „ostatni znak” do usunięcia. Jeśli nie możesz przyjąć tego założenia i / lub nie możesz poradzić sobie z wyjątkiem, który powstałby, gdyby założenie było niepoprawne, najpierw sprawdź długość StringBuilder; na przykład

// Readable version
if (sb.length() > 0) {
   sb.setLength(sb.length() - 1);
}

lub

// Concise but harder-to-read version of the above.
sb.setLength(Math.max(sb.length() - 1, 0));

if(sb.length() > 0){
    sb.deleteCharAt(sb.length() - 1);
}

Od wersji Java 8 klasa String ma metodę statyczną join. Pierwszy argument to ciąg znaków, który chcesz między każdą parą ciągów, a drugi to Iterable<CharSequence>(które są obydwoma interfejsami, więc coś w rodzaju List<String>działa. Więc możesz po prostu to zrobić:

String.join(",", serverIds);

Również w Javie 8 możesz użyć nowej StringJoinerklasy do scenariuszy, w których chcesz zacząć konstruować ciąg, zanim będziesz mieć pełną listę elementów do umieszczenia w nim.

Wystarczy uzyskać pozycję ostatniego wystąpienia postaci.

for(String serverId : serverIds) {
 sb.append(serverId);
 sb.append(",");
}
sb.deleteCharAt(sb.lastIndexOf(","));

Ponieważ lastIndexOfprzeprowadzi wyszukiwanie wsteczne i wiesz, że znajdzie się przy pierwszej próbie, wydajność nie będzie tutaj problemem.

EDYTOWAĆ

Ponieważ ciągle podnoszę swoją odpowiedź (dzięki ludzie 😊), warto wziąć pod uwagę, że:

W Javie 8 i nowszej byłoby po prostu bardziej czytelne i wyraźne korzystanie z StringJoiner . Ma jedną metodę prostego separatora oraz przeciążenie prefiksu i sufiksu.

Przykłady zaczerpnięte stąd: przykład

Przykład użycia prostego separatora:

    StringJoiner mystring = new StringJoiner("-");    

    // Joining multiple strings by using add() method  
    mystring.add("Logan");  
    mystring.add("Magneto");  
    mystring.add("Rogue");  
    mystring.add("Storm");  

    System.out.println(mystring);

Wynik:

Logan-Magneto-Rogue-Storm

Przykład z przyrostkiem i przedrostkiem:

    StringJoiner mystring = new StringJoiner(",", "(", ")");    

    // Joining multiple strings by using add() method  
    mystring.add("Negan");  
    mystring.add("Rick");  
    mystring.add("Maggie");  
    mystring.add("Daryl");  

    System.out.println(mystring);

Wynik

(Negan, Rick, Maggie, Daryl)

W tym przypadku,

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

jest preferowane, ponieważ po prostu przypisuje ostatnią wartość, '\0'podczas gdy usuwa ostatni znakSystem.arraycopy

Za Java-8pomocą statycznej metody Stringklasy

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));
    }
}

WYNIK

James,Harry,Roy

Kolejna alternatywa

for(String serverId : serverIds) {
   sb.append(",");
   sb.append(serverId); 
}
sb.deleteCharAt(0);

Alternatywnie,

StringBuilder result = new StringBuilder();
for(String string : collection) {
    result.append(string);
    result.append(',');
}
return result.substring(0, result.length() - 1) ;

StringBuilder sb = new StringBuilder();
sb.append("abcdef");
sb.deleteCharAt(sb.length() - 1);
assertEquals("abcde",sb.toString());
// true

Jeszcze jedna alternatywa:

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();
}

Aby uniknąć prefixponownego uruchomienia (wpłynąć na wydajność) użycia TextUtils.isEmpty:

            String prefix = "";
            for (String item : list) {
                sb.append(prefix);
                if (TextUtils.isEmpty(prefix))
                    prefix = ",";
                sb.append(item);
            }

Możesz spróbować użyć klasy „Joiner” zamiast usuwać ostatni znak z wygenerowanego tekstu;

                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"

Robię coś takiego:

    StringBuilder stringBuilder = new StringBuilder();
    for (int i = 0; i < value.length; i++) {
        stringBuilder.append(values[i]);
        if (value.length-1) {
            stringBuilder.append(", ");
        }
    }

Oto inne rozwiązanie:

for(String serverId : serverIds) {
   sb.append(",");
   sb.append(serverId); 
}

String resultingString = "";
if ( sb.length() > 1 ) {
    resultingString = sb.substring(1);
}

Osobiście lubię dodawać znak backspace (lub więcej dla dłuższego „separatora”) na końcu:

for(String serverId : serverIds) {
    sb.append(serverId);
    sb.append(",");
}

sb.append('\b');

Pamiętaj, że ma problemy:

• sposób \bwyświetlania zależy od środowiska,
• length()o Stringtreści może być różna od długości znaków „widoczny”

Kiedy \bwygląda OK, a długość nie ma znaczenia, np. Logowanie do konsoli, wydaje mi się to wystarczająco dobre.

stringBuilder.Remove (stringBuilder.Length - 1, 1);