Java. Inicializar wrappers o envoltorio Operadores aritméticos CU00633B

[josephsa]

Hola, he creado este código para resolver el ejercicio de la entrega CU00633B del tutorial de programador java. Creo que hace lo que pide, si hay forma de mejorarlo por favor diganme 

Código: [Seleccionar]

public class Medico{
private String nombre;
private String apellidos;
private int edad;
private boolean casado;
private String ci;
private String especialidad;
private int multiploedad, mod5;

//Constructor
public Medico(){
    nombre = "";
    apellidos = "";
    edad = 19;
    casado = true;
    ci = "";
    especialidad = "";
    multiploedad = 0;
}//cierre del constructor
   
//Metodo para establecer el nombre del doctor
public void setNombre (String valorNombre){
    nombre = valorNombre;//el nombre cambia por el indicado
}

//Metodo para establecer el apellido
public void setApellido (String valorApellido){
    apellidos = valorApellido;//el apellido cambia
}

//metodo para establecer la edad
public void setEdad (int valorEdad){
    edad = valorEdad;//el apellido cambia
}
 
//metodo para establecer el estado civil
public void setCasado (boolean valorCasado){
    casado = valorCasado;//el estado cambia
}

//metodo para establecer la ci
public void setCi (String valorCi){
    ci = valorCi;// el valor cambia
}

//metodo para la espacialidad
public void setEspecialidad (String valorEspecialidad){
    especialidad = valorEspecialidad;//valor cambia
}

//metodo para deviolver el nombre
public String getNombre () {return nombre; }//cierre del metodo

//metodo para deviolver el apellido
public String getApellido () {return apellidos; }//cierre del metodo

//metodo para devolver la edad
public int getEdad () { return edad; } //cierre del metodo

//metodo para devolver el estado civil
public boolean getCasado () {return casado; }//cierre del metodo

//metodo para devolver ci
public String getCi () {return ci; }//cierre del metodo

//metodo para devolver la especialidad
public String getEspecialidad () {return especialidad; }//cierre del metodo

//metodo para calculo de multiplo edad
public int getcalculoParaMultiploEdad () {
mod5= edad%5;
multiploedad= 5-mod5;
return multiploedad;
}

//para imprimir por consola
public void mostrar() {
System.out.println("El nombre del doctor es " + getNombre () + " " + getApellido () + " La especialidad del del Doctor es " + getEspecialidad () + " y su edad sera multiplo de 5 en " + getcalculoParaMultiploEdad () + " años");
}

}//cierre de la clase

[Agente_007]

Hola josephsa, llevo poco estudiando y voy por la entrega 40 casi igual que tu... Te ayudaré con lo q he aprendido...

Tienes varios errores y los voy a enumerar y aclarar:

1) El ejercicio nos pide 6 atributos o campos, no entiendo porque pones: private int multiploedad, mod5; , No nos los pide el ejercicio así q no le busquemos las 5 patas al gato.
Está mal colocarlo pero como vas por la entrega 33 está bien.
No es necesario declarar este tipo de variables ya que son "variables locales" q veras más adelante.

2) (Constructor)....Hay q recordar que para iniciar ciertos tipos wrappers o envoltorio (Tipo de Dato) sería de la siguiente manera:
Si es tipo String = "";
Si es tipo int = 0; Pusistes Edad =19.
Si es tipo boolean = false; Iniciaste el atributo con true.

3) Te dicen: Define un método de nombre “calculoParaMultiploEdad” que no recibe parámetros y es tipo procedimiento...

Como bien te dicen... Tipo procedimiento... Y lo que hiciste fue un método tipo función usando get junto con su return.

Los tipos procedimientos son los q van acompañados de (void) y pueden ir acompañados de (set) si es para modificar y sino solo llevan, indicador+void+nombreDelMétodo+Parámetro si lo piden, en este caso no piden parámetros.

