19

Quiero realizar la concatenación de dos String:

String nombre = adornoF.getNombre();
String prefijo = ga.getsid();
String all = nombre + prefijo;

Tanto getNombre() como getsid() devuelven un String.

¿Está bien planteado o hay una manera mejor?

3
  • Como un ejemplo puedes realizarlo así, pero como buena práctica, por ejemplo si es un proyecto en el cual deseas escalabilidad, es mejor optimizar desde un inicio.
    – Jorgesys
    Commented el 25 abr. 2017 a las 19:12
  • De acuerdo con lo planteado en Meta en ¿Alteración del significado de la pregunta: Concatenar 2 String? (aceptado y con varios votos a favor), procedo a revertir las dos últimas ediciones.
    – fedorqui
    Commented el 28 abr. 2017 a las 8:34
  • Borch: como ves, tu pregunta está recibiendo múltiples revisiones. ¿Cuál era tu objetivo final a la hora de plantearla? ¿Preguntar por dos o múltiples strings?
    – fedorqui
    Commented el 10 may. 2017 a las 8:29

7 respuestas 7

15

Las formas de concatenar dos String son:


Usando el operador + :

String a = "hola !";
String b = "\nStackoverflow !";    
String resultado = a + b;

mediante el método concat() :

String a = "hola !";
String b = "\nStackoverflow !";
String resultado = a.concat(b);

Usando la clase StringBuffer y el método append() :

String a = "hola !";
String b = "\nStackoverflow !";
StringBuffer sb = new StringBuffer();
sb.append(a);
sb.append(b);
String resultado = sb.toString();

Usando la clase StringBuilder, y el método append() :

String a = "hola !";
String b = "\nStackoverflow !";
StringBuilder sb = new StringBuilder();
sb.append(a);
sb.append(b);
String resultado = sb.toString();

En los 3 casos obtendrías el mismo resultado, sin embargo como buena practica lo mejor es usar StringBuilder para concatenar el text en el buffer y obtener un solo objeto String mediante toString().


Documentación Android:

3
  • 1
    Puesto que esto está etiquetado como android, entonces sería mejor poner enlaces a https://developer.android.com/reference. Commented el 25 abr. 2017 a las 18:25
  • 1
    A mi me gusta más apostar por la optimización y consistencia del código, a la larga gratifica más la experiencia de usuario @Jorseys +1
    – Codelaby
    Commented el 26 abr. 2017 a las 10:06
  • 1
    Dejando aparte un momento la cuestión de la mantenibilidad del código. Despues de analizar en profundidad esta cuestión he llegado a conclusión de que usar el operador +, en este caso en particular, es más eficiente que usar StringBuilder. Al final de mi respuesta tienen la demostración. @Webserveis Commented el 26 abr. 2017 a las 22:23
7

Así está bien, también puedes gastar la función concat() de java de la siguiente forma:

String all = prefijo.concat(nombre);
7

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 +.

11
  • 1
    ¿según tu, no optimizar al inicio es correcto?, tal vez piensas que el OP esta realizando un "hello world" ⤜(ʘ_ʘ)⤏ , lee el articulo que tu mismo comentas, "diseñar primero, luego codificar el diseño", se puede perder tiempo realizando código "sencillo" como concatenar con + que al final posiblemente terminarás cambiando!
    – Jorgesys
    Commented el 25 abr. 2017 a las 18:26
  • 1
    Lo dice Donald Knuth, The Art of Computer Programming Pág671:Programmers waste enormous amounts of time thinking about, or worrying about, the speed of noncritical parts of their programs, and these attempts at efficiency actually have a strong negative impact when debugging and maintenance are considered. We should forget about small efficiencies, say about 97% of the time: premature optimization is the root of all evil. Yet we should not pass up our opportunities in that critical 3% Commented el 25 abr. 2017 a las 18:48
  • 1
    He leído lo que comenta el DonaldK Knuth, Como ejemplo puedes realizar una aplicación desktop sin preocuparte en varios casos de "Programmers waste enormous amounts of time thinking about, or worrying about, the speed of noncritical parts of their programs" ...Pero en el caso de dispositivos con recursos limitados, créeme que es muy diferente.
    – Jorgesys
    Commented el 25 abr. 2017 a las 19:23
  • 1
    Bueno primeramente estaban mofandose con "No vas a necesitarlo " y ahora realizas este análisis, para que un análisis si no es necesario? chat.stackexchange.com/transcript/message/36947864#36947864 Bueno entonces la recomendación es usar + ?
    – Jorgesys
    Commented el 26 abr. 2017 a las 22:58
  • 1
    ¿Pero es prueba no seria mejor hacerla correr en runtime en Android con un dispositivo y ver si da lo mismo?
    – Codelaby
    Commented el 27 abr. 2017 a las 8:01
7

Hay otra manera es usar StringBuilder

StringBuilder sb = new StringBuilder();
sb.append(nombre);
sb.append(prefijo);
String cadena_final = sb.toString();

En bucles es más óptimo usar StringBuilder

StringBuilder sb= new StringBuilder() ;
for(int i= 0 ; i < 10000 ; i++ )
    sb.append( "a" );
