Revisión de código basada en patrones de diseño#4
Open
clopezno wants to merge 52 commits intoclopezno:masterfrom
Open
Revisión de código basada en patrones de diseño#4clopezno wants to merge 52 commits intoclopezno:masterfrom
clopezno wants to merge 52 commits intoclopezno:masterfrom
Conversation
Cambio de metodos y nombres de clases al ingles Se elimina el archivo test.txt que se creo que se creo como archivo de pruebas de commit
Getters y setters accesibles en el modelo Versión sin código útil
Creado el singleton
coge de un path del terminal (ruta relativa => mejor).
Añadida clase para test de la fachada binaria (clase vacía)
Conflicts: src/ubu/lsi/dms/agenda/persistence/BinaryFacade.java Conflicto de la clase BinaryFacade resuelto
Se han eliminado imports sobrantes
Main para realizar pruebas
Modificado el archivo .gitignore para que se puade hacer commit de los archivos .jar
ahorramos el crearla repetidas veces).
excepción). Se ha considerado que, ya que se ha pasado la fase de debugging, no es necesaria tanta información. Ahora, para asegurarnos de que las conexiones se emplean correctamente, se inicializan los recursos en bloques try-with-resource, en los que tenemos la certeza de que se cierre dicho recurso, al contrario que cerrándolo manualmente en bloques try-catch-finally en los que no se asegura su cierre. Cambio de los nombres de algunas variables para mejorar la comprensión del código. Retirada del main: las pruebas pasarán a realizarse desde el paquete correspondiente.
Se crean metodos para la inicializacion de los datos de prueba: - createContacts() - createCalls() - createContactTypes() Se crean metodos de prueba para los metodos de la fachada - testInsert() - testUpdate() - testGet()
Se ha creado una clase que inicialice una serie de datos comunes para poder realizar las pruebas correspondientes en las clases pertinentes.
…de creacion de datos CommonData
Se ha solucionado un bug por el cual, al seleccionar un contacto, no devolvía nada, ya que al no haber tipos de contacto, la select no devolvía nada. Ídem con la sentencia que devuelve todos los contactos con un mismo apellido.
Solucionado un problema que impedía que las fechas se insertasen correctamente en la base de datos.
Para evitar problemillas, como la BBDD tiene una precisión en tiempo de nanosegundos, pues le insertamos el valor con dicha precisión, y así esperamos exactamente el mismo valor.
La actualización de las llamadas, pues eso, hay que hacerla sobre las llamadas y no sobre los contactos.
Métodos equals() generados para las clases Call, Contact y ContactType. Además, en la clase Contact se ha eliminado el método compareTo() por no tener la utilidad esperada.
Se han creado las pruebas necesarias para comprobar el correcto funcionamiento de la persistencia en BBDD.
Cambiada la forma de comparar contactos en BinaryFacade (se utiliza equals en vez de compareTo)
- Eliminados metodos loadContacts, loadCalls, loadContactTypes
- Añadido metodo loadFile que sustituye a los 3 anteriores
- Retocados nombres de variables para que sigan la misma convencion de nombres
Modificada la clase BinaryFactory:
- Añadido patron singleton a la clase
Modificada la clase BinaryFacadeTest.java:
- Correcta instanciacion de la fabrica y de la fachada
Modificada la clase DBFactory: - Añadido patron singleton a la clase Modificada la clase BinaryFacadeTest.java: - Correcta instanciación de la fabrica y de la fachada
En referente a documentación añadidos algunos comentarios en inglés y parte del informe.
correctamente formateadas
Cambio de paquete de ubu.lsi.dms.agenda.test a ubu.lsi.dms.agenda.test.persistencia
Owner
Author
There was a problem hiding this comment.
Nombre de la clase en español y resto de la documentación en Inglés
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
5 participants
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.