practica CU00650B Combinar cadenas en Java con método substring

[resakoso]

En principio funciona bien, no sé si se puede mejorar algo. Gracias

Código: [Seleccionar]

public class CadenasCombinadas {

    public CadenasCombinadas(){
        //nada
    }
    public String combina (String s1,String s2,String s3){
        if ((s1.length()<3)||(s2.length()<3)||(s3.length()<3))
            return "No aporta cadenas válidas";
        else
            return s1+s2+s3;
         
    }
    public String combinadoSiempre(String s1,String s2,String s3){
         int long1;
         int long2;
         int long3;
        if ((s1.length()<=3)||(s2.length()<=3)||(s3.length()<=3))
            return s1.substring(0)+s2.substring(0)+s3.substring(0);   
        else{
            long1=s1.length();
            long2=s2.length();
            long3=s3.length();
            return s1.substring(0,long1)+s2.substring(0,long2)+s3.substring(0,long3);
                   
        }
    }
    public static void main(String[] args) {
       
        CadenasCombinadas c=new CadenasCombinadas();
        System.out.println(c.combina("p33r","se","ter"));
        System.out.println(c.combina("primera","segunda","tercera"));
        System.out.println(c.combinadoSiempre("33 r","   ","i"));
    }
   
}

[César Krall]

Hola! Dado que long es un tipo de variable creo que sería preferible no usarlo porque puede confundir.

int long1;  --- > int longitud1;


Por otro lado, el ejercicio no veo que se ajuste a lo que pide el enunciado de CU00650B. El enunciado te pide que uses la clase Combinador que figura en el contenido del curso, pero tú has hecho algo parecido pero no exactamente eso. Como no sabemos exactamente qué es lo que querías hacer, es difícil decir si está bien o mal.

¿Es posible que esté confundido el código CU00650 y se trate de otro ejercicio?


Nota: hay una cosa que no entiendo. Si el código del método CombinadoSiempre lo sustituyo por este código:

    public String combinadoSiempre(String s1,String s2,String s3){
        return s1.substring(0)+s2.substring(0)+s3.substring(0);
       
    }

Parece que obtengo los mismos resultados y si eso fuera así habría un montón de líneas que no sirven para nada. ¿Cuál es la lógica de este método?


Saludos