[IV-24-25] Objetivo 5 #38
Open
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
Se decide emplear el estándar de golang
Se emplea el estándar de golang
…mantenimiento y corrige redacción #20
Se eliminan números mágicos, comentarios innecesarios, constructor y otras funciones innecesarias
Deshacemos cambios en el código para reestructurarlos y que se muestren adecuadamente en los issues
Este test comprueba que no haya más de un enfermero asignado al mismo área al mismo tiempo en el plan generado
Comprueba que no falte nadie en un área en cualquier momento
Comprueba que la información sobre sus turnos se muestra correctamente
Crea el planning de turnos para todo un año dados los enfermeros contratados
Convierte los turnos asignados a un enfermero en string y lo devuelve
Un año puede durar 365 o 366 días por lo que es incorrecto llamar a la constante de esa manera
Este dato es empleado en la comprobación de las áreas ocupadas cada día y turno
Se corrige el comando elegido para comprobar la sintáxis de los archivos fuente, debido a un error en la corrección
La modificación del modelo ha sido un error y no debería haber abordado el tema de la fecha de esa manera, no aporta valor a la lógica del milestone 1 ni forma parte del pmv
Nos deshacemos de ella porque no forma parte del pmv
Para obtener el plan deberemos generarlo a través de los datos que se nos ofrezca por lo que se creará la función en base a los tests siguiendo TDD
Este test comprueba si hay áreas sin ocupar y las notifica, en lo que será un error
Este test comprueba si hay más de una persona asignada a un área al mismo tiempo, que indica sobreocupación, y lo notifica
Aplicamos el principio DRY para evitar repetición de código creando el dato Ocupación que guarda el número de enfermeros en cada área en cada momento
Tras completar los tests a tener en cuenta, completamos la función de creación del plan siguiendo el TDD
Se establecen las bases para considerar correcta la solución a aplicar para que los enfermeros comprueben sus turnos
Se crea la función que muestra al enfermero el área al que debe acudir
Al incluir una hora indeterminada usando time.Now() las igualdades entre fechas eran erróneas, se opta por crear una función que normalice las fechas dejando la hora en 0
|
Buenas, se me ha asignado para revisar tu PR. Te he dejado algunos comentarios, pero antes dos cosas:
|
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.
No description provided.