Ejercicio CU00633B curso java para principiantes ejemplo clase y métodos

[Ronald]

Código: [Seleccionar]

public class Medico2{
    public String nombre;
    public String apellidos;
    public int edad;
    public boolean casado;
    public String numeroDocumentoIdentidad;
    public String especialidad;
    public int multiploEdad;
    public Medico2(){
        nombre="Desconocido";
        apellidos="Desconocido";
        edad=0;
        casado=false;
        numeroDocumentoIdentidad="Desconocido";
        especialidad="Desconocido";
        multiploEdad=0;
    }//cierre del constructor

    /*Métodos*/

    public void setNombre(String valorNombre){
        nombre=valorNombre;
    }

    public void setApellidos(String valorApellidos){
        apellidos=valorApellidos;
    }

    public void setEdad(int valorEdad){
        edad=valorEdad;
    }

    public void setCasado(boolean valorCasado){
        casado=valorCasado;
    }

    public void setNumeroDocumentoIdentidad(String valorNumeroDocumentoIdentidad){
        numeroDocumentoIdentidad=valorNumeroDocumentoIdentidad;
    }

    public void setEspecialidad(String valorEspecialidad){
        especialidad=valorEspecialidad;
    }

/*Método para obtener el múltiplo de la edad*/

    public void calculoParaMultiploEdad(){
        int multiplode5=edad%5;
        multiploEdad=5-multiplode5;

        System.out.println("El médico "+nombre+", con especialidad "+especialidad+", tendrá una edad múltiplo de 5 dentro de "+multiploEdad+"años");
    }
    /*Fin de los métodos Setters*/

    /*Inicio de los métodos Getters*/

    public String getNombre(){return nombre;}

    public String getApellidos(){return apellidos;}

    public int getEdad(){return edad;}

    public boolean getCasado (){return casado;}

    public String getNumeroDocumentoIdentidad(){return numeroDocumentoIdentidad ;}

    public String getEspecialidad(){return especialidad;}

}

[toni_apr]

Hola Ronald

Tu clase está bastante bien.
Con todos los setters y getters y el método calculoParaMultiploEdad()

Quiero darte unas notas con el propósito de mejorar el código.

1º
Los atributos de una clase (siempre) se declaran private. Razón: los atributos de una clase solo debe manipularlos la propia clase.

2º
El int multiploEdad no es un atributo. Eso ya lo sabías pues no le creaste getter ni setter.
Esta variable solo la usará el método calculoParaMultiploEdad(). Así pues créala dentro de ese método, igual que la variable multiplode5.

3º
Asignar los valores por defecto a los atributos significa que los String se inicializan con cadena vacía, y tal como tu has hecho los int a cero y los boolean a false.
Ejem. nombre = "";

4º
Por convención, (no por obligación) los métodos y los atributos bautízalos usando el estilo lowerCamelCase.
Este estilo consiste en poner en mayúcula la primera letra de cada palabra excepto la primera.
Ejem. multiploDe5, valorDeLaVariable, numeroDeVecinos, etc.

Con esto la clase estará más que perfecta.

Saludos y adelante.

[Ronald]

Gracias por los consejos

En el 3º punto, creí que estaba bien asignarle "Desconocido" por defecto, gracias por la observación

En el 4º, olvidé, en la variable local multiplode5, escribir multiploDe5

En serio te lo agradezco