POO + sentencia + algoritmo Duda

[sixtoex]

Bueno, antes que nada, gracias por la ayuda que me ofrece este sitio, estoy agradecido.

Ahora vamos al lío:

Ejercicio:

El problema planteado consiste en hallar la edad en años a partir de la fecha de nacimiento y la fecha actual.


Aquí mi código:

Subclase Fecha_Años:

Código Java:

package poo;
 
public class Fecha_Años {
    
int dia_Nacimiento;
int mes_Nacimiento;
int año_Nacimiento;
 
int dia_Actual;
int mes_Actual;
int año_Actual;
 
public Fecha_Años(int dia_Nacimiento,int mes_Nacimiento,int año_Nacimiento,int dia_Actual,int mes_Actual,int año_Actual){
this.dia_Nacimiento = dia_Nacimiento;
this.mes_Nacimiento = mes_Nacimiento;
this.año_Nacimiento = año_Nacimiento;
 
this.dia_Actual = dia_Actual;
this.mes_Actual = mes_Actual;
this.año_Actual = año_Actual;
        
    
}   
 
public int Calcular_Años(){
    
    int Edad_Años = 0;
    
    if(dia_Actual < dia_Nacimiento && mes_Actual <= mes_Nacimiento){
        Edad_Años = (año_Actual - año_Nacimiento)-1;
    }
    if(dia_Actual < dia_Nacimiento && mes_Actual > mes_Nacimiento){
        Edad_Años = (año_Actual - año_Nacimiento);
    }
    if(dia_Actual >= dia_Nacimiento && mes_Actual >= mes_Nacimiento ){
        Edad_Años = (año_Actual - año_Nacimiento);
    }
    if(dia_Actual >= dia_Nacimiento && mes_Actual < mes_Nacimiento ){
        Edad_Años = (año_Actual - año_Nacimiento)-1;
    }
   
    
    
    return Edad_Años;
}
    
    
 
}
 

Clase padre Configurar_FechaAño:

Código Java:

package poo;
 
public class Configurar_FechaAño {
 
    public static void main(String[] args) {
        // TODO Auto-generated method stub
        
Fecha_Años ConfigurarFecha = new Fecha_Años(2,3,1995,2,3,2015);
 
System.out.println("La edad en años es: "+ConfigurarFecha.Calcular_Años());
    }
 
}
 

Aquí va la pregunta:

¿Como ves el algoritmo que uso? es decir, están bien definido los nombres de las clases? uso bien el constructor? ¿tu lo harías de otra forma? ¿Si no uso getter y setters esta mal la clase? etc etc...

En definitiva si esta bien o mal y lo que este mal decirmelo

Gracias a todos los que contesten

[Profesor_Falken]

Buenas,

Los nombres de clases y variables no siguen el standard.

Antes de nada estudiate bien esto:
http://www.oracle.com/technetwork/ja...ons-150003.pdf


Sobre todo, no uses la ñ ni los acentos para los nombres de variables/metodos/clases.


Un saludo

[sixtoex]

Iniciado por Profesor_Falken

Buenas,

Los nombres de clases y variables no siguen el standard.

Antes de nada estudiate bien esto:
[url]http://www.oracle.com/technetwork/java/codeconventions-150003.pdf[/url]


Sobre todo, no uses la ñ ni los acentos para los nombres de variables/metodos/clases.


Un saludo

y el constructor y el método? eso si esta bien estructurado?

[Xerelo]

Iniciado por sixtoex

y el constructor y el método? eso si esta bien estructurado?

Tú decides qué parámetros necesitas y el formato en que lo pasas, un constructor que sólo asigna valores difícilmente puede estar mal.

Como pega, es un poco absurdo tener unas variable llamadas x_actual, si implica que es el día de hoy no te haría falta pedirla, y si puede ser cualquier fecha, el nombre que le has puesto produce confusión.

Aparte de eso podías haber hecho una única clase, Configurar_FechaAño sólo aporta que tiene el main, y eso podías haberlo hecho en Fecha_Años.

Deberías buscarte ejemplos donde se vea más claro qué es un objeto/clase y què es una acción/método.

Tu método, pues código innecesario y repetitivo. Y debes tener cuidado con los if, si son condiciones excluyentes usa else if.

i

Código Java:

nt Edad_Años  = (año_Actual - año_Nacimiento);
        
        if(mes_Actual < mes_Nacimiento){
            Edad_Años = Edad_Años -1
        }
        else if(mes_Actual == mes_Nacimiento && dia_Actual < dia_Nacimiento){
            Edad_Años = Edad_Años -1
        }
 

Hace exactamente lo mismo (he simplificado el tuyo pero no lo he probado).

[sixtoex]

Iniciado por Xerelo

Tú decides qué parámetros necesitas y el formato en que lo pasas, un constructor que sólo asigna valores difícilmente puede estar mal.

Como pega, es un poco absurdo tener unas variable llamadas x_actual, si implica que es el día de hoy no te haría falta pedirla, y si puede ser cualquier fecha, el nombre que le has puesto produce confusión.

Aparte de eso podías haber hecho una única clase, Configurar_FechaAño sólo aporta que tiene el main, y eso podías haberlo hecho en Fecha_Años.

Deberías buscarte ejemplos donde se vea más claro qué es un objeto/clase y què es una acción/método.

Tu método, pues código innecesario y repetitivo. Y debes tener cuidado con los if, si son condiciones excluyentes usa else if.

i

Código Java:

nt Edad_Años  = (año_Actual - año_Nacimiento);
        
        if(mes_Actual < mes_Nacimiento){
            Edad_Años = Edad_Años -1
        }
        else if(mes_Actual == mes_Nacimiento && dia_Actual < dia_Nacimiento){
            Edad_Años = Edad_Años -1
        }
 

Hace exactamente lo mismo (he simplificado el tuyo pero no lo he probado).

Lo de la unica clase no lo hice por que me dijeron que no es etico hacerlo así, que no se suele hacer y es mala practica y en cuanto a no pedir fecha actual

¿entonces como saco la fecha? con clases predefinidas no? no hay otra (creo)

[Xerelo]

Lo de la unica clase no lo hice por que me dijeron que no es etico hacerlo así, que no se suele hacer y es mala practica

No sé de dónde has sacado eso, pero seguramente no lo has entendido bien. Una cosa es que las partes de un proyecto se dividan en clases y que haya una que se encargue de empezar la ejecución y de ir llamando al resto, y otra cosa es que para hacer un ejercicio donde no hay nada más que un método hagas otra clase sólo para llamarla.

Tu ejemplo no es que no necesite usar dos clases, es que hasta sobra el constructor. El hecho de que hagas un new no significa que estés programando con orientación a objetos, igual que calentar leche en el microondas no es cocinar.

Lo de que no es ético...

¿entonces como saco la fecha? con clases predefinidas no?

Evidentemente, pero también puedes cambiar de nombre a las variables para que tenga más coherencia.