Ist es immer eine schlechte Idee + verwenden Strings verketten

Question

I-Code haben wie folgt:

String s = ""; 
for (My my : myList) { 
    s += my.getX(); 
} 

Findbugs immer meldet Fehler, wenn ich dies tun.

Answer 1

würde ich + verwenden, wenn Sie manuell verketten,

String word = "Hello"; 
word += " World!"; 

Wenn Sie jedoch iterieren und verketten Ich würde StringBuilder vorschlagen,

StringBuilder sb = new StringBuilder(); 
for (My my : myList) { 
    sb.append(my.getX()); 
} 

Answer 2

Das String-Objekt ist in Java unveränderlich. Jedes + bedeutet ein anderes Objekt. Sie können StringBuffer verwenden, um die Anzahl der erstellten Objekte zu minimieren.

Answer 3

Der Compiler etwas, was wie

optimieren

"foo" + "bar"

zu

StringBuilder s1 = new StringBuilder(); s1.append ("foo"). Append ("bar");

Dies ist jedoch immer noch suboptimal, da es mit einer Standardgröße von 16 beginnt. Wie bei vielen Dingen sollten Sie jedoch Ihre größten Flaschenhälse finden und sich in der Liste nach unten arbeiten. Es tut nicht weh, in der Gewohnheit zu sein, ein SB-Muster von dem Get-Go zu verwenden, besonders wenn Sie in der Lage sind, eine optimale Initialisierungsgröße zu berechnen.

Answer 4

Vorzeitige Optimierung kann sowohl schlecht als auch oft die Lesbarkeit beeinträchtigen und ist normalerweise völlig unnötig. Verwenden Sie +, wenn es lesbarer ist, es sei denn, Sie haben tatsächlich ein vorrangiges Problem.

Answer 5

Es ist nicht "immer schlecht", "+" zu verwenden. Die Verwendung von StringBuffer überall kann Code sehr umfangreich machen.

Wenn jemand eine Menge "+" mitten in eine intensive, zeitkritische Schleife steckt, wäre ich genervt. Wenn jemand eine Menge "+" in ein selten verwendetes Stück Code steckt, wäre mir das egal.

Answer 6

würde ich sagen, Verwendung und in den folgenden:

String c = "a" + "b"

und überall sonst Stringbuilder-Klasse verwenden. Wie bereits im ersten Fall erwähnt, wird es vom Compiler optimiert und besser lesbar.

Answer 7

Jedes Mal, wenn Sie string+=string tun, ruft es Verfahren wie folgt aus:

private String(String s1, String s2) { 
    if (s1 == null) { 
     s1 = "null"; 
    } 
    if (s2 == null) { 
     s2 = "null"; 
    } 
    count = s1.count + s2.count; 
    value = new char[count]; 
    offset = 0; 
    System.arraycopy(s1.value, s1.offset, value, 0, s1.count); 
    System.arraycopy(s2.value, s2.offset, value, s1.count, s2.count); 
} 

Bei String, kommt es zu:

final void append0(String string) { 
    if (string == null) { 
     appendNull(); 
     return; 
    } 
    int adding = string.length(); 
    int newSize = count + adding; 
    if (newSize > value.length) { 
     enlargeBuffer(newSize); 
    } 
    string.getChars(0, adding, value, count); 
    count = newSize; 
} 

Wie man deutlich abschließen kann, string + string schafft eine Menge Overhead, und meiner Meinung nach sollte vermieden werden, wenn möglich.Wenn Sie denken, mit String sperrig ist oder zu lange können Sie nur eine Methode machen und es indirekt verwenden, wie:

public static String scat(String... vargs) { 
    StringBuilder sb = new StringBuilder(); 

    for (String str : vargs) 
     sb.append(str); 

    return sb.toString(); 
} 

Und verwenden Sie es mögen:

String abcd = scat("a","b","c","d"); 

In C# ich seine über gesagt bin genauso wie string.Concat();. In Ihrem Fall wäre es klug, Überlastung für kv zu schreiben, wie:

public static String scat(Collection<?> vargs) { 
    StringBuilder sb = new StringBuilder(); 

    for (Object str : vargs) 
     sb.append(str); 

    return sb.toString(); 
} 

Dann können Sie es mit rufen:

result = scat(myList) 

Answer 8

Einer der Gründe, warum FindBugs über die Verwendung von Verkettungsoperator argumentieren sollte (sein es "+" oder "+ =") ist Lokalisierbarkeit. Im Beispiel gab man es nicht so offensichtlich ist, aber im Fall des folgenden Codes ist:

String result = "Scanning found " + Integer.toString(numberOfViruses) + " viruses"; 

Wenn dies ein wenig vertraut aussieht, müssen Sie Ihren Programmierstil ändern. Das Problem ist, dass es auf Englisch gut klingen wird, aber es könnte ein Albtraum für Übersetzer sein. Das liegt nur daran, dass Sie nicht garantieren können, dass die Reihenfolge des Satzes nach der Übersetzung immer noch dieselbe ist - manche Sprachen werden in "1 blah blah" übersetzt, einige in "blah blah 3". In solchen Fällen sollten Sie immer MessageFormat.format() verwenden, um zusammengesetzte Sätze zu erstellen, und den Verkettungsoperator zu verwenden, ist eindeutig ein Internationalisierungsfehler.

BTW. Ich habe hier einen weiteren Fehler gemacht, könntest du es entdecken?

Answer 9

Die Laufzeit der Verkettung zweier Strings ist proportional zur Länge der Strings. Wenn es in einer Schleife verwendet wird, wird die Laufzeit immer größer. Also, wenn Verkettung in einer Schleife benötigt wird, ist es besser, StringBuilder wie Anthony vorgeschlagen.