4) Después te dicen:  A continuación deberá mostrar un mensaje por consola del tipo: “El médico de nombre … con especialidad … tendrá una edad múltiplo de 5 dentro de … años”.

Tienes un error de tipeo, pusistes "del" dos veces... Otra cosa, con solo poner el nombre del atributo en minúscula como lo pusiste en el campo tienes, en vez de poner getApellido()+.... luego getEspecialidad ()...
Simplemente colocando ...+ apellidos +.....+ especialidad +... y así sucesivamente con cada atributo sería más fácil, no crees. Lo q no sé si de esa forma en q lo pusiste, este bien en cuanto a la forma de programar como tal.

Acomódalo y vemos q tal.....

[josephsa]

hola disculpa por no haber respondido antes, lo había olvidado

Código: [Seleccionar]

public class Medico{
private String nombre;
private String apellidos;
private int edad;
private boolean casado;
private String ci;
private String especialidad;

//Constructor
public Medico(String valorNombre, String valorApellido, Integer valorEdad, boolean valorCasado, String valorCi, String valorEspecialidad){
    nombre = valorNombre;
    apellidos = valorApellido;
    edad = valorEdad;
    casado = valorCasado;
    ci = valorCi;
    especialidad = valorEspecialidad;
}//cierre del constructor
   
//Metodo para establecer el nombre del doctor
public void setNombre (String valorNombre){
    nombre = valorNombre;//el nombre cambia por el indicado
}

//Metodo para establecer el apellido
public void setApellido (String valorApellido){
    apellidos = valorApellido;//el apellido cambia
}

//metodo para establecer la edad
public void setEdad (int valorEdad){
    edad = valorEdad;//el apellido cambia
}
 
//metodo para establecer el estado civil
public void setCasado (boolean valorCasado){
    casado = valorCasado;//el estado cambia
}

//metodo para establecer la ci
public void setCi (String valorCi){
    ci = valorCi;// el valor cambia
}

//metodo para la espacialidad
public void setEspecialidad (String valorEspecialidad){
    especialidad = valorEspecialidad;//valor cambia
}

//metodo para deviolver el nombre
public String getNombre () {return nombre; }//cierre del metodo

//metodo para deviolver el apellido
public String getApellido () {return apellidos; }//cierre del metodo

//metodo para devolver la edad
public int getEdad () { return edad; } //cierre del metodo

//metodo para devolver el estado civil
public boolean getCasado () {return casado; }//cierre del metodo

//metodo para devolver ci
public String getCi () {return ci; }//cierre del metodo

//metodo para devolver la especialidad
public String getEspecialidad () {return especialidad; }//cierre del metodo

//metodo tipo procedimiento para calcular si es multiplo de la edad
public int calculoParaMultiploEdad () {return 5-(edad%5);}

//para imprimir por consola
public void mostrar() {
System.out.println("El nombre del doctor es " + nombre + " " + apellidos + " La especialidad del Doctor es " + especialidad + " y su edad sera multiplo de 5 dentro de " + calculoParaMultiploEdad() + " años");
}//cierre del mostrar
}//cierre de la clase


[Agente_007]

Ahora si está correcto el ejercicio, no veo ningún tipo de error excepto por el constructor, q lo hiciste con parámetros, no está mal pero el ejercicio no te pide eso....

El Ejercicio te dice:

Define un constructor asignando unos valores de defecto a los atributos...

Cuales son esos valores de defecto?... Como lo hiciste antes pero acomodando la forma en q los iniciastes...
 

2) (Constructor)....Hay q recordar que para iniciar ciertos tipos wrappers o envoltorio (Tipo de Dato) sería de la siguiente manera:
Si es tipo String = "";
Si es tipo int = 0; Pusistes Edad =19.
Si es tipo boolean = false; Iniciaste el atributo con true.

Cya

[josephsa]

gracias, cierto debí corregir como me dijiste, pero lo puse así por comodidad, lo probé de ambas maneras y funciono.