Ejercicio resuelto CU00637B ejemplo condicional de selección switch en Java

[matru]

Código: [Seleccionar]

  public class Motor{
    private int tipoBomba;
    private String tipoFluido;
    private String combustible;

    public Motor(){
        tipoBomba=0;
        tipoFluido="";
        combustible="";

    }
   
    //metodos setter
    public void setTipoBomba(int valorTipoBomba){
    tipoBomba=valorTipoBomba;}
   
    public void setTipoFluido(String valorTipoFluido){
    tipoFluido=valorTipoFluido;}
   
    public void setCombustible(String valorCombustible){
    combustible=valorCombustible;}
   
    //metodos getter
    public int getTipoBomba(){return tipoBomba;}
   
    public String getTipoFluido(){return tipoFluido;}
   
    public String getCombustible(){return combustible;}
   
    //otros metodos
   
    public void dimeTipoMotor(){
        switch(tipoBomba){
            case 0: System.out.println("No hay establecido un valor definido para el tipo de bomba");break;
            case 1: System.out.println("la bomba es una bomba de agua");break;
            case 2: System.out.println("la bomba es una bomba de gasolina");break;
            case 3: System.out.println("la bomba es una bomba de hormigon");break;
            case 4: System.out.println("la bomba es una bomba de pasta almenticia");break;
            case 5: System.out.println("No existe un valor valido para tipo de bomba");break;
       
         
        }
   
   
   
    }
   
   
   
   
   
   
   





}//cierre de la clase

[Jorge lopez]

 Hola matru!

Tu método dimeTipoMotor presenta una deficiencia, un buen programador siempre piensa en como lograr que su código o programa responda a situaciones inesperadas. Que pasaría si yo asigno como valor de variable el valor 6 a la variable tipoMotor, y luego invoco el método dimeTipoMotor? Se imprime algo por pantalla? No es mas lógico que el programa imprima un mensaje informando al usuario que algo no anda bien?

La eficiencia de un programa es algo muy importante que debemos de tener en cuenta a la hora de programar.

No es que tu ejercicio este mal, solo que soy un poquito exigente  . Para mejorar tu código basta con usar la palabra clave default dentro del condicional switch
para que muestre un mensaje en caso de que no se cumplan ninguno de los anteriores casos, (case 0: ... case 1: ... ext) así por ejemplo, si el valor del tipo de bomba es es un numero diferente del 0 al 4 (que son los casos que as declarado) pues se mostrara el mensaje correspondiente al default.

Seria algo mas o menos así:

Código: [Seleccionar]

default: System.out.println(tipoBomba + " no es un valor valido para un tipo de bomba, por favor indique un valor del 1 al 4");break;en lugar de:

Código: [Seleccionar]

case 5: System.out.println("No existe un valor valido para tipo de bomba");break;