Entrega ejercicio CU00649B extraer letras con substring java

[jogonzalezmor]

He aquí mi código.

Código: [Seleccionar]

public class ExtraeLetras
{
    // instance variables - replace the example below with your own

    /**
     * Constructor de objects de la clase ExtraeLetras
     */
    public ExtraeLetras()
    {
        // initialise instance variables
    }//Fin del constructor

    public String combinaCadenas(String valorCadena1, String valorCadena2, String valorCadena3)
    {
        // put your code here
        String cadena1 = valorCadena1;
        String cadena2 = valorCadena2;
        String cadena3 = valorCadena3;
        String combinacion = valorCadena1.substring(0, 2)+ " " + valorCadena2.substring(0, 4) + " - " + valorCadena3.substring(0, 6);;
        return combinacion;
    }//Fin del método combinaCadenas
}//Fin de la clase


[Ogramar]

Está bien aunque hay una cosa que no es correcta.

Cuando vi valorCadena3.substring(0, 6);; pensé que iba a dar un error por los dobles punto y coma, pero no da error  así que eso lo puedes tener en cuenta pero da igual

Lo que no está bien es:

        String cadena1 = valorCadena1;
        String cadena2 = valorCadena2;
        String cadena3 = valorCadena3;

¿Para qué declaras estas variables si luego no las usas? Eso sería lo que se llama "código inútil" y hay que tratar de que en nuestras clases y programas no quede ese tipo de código porque al final complica la corrección de los programas y puede dar lugar a problemas a la larga.

Salud2s