search

EstebanRodriguezUBA / miProyectoWeb

0 stars 0 forks source link

La clase rango de fechas deberia ser inmutable (o al menos comunicarlo) #7

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/RangoFecha.groovy

maurociancio commented 6 years ago

Por ser un value object

EstebanRodriguezUBA commented 6 years ago

Buenos días profesor: No estoy seguro de comprender como comunicarlo.Sé que no se edita, no hay setters. En principio investigué y la manera de definirlo inmutable en la implementación en Java es con "blank Final". Asi que usé esto. Espero su devolución si es correcto.

maurociancio commented 6 years ago

Deberia tener un constructor que reciba los parámetros. Y en ninguna parte del codigo se deberia modificar sus atributos, siempre que se necesite hacer un cambio se construye un nuevo objeto. Y probablemente tenga métodos. No solo atributos.