Java StringBuilder e sicurezza thread

Question

Sto costruendo una stringa di più parti e voglio usare StringBuffer o StringBuilder per farlo. Dalle Java 5 documenti, vedo che StringBuilder è preferito quando possibile, con l'avvertenza che

istanze di StringBuilder non sono sicure per i più thread.

Da questa affermazione, ho capito che non dovevo avere un unico StringBuilder istanza condivisa da più thread. Ma per quanto riguarda questo caso:

//Is this safe? 
//foo() is called simultaneously by multiple threads 
String foo(String a, String b) { 
    return new StringBuilder(a).append(b).toString(); 
} 

Qui ci potrebbero essere più filettature in funzione contemporaneamente, utilizzando la classe StringBuilder allo stesso tempo (ad esempio, l'accesso concorrente di variabili statiche, se ve ne sono), ma ogni thread avrebbe una propria istanza separata di StringBuilder. Dalla documentazione, non riesco a decidere se questo conta come se fosse usato da più thread o meno.

Answer 1

Questo va benissimo. Le variabili locali non hanno problemi con la sicurezza del thread, a condizione che non accedano o mutino variabili di istanza o classe.

Answer 2

Sì, questo è sicuro, perché l'oggetto StringBuilder viene utilizzato solo localmente (ogni thread che chiama foo() genererà il proprio StringBuilder).

Si dovrebbe inoltre notare che il codice che avete inviato è praticamente identico al bytecode generato da questo:

String foo(String a, String b) { 
    return a + b; 
} 

Answer 3

d'accordo con le altre risposte - solo una nota.

Se esisteva un caso in cui StringBuffer veniva utilizzato da più thread, è probabilmente un caso d'uso completamente interrotto perché significherebbe che una singola stringa veniva creata in un ordine quasi casuale, quindi non avrebbe avuto senso rendere sicuro il thread StringBuffer.

Answer 4

Non sono sicuro se questo codice è necessario, perché Java sceglie automaticamente lo StringBuilder. Se non si dispone di un problema di prestazioni, andare con un + b.

In caso di necessità prestazioni, provare che:

return new StringBuilder(
a.length() + b.length()).append(a).append(b).toString(); 

E formati correttamente il buffer e impedisce la VM di ridimensionare e creando spazzatura per raccogliere sulla strada.

Answer 5

Il codice che hai è sicuro.

Questo codice non è.

public class Foo 
{ 
    // safe 
    private final static StringBuilder builder; 

    public static void foo() 
    { 
     // safe 
     builder = new StringBuilder(); 
    } 

    public static void foo(final String a) 
    { 
     // unsafe 
     builder.append(a); 
    } 

    public synchronized void bar(final String a) 
    { 
     // safe 
     builder.append(a); 
    } 
} 

Le variabili locali che utilizzano solo dati locali non presentano problemi di tipo thread. È possibile avere problemi con il thread solo dopo aver iniziato a gestire i dati visibili alla classe o al metodo di istanza/livello di variabile.