Java StringBuilder and Thread Safety

Question

Estoy construyendo una cadena de múltiples piezas y quiero usar StringBuffer o StringBuilder para hacerlo. A partir de los documentos de Java 5, veo que StringBuilder se prefiere cuando sea posible, con la advertencia de que

Las instancias de StringBuilder no son seguros para su uso por varios subprocesos.

De esta declaración, entiendo que no debería tener una única instancia StringBuilder compartida por varios subprocesos. Pero ¿qué pasa con este caso:

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

Aquí podría haber múltiples hilos en la función, al mismo tiempo, utilizando la clase StringBuilder al mismo tiempo (por ejemplo, el acceso simultáneo de variables estáticas, si los hay), pero cada hilo tendría su propia instancia separada de StringBuilder. De la documentación, no puedo decidir si esto cuenta como uso por múltiples hilos o no.

Answer 1

Eso está perfectamente bien. Las variables locales no tienen problemas con la seguridad de subprocesos, siempre que no accedan o muten las variables de instancia o clase.

Answer 2

Sí, eso es seguro, porque el objeto StringBuilder se usa solo localmente (cada hilo que llame a foo() generará su propio StringBuilder).

También debe tomar nota de que el código que envió es prácticamente idéntico al código de bytes generada por este:

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

Answer 3

acuerdo con las otras respuestas - sólo una nota.

Si hubo un caso en el que StringBuffer estaba siendo utilizado por varios hilos, probablemente sea un caso de uso completamente roto porque significaría que se estaba creando una sola secuencia en un orden casi aleatorio, por lo que no tendría sentido para hacer que el hilo StringBuffer sea seguro.

Answer 4

No estoy seguro de si este código es necesario, porque Java elige el StringBuilder automáticamente, supongo. Si no tiene un problema de rendimiento, vaya con a + b.

En caso de una necesidad de rendimiento, pruebe lo siguiente:

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

correctamente los tamaños de la memoria intermedia y evita que la máquina virtual de cambiar el tamaño y la creación de basura para recoger en el camino.

Answer 5

El código que tiene es seguro.

Este código no lo es.

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

Las variables locales que sólo hacen uso de datos locales no tienen hebras cuestiones. Solo puede tener problemas de seguridad de subprocesos una vez que comience a tratar con los datos que están visibles en la clase o en el método de instancia/nivel de variable.