String s= sb.toString() ;

Extraído en SO

Aquí hay la discusión cual es mejor evidentemente depende de su uso, si es solo una vez pues mejor usar + pero si es para ir construyendo una cadena a partir de un bucle, pues mejor StringBuilder

6
  • 1
    has sido ninja'deado ;) xDD
    – lois6b
    Commented el 25 abr. 2017 a las 14:52
  • 1
    La discusión no es del todo exacta. Hay que tomar en cuenta que Android Runtime ≠ Java Runtime Environment. Commented el 25 abr. 2017 a las 15:05
  • 1
    @PaulVargas el Android Studio cuando detecta que usas string + string, te sale un warning que te recomiendo usar StringBuilder, hasta tiene un conversor automatico alt+intro. Supongo que es solo si usas muchos concats donde realmente le sacas el jugo.
    – Codelaby
    Commented el 25 abr. 2017 a las 15:07
  • 1
    Android Studio puede recomendar lo que quiera... También eclipse, NetBeans, etc., Pero... ¿qué hace el compilador de Android? ¿Utiliza internamente una instancia (o varias) de StringBuilder para reemplazar el operador +? ¿Qué tipo de optimización hace (como la JVM de IBM)? Commented el 25 abr. 2017 a las 15:30
  • Esto es lo mejor, usar StringBuilder. como buena práctica, donde podemos observar beneficios cuando nuestros sistemas tienden a ser robustos!.
    – Jorgesys
    Commented el 25 abr. 2017 a las 15:32
1

Una opción que nadie ha mencionado es usar String.format:

String s1 = "hola ";
String s2 = "mundo!";
String concatenado = String.format("%s%s", s1, s2);
System.out.println(concatenado);

Resultado:

hola mundo!
2
  • ¿Qué ventajas aporta esa forma de hacerlo con respecto a la que usa el autor de la pregunta? Commented el 9 may. 2017 a las 18:30
  • Que puedes aplicar algún formato extra sin concatenar más cadenas y es más sencillo de mantener que la concatenación por +. En términos de performance, 0.0001 y 0.001 no hacen daño respecto al performance general de la aplicación.
    – user227
    Commented el 9 may. 2017 a las 19:06
0

Bueno, vamos a hacer un reto a los cuatro. Digamos a:

  • operador +
  • concat
  • StringBuffer
  • StringBuilder

que nos concatenen una cadena:

  • a. 100,000 veces
  • b. 10,000 veces
  • c. 1,000 veces
  • d. 100 veces
  • e. 2 veces

y veamos cuál es más eficiente de los 4.

En cada prueba se usa el código tal y como funciona internamente.

Nota:

La prueba ha sido hecha en Java, en una PC. Sería interesante correrla en Android (quizá con menos valores, por ejemplo 10,000 cadenas)... sería interesante ver el resultado.

El código original es de las 100,000 veces. Para otros valores cambiar la variable ITERATION:

Código:

import java.util.*;
import java.lang.*;
import java.io.*;

class Pruebas{

    public static void main(String args[]) throws IOException {
        final int ITERATION = 100_000;
        String s = "";

        // String Concatenation using + operator
        long startTime = System.nanoTime();
        for (int i = 0; i < ITERATION; i++) {
            s = s + Integer.toString(i);
        }
        long duration = (System.nanoTime() - startTime) / 1000;
        System.out.println("operador +  \t Tiempo (en micro) para " + ITERATION + " cadenas " + duration);

        // Using String concat() method
        startTime = System.nanoTime();
        for (int i = 0; i < ITERATION; i++) {
            s.concat(Integer.toString(i));

        }
        duration = (System.nanoTime() - startTime) / 1000;
        System.out.println("concat () \t Tiempo (en micro) para " + ITERATION + " cadenas " + duration);

    // StringBuffer example to concate String in Java
        StringBuffer buffer = new StringBuffer(); // default size 16
        startTime = System.nanoTime();
        for (int i = 0; i < ITERATION; i++) {
            buffer.append(i);
        }

        duration = (System.nanoTime() - startTime) / 1000;
        System.out.println("StringBuffer \t Tiempo (en micro) para " + ITERATION + " cadenas "  + duration);

        // StringBuilder example to concate two String in Java
        StringBuilder builder = new StringBuilder(); 
        startTime = System.nanoTime();
        for (int i = 0; i < ITERATION; i++) {
            builder.append(i);
        }
        duration = (System.nanoTime() - startTime) / 1000;
        System.out.println("StringBuilder \t Tiempo (en micro) para " + ITERATION + " cadenas "  + duration);
    }
}

Resultado (a)

operador +       Tiempo (en micro) para 100000 cadenas 123968461
concat ()        Tiempo (en micro) para 100000 cadenas 76363967
StringBuffer     Tiempo (en micro) para 100000 cadenas 25808
StringBuilder    Tiempo (en micro) para 100000 cadenas 1131

Resultado (b)

