Mejoras en entidades - K3001 - Sartorio Delfina#2
Open
delfisartorio wants to merge 1 commit intodds-utn:mainfrom
Open
Mejoras en entidades - K3001 - Sartorio Delfina#2delfisartorio wants to merge 1 commit intodds-utn:mainfrom
delfisartorio wants to merge 1 commit intodds-utn:mainfrom
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Cree la interfaz IAlertas para poder hacer uso de la nueva clase AlertaTyH (el ejercicio pedía una alerta que solo se fijaba en la temperatura y humedad). La clase clima tiene un atributo de tipo IAlertas para seguir con los principios Responsabilidad Única y Sustitución de Liskov, asi la lógica de cuando es una alerta o no esta encapsulada en las implementaciones de la interfaz y a su vez nos permite tener multiples tipos de alerta sin modificar al clima, haciéndola una clase con mejor mantenibilidad y extensibilidad. Por otro lado intente completar el TODO de enviar email, aplicando el patron adapter pero lo deje por la mitad porque sentia que me estaba metiendo en cosas que no sabia e iba a caer en un copypasteo de todo. Por ultimo, para implementar estos cambios en el AlertaServices, cree un metodo en el clima que se encarga de verificar si hay alerta activa, el cual verifica si el atributo es distinto de null y si el clima cumple con las condiciones de alerta.