Consulta duplicidad de hechos

[nicolasrey9]

Buena queria consultar si estaria bien redefinir equals y hashCode en el Hecho.

Esto con el objetivo de tener la logica de cuando esta repetido un hecho concentrada en el mismo hecho, ya que esta logica seria utilizada a la hora de ver si un hecho esta duplicado en la fuente de datos estatica y tambien cuando se quiere averiguar si un hecho fue eliminado, segun nuestra implementacion.

[flbulgarelli]

Hola! Pudiste ver las aclaraciones al respecto en el sitio de la materia, sección sobre los trabajos prácticos?

[flbulgarelli]

Tené en cuenta que el criterio de equivalencia va a ser parametrizable más adelante.

[nicolasrey9]

si, ya habia leido eso, pero me parecia una buena opcion para que este concentrado en el hecho. Pero teniendo en cuenta que va poder ser parametrizable cambia mi punto de vista.

Apuntando a que sean parametrizables una opcion seria que se inyecten en la fuente de datos estatica y en caso de mi grupo en la estructura que relaciona los hechos con sus solicitudes?

[flbulgarelli]

Pero teniendo en cuenta que va poder ser parametrizable cambia mi punto de vista.

Si, coincido en que éste es un motivo de peso, que hace impracticable la solución basada en equals.

Aún así, igual desaconsejo implementar equals en casos en que no se compara la totalidad de los campos (significativos, dejando de lado estado superfluo o calculable). Pensá que equals se utiliza en muchos contextos, como por ejemplo para saber si una lista o un diccionario contiene un elemento. Eso significa que en cualquier situación donde se haga tal comparación (incluso en código que vos no escribas, ya sean aserciones de JUnit, código de persistencia en JPA/Hibernate, etc) se consideran equivalentes a dos hechos que solo coincidan en el título, lo cual puede tener consecuencias inesperadas.

Es más, la restricción de considerar repetidos a dos hechos con el mismo nombre solo aplica para las fuentes estáticas, porque podemos asumir que las mismas tienen cierto nivel de curación por quienes trabajaron en su generación. En cambio, cuando dos personas, de forma descoordinada, carguen el futuro dos hechos con mismo nombre, los mismos conflictuarán.

[flbulgarelli]

Apuntando a que sean parametrizables una opcion seria que se inyecten en la fuente de datos estatica y en caso de mi grupo en la estructura que relaciona los hechos con sus solicitudes?

No entendí tu propuesta. ¿Podrías reformular?

[nicolasrey9]

Me referia a que se la FuenteDeDatosEstatica y el GestorDeSolicitudesDeHechos (hablado en la discusion #490) reciban por constructor el criterio con el que 2 hechos son iguales. Para que la fuente elimine los repetidos, y para que el gestor te sepa decir si un hecho ya fue eliminado. Haciendo algo del siguiente estilo:

public boolean fueEliminado(hecho) { return hechosEliminados.anyMatch( h -> criterioIgualdad.test(h, hecho) ) }

Esto cumpliria el criterio de que sea parametrizable. Y entiendo que esta bien que usen el mismo criterio(?).