operador +       Tiempo (en micro) para 10000 cadenas 3631316
concat ()        Tiempo (en micro) para 10000 cadenas 1058666
StringBuffer     Tiempo (en micro) para 10000 cadenas 5719
StringBuilder    Tiempo (en micro) para 10000 cadenas 1851

Resultado (c)

operador +       Tiempo (en micro) para 1000 cadenas 41713
concat ()        Tiempo (en micro) para 1000 cadenas 33116
StringBuffer     Tiempo (en micro) para 1000 cadenas 1885
StringBuilder    Tiempo (en micro) para 1000 cadenas 1585

Resultado (d)

operador +       Tiempo (en micro) para 100 cadenas 20303
concat ()        Tiempo (en micro) para 100 cadenas 1417
StringBuffer     Tiempo (en micro) para 100 cadenas 2183
StringBuilder    Tiempo (en micro) para 100 cadenas 122

Resultado (e)

operador +       Tiempo (en micro) para 2 cadenas 448
concat ()        Tiempo (en micro) para 2 cadenas 35
StringBuffer     Tiempo (en micro) para 2 cadenas 29
StringBuilder    Tiempo (en micro) para 2 cadenas 18
14
  • 4
    ¿Y el tiempo de calentamiento para la JVM? Deberías obtener menos de 1 μs para 2 cadenas, sea el método que sea. Es una muy buena idea, pero sin ese tiempo de calentamiento, esto debería merecer -1. Commented el 25 abr. 2017 a las 17:50
  • 1
    Cedano, tu respuesta puede acabar siendo magnífica. La optimización temprana es la raiz de todos los males. Pero para cuando haya que optimizar tu respuesta sería super útil. Si tiene los benchmarks bien hechos. ¡Animo hombre, tu eres capaz! Seguro que cuando la revise mañana (o cuando puedas corregirla), te podré dar un +2. Commented el 25 abr. 2017 a las 18:42
  • 2
    Esta respuesta podría considerarse también como correcta, aporta más información e incluso demostración por medio de benchmarking sobre lo que pregunta el OP, los argumentos para calificarla me parecen algo estrictos, incluso las votaciones, en algunos caso veo que responder poco es mejor que esforzarse en una mejor respuesta! ಠ_ಠ
    – Jorgesys
    Commented el 25 abr. 2017 a las 18:43
  • 3
    Perdonen ustedes pero la pregunta no era sobre los benchmark. La intención era reforzar con un ejemplo lo afirmado en la mayoría de las respuestas indicando al mismo tiempo , con código, como funciona internamente cada método. Es curioso que los esfuerzos que se hacen por ayudar o aportar mejores soluciones son en cierto sentido descalificados por gente que no lo piensa dos veces antes de apretar el botoncito. Formular está respuesta toma su tiempo y apretar el botón menos de un segundo. Hay ahí una gran diferencia.
    – A. Cedano
    Commented el 25 abr. 2017 a las 21:44
  • 3
    @JoseAntonioDuraOlmos te recuerdo que el OP no preguntaba sobre cómo se hace un benchmark optimizado , sino sobre cuál es la forma más óptima de concatenar cadenas. Si consideras que eso es lo importante podrías tomarte el tiempo de hacer un ejemplo con un benchmark optimizado y publicarlo, entonces te cedería el paso borrando mi respuesta. Saludos.
    – A. Cedano
    Commented el 25 abr. 2017 a las 21:51
-2

Lo que te sugiero es que uses StringBuilder.

El problema con los Strings en java es que son inmutables y cada vez que trabajas con ellas crea un objeto. Con pocos procesos, el coste es irrelevante pero ante un gran proyecto, StringBuilder da mejor rendimiento y a su vez menor gasto de memoria.

(aunque el GarbajeCollector ya se ocupará de esto ultimo)

Ej:

StringBuilder stringBuilder = new StringBuilder();

stringBuilder.append(nombre);
stringBuilder.append(prefijo);

String all = stringBuilder.toString();
4
  • 1
    StringBuilder no da un mejor rendimiento en este caso particular. Commented el 26 abr. 2017 a las 22:37
  • 1
    @JoseAntonioDuraOlmos " Con pocos procesos, el coste es irrelevante pero ante un gran proyecto, StringBuilder da mejor rendimiento". No digo que en este caso de mejor rendimiento. Y no entiendo por que la respuesta de WebServeis tiene menos votos negativos (antes de la edicion era igual que la mia y tenia los mismos votos que tiene ahora)
    – lois6b
    Commented el 2 may. 2017 a las 7:31
  • 1
    StringBuilder da mejor rendimiento en bucles y peor en concatenación directa de dos cadenas (y probablemente tam bién de un número constante de cadenas concatenadas diréctamente). Lo cual no tiene nada que ver con tener muchos o pocos procesos o con el tamaño del proyecto. Lo de los votos da para un estudio sociológico :) Yo os he votado igual a ambos. Commented el 2 may. 2017 a las 7:42
  • 1
    @JoseAntonioDuraOlmos con procesos me refiero al que se está tratando, la concatenacion de cadenas. gracias por la aclaracion y tu voto (sea cual sea xD)
    – lois6b
    Commented el 2 may. 2017 a las 7:44

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.