Revision Ejercicio CU00633 uso operador mod java

[Zikem10]

buen dia solo anexe el metodo de procedimiento al anterior programa de Medico pero a este nuevo lo renombre como class MedicoMulEdad.
Gracias

Código: [Seleccionar]

//Programa 2 Medico este programa nos dara los atributos de Nombre, Apellidos, edad, casado, ife, espeialidad
//Se incorpora calculoParaMultiploEdad
//Programa Hecho por Osvaldo Maciel Julio 15 2014


public class MedicoMulEdad{
   
    String nombre, apellido1, apellido2, especialidad, ife;
    //String nombapell=apellido1+apellido2;
    int edad;
    boolean casado;
    int anosfaltantes, multiplo5=5;[quote][/quote]
   
    //Definimos el constructor cuando se cree los objetos medico
   
    public MedicoMulEdad(){
        nombre= "Ninguno";
        apellido1= "Ninguno";
        apellido2= "Ninguno";
        edad= 0;
        casado= true;
        ife= "Ninguno";
        especialidad= "Ninguno";
       
    }// Cierre constructor
   
    // Metodo para establecer nombre
    public void setNombre(String ValorNombre){
        System.out.println("Ha cambiado el Nombre del Medico:   "+ nombre + "El nuevo Nombre del Medico es:   "+ ValorNombre);
   nombre=ValorNombre;//Maniene guardado los datos en las casillas del objeto, mientras este este abierto
    }
   
    //Metodo para establecer Apellidos
    public void setApellidos(String ValorApellido1, String ValorApellido2){
        System.out.println("Ha cambiado los Apellidos del Medico:  " +apellido1 +apellido2 +"Los nuevos apellidos son:   " +ValorApellido1+ValorApellido2);
    apellido1=ValorApellido1;
    apellido2=ValorApellido2;
    //Maniene guardado los datos en las casillas del objeto, mientras este este abierto
    }
   
    //Metodo para establecer la edad del Medico
    public void setEdad(int ValorEdad){
        System.out.println("Ha cambiado la edad del Medico:   "+ nombre +apellido1+apellido2+edad +"La Nueva edad es:   "+ValorEdad);
        edad=ValorEdad;//Maniene guardado los datos en las casillas del objeto, mientras este este abierto
    }
   
    //Metodo para establecer el estado civil del Medico
    public void setCasado(boolean ValorCasado){
        if (casado=ValorCasado)
        {
            System.out.println("El estado Civil del Medico:   "+ nombre +apellido1+apellido2+casado);//+casado +"El nuevo estado Civil es:   "+ValorCasado);
        }
        else {
        System.out.println("Ha cambiado el estado Civil del Medico:   "+ nombre +apellido1+apellido2  +"El nuevo estado Civil es:   "+ValorCasado);
    }
    casado=ValorCasado;//Maniene guardado los datos en las casillas del objeto, mientras este este abierto
    }
   
    //Metodo para establecer los datos de la credencial del IFE
    public void setIfe(String ValorIfe){
        System.out.println("Han cambiado los datod de la credencial del IFE:   "+ nombre +apellido1+apellido2+ife +"Los nuevos datos de la credencial del IFE son:   "+ ValorIfe);
    ife=ValorIfe;//Maniene guardado los datos en las casillas del objeto, mientras este este abierto
    }
   
    //Metodo para establecer la Especialidad del Medico
    public void setespecialidad(String ValorEspecialidad){
        System.out.println("Ha cambiado la Especialidad del Medico:   "+ nombre +apellido1+apellido2+especialidad +"Lanueva especialidad es:   "+ValorEspecialidad);
    especialidad=ValorEspecialidad;//Maniene guardado los datos en las casillas del objeto, mientras este este abierto
}



/*Metodo para establecer el Nombre y Apellidos
public void getsetNombApell(String ValorNombApell){
    nombapell=ValorNombApell;
}*/

   
 
  //Metodo para obtener los Atributos
  public String getNombre(){return nombre;}
  public String getApellido1(){return apellido1;}
  public String getApellido2(){return apellido2;}
  public int getEdad(){return edad;}
  public boolean getCasado(){return casado;}
  public String getIfe(){return ife;}
  public String getEspecialidad(){return especialidad;}
 // public String getNombApell(){return nombapell;}
 
  //Metodo para obtener la edad que sea Multiplo de 5
  public void setCalculoParaMultiploEdad()
 
      {
   
    anosfaltantes=multiplo5-(edad%multiplo5);
    System.out.println("El Medico de Nombre:"+nombre+apellido1+apellido2+"Especialidad:"+especialidad+"Tendra una edad Multiplo de 5 dentro de :"+anosfaltantes +"anos");
 
    }
 
 
}


[César Krall]

Hola, el código es correcto aunque tendrías que hacer algunas mejoras.

Hay un aspecto de diseño que no es correcto. Estableces un atributo multiplo5=5; y lo inicializas al mismo tiempo que lo declaras.

En general un atributo se debe inicializar en el constructor, no al mismo tiempo que lo declaras.

En este caso, el atributo podría eliminarse o renombrarse. Eliminarse ya que puede sustituirse por 5, no hay necesidad de usarlo. Si lo usamos previendo que en el futuro en vez de 5 pueda ser otro número, habría que renombrarlo y dejar el nombre como multiplo en lugar de como multiplo5.

Por otro lado cuando se establece la edad del médico con set aparece un mensaje como:

Ha cambiado la edad del Medico:   NingunoNingunoNinguno0La Nueva edad es:   72


La presentación no queda bien (por ejemplo faltan espacios). Por otro lado tener en cuenta que los métodos set y get no deben incluir mensajes por consola (a no ser que el ejercicio específicamente los pida)

El nombre de método setCalculoParaMultiploEdad no es correcto. set es un prefijo que se usa para establecer valor de atributos. En este caso el método no es para establecer valor de un atributo, sino para hacer un cálculo, por tanto el nombre correcto sería calcularCuantoFaltaParaMultiploEdad o similar.

Saludos.