Está muy bien, no lo cambies prematuramente.
A la hora de hacer un programa hay varias cosas que son importantes, entre otras:
- El consumo de CPU
- El consumo de memoria, disco y otros recursos.
- La corrección. Que el programa haga lo que de el se espera.
El tercer punto siempre es importante. En todas las línes de código es importante que hagan lo que de ellas se espera.
Los otros dos puntos no siempre son igual de importantes. Si tu concatenación de cadenas consume el 0.01% del tiempo de tu programa poco vas a ganar optimizándola. Si consume el 20% entonces sí vale la pena.
Para hacer que un programa sea correcto es importante utilizar código que sea sencillo y facil de entender y mantener.
El código que has escrito :
String all = nombre + prefijo;
cumple esos requisitos.
Un código potencialmente más eficiente como :
StringBuilder b = new StringBuilder( nombre.length() + prefijo.length() );
b.append(nombre);
b.append(prefijo);
String all = b.toString();
no es mucho más difícil de entender. Pero si siempre programas así un programa de 10,000 líneas tendrá 40,000 y ahí sí vas a notar la diferencia.
¿Por qué perjudicar la legibilidad de tu programa para obtener una mejora de eficiencia que podría ser insignificante?
Conclusión : esfuérzate en que tu código sea sencillo, mantenible y correcto.
Y cuando funcione bien pásale un profiler y optimiza los cuellos de botella.
¿Es StringBuilder más rápido en este caso?
Supongamos que, por el motivo que sea, hemos llegado a la conclusión de que su rendimiento es crítico. ¿Usar StrinBuilder es más rápido?
Consideremos el código con el operador +:
class A {
static void test() {
String a = "hola !";
String b = "\nStackoverflow !";
String resultado = a + b;
}
}
Y el código con StringBuilder:
class A {
static void test() {
String a = "hola !";
String b = "\nStackoverflow !";
StringBuilder sb = new StringBuilder();
sb.append(a);
sb.append(b);
String resultado = sb.toString();
}
}
Este es el código en bytecodes para el caso del operador + :
C:\test>"C:\dev\Android\Android Studio\jre\bin\javac" A.java
C:\test>"C:\dev\Android\Android Studio\jre\bin\javap" -c A
Compiled from "A.java"
class A {
A();
Code:
100: aload_0
101: invokespecial #1 // Method java/lang/Object."<init>":
()V
102: return
static void test();
Code:
1: ldc #2 // String hola !
2: astore_0
3: ldc #3 // String \nStackoverflow !
4: astore_1
5: new #4 // class java/lang/StringBuilder
6: dup
7: invokespecial #5 // Method java/lang/StringBuilder."<init>":()V
8: aload_0
9: invokevirtual #6 // Method java/lang/StringBuilder.append:(Ljava/lang/String;)Ljava/lang/StringBuilder;
10: aload_1
11: invokevirtual #6 // Method java/lang/StringBuilder.append:(Ljava/lang/String;)Ljava/lang/StringBuilder;
12: invokevirtual #7 // Method java/lang/StringBuilder.toString:()Ljava/lang/String;
13: astore_2
14: return
}
Y este es el código en bytecodes para el caso del StringBuilder :
C:\test>"C:\dev\Android\Android Studio\jre\bin\javac" A.java
C:\test>"C:\dev\Android\Android Studio\jre\bin\javap" -c A
Compiled from "A.java"
class A {
A();
Code:
100: aload_0
101: invokespecial #1 // Method java/lang/Object."<init>":
()V
102: return
static void test();
Code:
1: ldc #2 // String hola !
2: astore_0
3: ldc #3 // String \nStackoverflow !
4: astore_1
5: new #4 // class java/lang/StringBuilder
6: dup
7: invokespecial #5 // Method java/lang/StringBuilder."<init>":()V
7b: astore_2
8: aload_2
8b: aload_0
9: invokevirtual #6 // Method java/lang/StringBuilder.append:(Ljava/lang/String;)Ljava/lang/StringBuilder;
9b: pop
9c: aload_2
10: aload_1
11: invokevirtual #6 // Method java/lang/StringBuilder.append:(Ljava/lang/String;)Ljava/lang/StringBuilder;
11b: pop
11c: aload_2
12: invokevirtual #7 // Method java/lang/StringBuilder.toString:()Ljava/lang/String;
13: astore_3
14: return
}
Los números que salen (del 1 al 14, y los de las letras) no son reales, los he modificado yo para que resulte más fácil de entender la explicación.
Lo primero que llama la atención es que el código del operador + se compila a StringBuilder.
Lo segundo es que el código del StringBuilder tiene las mismas 14 instruciones de bytcode que el del operador + (las líneas sin letra b o c). Excepto por diferencias en el parámetro, como en la línea 8 (astore_0 o astore_2), pero esta diferencia no afecta al rendimiento.
Lo que sí afecta son las 6 instrucciones extra de bytecode (7b, 8b, 9b, 9c, 11b y 11c) que tiene el código de StringBuilder.
No es código bytecode de JVM lo que le llega al dispositivo Android. Este se convierte a DEX. Y luego a código nativo. Pero con lo visto ya está claro. Si uno de esos pasos intermedios optimizase el código del StringBuilder como mucho alcanzaría al del operador + optimizado, en ningún caso lo mejoraría.
Conclusión: StringBuilder es más lento en este caso en particular.
¿Debemos renegar pues de Stringbuilder y usar siempre el operador +?
Por supuesto que no. Para concatenar dos cadenas es mejor el operador +, pero considera el caso de un array de 1000 cadenas en que quieres concatenarlas todas para hacer una sola cadena.
Me atrevería a decir que con StringBuilder será más rápido:
String[] cadenas = new String[1000];
// ... Código que pone 1000 cadenas en el array ...
StringBuilder sb = new StringBuilder();
for ( String cad : cadenas )
sb.append(cad);
String resultado = sb.toString();
Y no solo será más rápido(1) sino que ese código también es igual de fácil de entender y mantenible que la alternativa con el operador +:
String[] cadenas = new String[1000];
// ... Código que pone 1000 cadenas en el array ...
String resultado = "";
for ( String cad : cadenas )
resultado += cad;
(1) Salvo que el compilador también sea capaz de optimizar el código del operador +.