search

EstebanRodriguezUBA / miProyectoWeb

0 stars 0 forks source link

La clase fecha no aporta nada, usar directamente date o LocalDate (nueva api de java) #6

Open maurociancio opened 6 years ago

maurociancio commented 6 years ago

https://github.com/EstebanRodriguezUBA/miProyectoWeb/blob/master/gitHubrepositorio/proyecto/src/main/groovy/fiuba/sii7571/miproyecto/util/fecha/Fecha.groovy

EstebanRodriguezUBA commented 6 years ago

Buenos días profesor: Entendido. Lo hice con el propósito de encapsular, así sea Date u otra cosa. Relacionado con otra pregunta sobre la clase RangoFecha que lo hice heredar de clase Fecha. Si este cambio es incorrecto,innecesario, vuelta atrás.

maurociancio commented 6 years ago

No está bien. Un rango de fechas no es una fecha. Esa herencia está mal. Utilizá composición. Un rango de fechas tiene dos fechas. Y no es necesario crear una clase fecha: cual es el criterio entonces con Integer, Long, String, creamos otras clases para encapsularlos? Es demasiado.