Variables locales a un método en java (CU00638B)

[aleexx1724]

Saludos

Código: [Seleccionar]

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

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

    }

    //getters  metodos procedimiento
    public void setFluido(String valorFluido){
        tipoFluido=valorFluido;
    } 

    public void setCombustible(String valorCombustible){
        tipoCombustible=valorCombustible;
    }   

    public void  setBomba(int valorBomba){
        tipoBomba= valorBomba;

    }
    // getters  metodos funcion

    public String getFluido(){
        return tipoFluido;
    }

    public String getCombustible(){
        return tipoCombustible;
    }   

    public int getBomba(){
        return tipoBomba;
    }   

    public boolean dimeSiMotorEsParaAgua(int tipoDeMotor ){
       boolean  motorEsParaAgua=false;
   
            if(tipoDeMotor==1){
            motorEsParaAgua=true;
            }
            else{
            motorEsParaAgua=false;
            }
   
       return motorEsParaAgua;
    }
}

[toni_apr]

Hola aleexx1724

Tu código está perfecto, solo quiero comentarte un detalle.
Cito del enunciado del ejercicio

Crea un método tipo función que devuelva un booleano (true o false) denominado dimeSiMotorEsParaAgua() donde se cree una variable local booleana motorEsParaAgua de forma que si el tipo de motor tiene valor 1 tomará valor true y si no lo es tomará valor false. El método debe devolver la la variable local booleana motorEsParaAgua.

En este método

dimeSiMotorEsParaAgua(int tipoDeMotor )

Primero
 le das un parámetro, tipoDeMotor, eso quiere decir que cuando lo llames tendrás que darle un tipo de motor, un número, puedes darle el que le diste con tipoBomba o darle un número distinto.
Lo correcto, es usar la signatura sin parámetro

dimeSiMotorEsParaAgua()

Y dentro del método tener en cuenta el valor de tipoBomba para tomar decisiones

Segundo
Inicializias motorEsParaAgua como false. Bien
Luego en el if si has de cambiar el valor, lo haces a true.
Pero, aquí está el pero, no es necesario ponerlo a false, si nadie lo cambió a true.
Resumiendo el else del if puede eliminarse y haces el método más compacto y eficiente.

Saludos

[aleexx1724]

Entonces si lo hago sin parametros. Tengo que
Crear un entero dentro del metodo ?

[toni_apr]

Lo que quiero decir es que el método

public boolean dimeSiMotorEsParaAgua()

en lugar de trabajar con el parámetro int tipoDeMotor, ha de trabajar con el atributo de la clase 'tipoBomba' que es el integer que contiene el número del tipo de bomba o también llamado aquí tipo de motor
Es decir algo como esto

public boolean dimeSiMotorEsParaAgua( ){
       boolean  motorEsParaAgua=false;
   
            if(tipoBomba==1){
            motorEsParaAgua=true;
            }

       return motorEsParaAgua;
    }

Al atributo tipoBomba, ya le has dado un valor en algún momento. Este método solo consulta el valor de tipoBomba para tomar sus decisiones.

Lo de crear un entero dentro del método, no se que quiere decir

Si hay algo que no entiendas, dilo

Saludos

[aleexx1724]

ahy fue donde me confundi porque el ejercicio dice
 si el tipo de motor tiene valor 1 tomará valor true y si no lo es tomará
valor false. 
y nosabia como devolver elvalor 1 o 2 poreso cree el metodo asi
pero gracias por la explicacion y la aclaracion
toni_apr

saludos