practica CU00650B (modificada) comprobar longitud cadenas java

[BelCrow]

En esta practica solo se pide que copiemos el código y creemos objetos y realicemos pruebas.. cito ejercicio:

Crea un objeto de tipo Combinador y prueba a ejecutar los dos métodos disponibles pasando distintos parámetros: cadenas con más de tres caracteres, una cadena con más de tres y otra con menos de tres y ambas con menos de tres caracteres. Fíjate en el resultado que ofrece el método CombinadoSiempre cuando se le pasan como parámetros dos cadenas vacías. ¿Te parece lógico el resultado?

En la clase tenemos definidos dos atributos o campos. ¿Tienen utilidad conocida a la vista del código de la clase? ¿Se usan en los métodos que tenemos definidos?

Pero yo decidí rehacer el ejercicio CU00649B con lo que aprendí en este.... Favor revisar para ver si procedimiento es correcto o se puede mejorar .

InicialesDePalabras2:

Código: [Seleccionar]

//clase
public class InicialesDePalabras2 {
    String primeraPalabra;
    String segundaPalabra;
    String terceraPalabra;

    //constructor
    public InicialesDePalabras2 (String valorPrimeraPalabra, String valorSegundaPalabra, String valorTerceraPalabra) {
        primeraPalabra = valorPrimeraPalabra;
        segundaPalabra = valorSegundaPalabra;
        terceraPalabra = valorTerceraPalabra;
    }

    //metodo con length que solo suma y devuelve mensaje
    public String iniciales1 () {
        if (primeraPalabra.length() >= 2 && segundaPalabra.length() >= 4 && terceraPalabra.length() >= 6) {
            String union = primeraPalabra.substring(0,2)+" "+segundaPalabra.substring(0,4)+"-"+terceraPalabra.substring(0,6);
            return union;
        }else {
            return "Las palabras no constan con letras necesarias para ser sumadas.";
        }

    }

    //metodo con length que te suma todas las que esten
    public String iniciales2 () {
        int palabra1 = 2;
        int palabra2 = 4;
        int palabra3 = 6;

        if (primeraPalabra.length() < 2) {
            palabra1 = primeraPalabra.length();
        }

        if (segundaPalabra.length() < 4) {
            palabra2 = segundaPalabra.length();
        }

        if (terceraPalabra.length() < 6) {
            palabra3 = terceraPalabra.length();
        }

        String union = primeraPalabra.substring(0,palabra1)+" "+segundaPalabra.substring(0,palabra2)+"-"+terceraPalabra.substring(0,palabra3);
        return union;
    }
}


[Mario R. Rancel]

Hola, el código está bien planteado. Sobre el nombre de la clase, lo ya comentado. Sería más adecuado CombinadorDeSubcadenas o algo así en vez de InicialesDePalabras.

Más cosas para que reflexiones sobre ellas:

- Una clase no siempre tiene que tener atributos. En particular si su misión es procesar información que le llega desde fuera es posible que no debamos declarar atributos en la clase. Una clase normalmente tiene atributos, pero puede no tenerlos. Esta clase posiblemente hubiera quedado mejor sin atributos (aunque dependería del diseño global dentro del programa más amplio donde estuviera).

- ¿Estás haciendo un buen diseño de nombres de variables? Por ejemplo escribes palabra1 = 2; ¿Crees que esto describe bien la variable? Una palabra es una cadena de texto, pero tú estás diciendo que es un número. ¿No hubiera sido mejor usar longitudMinimaPalabra1 = 2; ?

Trata siempre de crear un código entendible y eficiente, además de que funcione. Si lo vas practicando con pequeños programas lo irás aprendiendo para cuando tengas que enfrentarte a grandes programas. En estos casos no tiene demasiada importancia, pero para ir mejorando hay que intentarlo...

Saludos,

[BelCrow]

heheh si tu supieras que lo pense.. pero opte por escribir el nombre de la variable mas corto para hacerlo mas entendible, al parecer hice mal.... corregire esto..

ahhh con respecto a loa atributos de la clase.. decidi ponerlos porque queria usar los dos metodos con loa mismos atributos para comparar y no tener que introducirlos de nuevo... aunque si te fijas en el anterior no tenia dichos atributos.. osea uqe los puse intencional...

[BelCrow]

gracias de todas formas... mejorare eso mario

[Mario R. Rancel]

Sí, ahora veo que lo hicistes con intención, perdona porque no me había dado cuenta 

[BelCrow]

haha ok no hay problemas , como quiera esos consejos no están de mas .....