Diferencia entre revisiones de «Taller 2 -- Definición de cambios a realizar»
De Wiki de EGC
(→Resultados) |
|||
(No se muestran 4 ediciones intermedias de otro usuario) | |||
Línea 1: | Línea 1: | ||
+ | |||
+ | == Datos contextuales == | ||
+ | '''Fecha:''' 5 de Noviembre de 2015 | ||
+ | |||
+ | '''Hora de inicio:''' 17:00 | ||
+ | |||
+ | '''Hora de fin:''' 17:30 (aunque con trabajos posteriores telemáticos) | ||
+ | |||
+ | == Resultados == | ||
+ | |||
+ | |||
'''General''' | '''General''' | ||
− | + | *Modificar el nombre de las clases para que el código sea más claro: | |
− | :Por ejemplo: Respuesta y Answer son dos clases diferentes que significan lo mismo pero el uso que se le da a una y otra es distinto. | + | :*Por ejemplo: Respuesta y Answer son dos clases diferentes que significan lo mismo pero el uso que se le da a una y otra es distinto. |
+ | |||
+ | *Usar el mismo idioma para todo: | ||
+ | :*Por ejemplo: hay clases que están en castellano y otras en inglés. | ||
− | + | *Clarificación de variables y métodos: | |
− | :Por ejemplo | + | :*Por ejemplo, nos encontramos con métodos cuyo nombre es "algoritmo1", "algoritmo2"... |
− | + | *Eliminar dependencias innecesarias: | |
− | :Por ejemplo | + | :*Por ejemplo: Hibernate. |
'''En subsistema: Recuento''' | '''En subsistema: Recuento''' | ||
− | + | *Tests | |
− | :No hay ningún test realizado al código (ni unitarios, de integración...) | + | :*No hay ningún test realizado al código (ni unitarios, de integración...) |
− | + | *Versiones | |
− | :Actualizar a las últimas versiones de componentes (tanto internas de agora us, como externas) | + | :*Actualizar a las últimas versiones de componentes (tanto internas de agora us, como externas) |
− | + | *Código | |
− | :Actualmente no se gestión los errores producidos. Gestionarlos mediante lanzamientos de excepción y de status en los resultados devueltos por el subsistema. | + | :*Actualmente no se gestión los errores producidos. Gestionarlos mediante lanzamientos de excepción y de status en los resultados devueltos por el subsistema. |
− | Eliminar código comentado olvidado (hay código que no se usa que simplemente se ha comentado, hay que investigar por qué y eliminarlo en caso de ser innecesario). | + | :*Eliminar código comentado olvidado (hay código que no se usa que simplemente se ha comentado, hay que investigar por qué y eliminarlo en caso de ser innecesario). |
− | Realizar abstracción de algoritmos donde sea oportuno (hay algoritmos que comparten buena parte de la funcionalidad) | + | :*Realizar abstracción de algoritmos donde sea oportuno (hay algoritmos que comparten buena parte de la funcionalidad) |
'''En subsistema: Modificación de resultados''' | '''En subsistema: Modificación de resultados''' | ||
− | + | *Controller.java | |
− | :Quitar el valor por defecto del parámetro votacionId (en el request) y si es null enviar un JSON que avise de eso. | + | :*Quitar el valor por defecto del parámetro votacionId (en el request) y si es null enviar un JSON que avise de eso. |
− | Simplificar el último for del método respuestas por un for extendido. | + | :*Simplificar el último for del método respuestas por un for extendido. |
− | Controlar las excepciones y devolver algo si se produce algún error interno, que ahora solo muestra la traza del error en la consola. | + | :*Controlar las excepciones y devolver algo si se produce algún error interno, que ahora solo muestra la traza del error en la consola. |
− | + | *Anotaciones | |
− | :Comprobar la gestión de las respuestas, ahora mismo se están comprobando si el valor es "SI" o "NO" exclusivamente. | + | :*Comprobar la gestión de las respuestas, ahora mismo se están comprobando si el valor es "SI" o "NO" exclusivamente. |
− | En caso de que se produzca un cambio en el subsistema de verificación, se debe actualizar el uso de la librería verificacion.jar.. | + | :*En caso de que se produzca un cambio en el subsistema de verificación, se debe actualizar el uso de la librería verificacion.jar.. |
Revisión actual del 19:17 17 nov 2015
Datos contextuales
Fecha: 5 de Noviembre de 2015
Hora de inicio: 17:00
Hora de fin: 17:30 (aunque con trabajos posteriores telemáticos)
Resultados
General
- Modificar el nombre de las clases para que el código sea más claro:
- Por ejemplo: Respuesta y Answer son dos clases diferentes que significan lo mismo pero el uso que se le da a una y otra es distinto.
- Usar el mismo idioma para todo:
- Por ejemplo: hay clases que están en castellano y otras en inglés.
- Clarificación de variables y métodos:
- Por ejemplo, nos encontramos con métodos cuyo nombre es "algoritmo1", "algoritmo2"...
- Eliminar dependencias innecesarias:
- Por ejemplo: Hibernate.
En subsistema: Recuento
- Tests
- No hay ningún test realizado al código (ni unitarios, de integración...)
- Versiones
- Actualizar a las últimas versiones de componentes (tanto internas de agora us, como externas)
- Código
- Actualmente no se gestión los errores producidos. Gestionarlos mediante lanzamientos de excepción y de status en los resultados devueltos por el subsistema.
- Eliminar código comentado olvidado (hay código que no se usa que simplemente se ha comentado, hay que investigar por qué y eliminarlo en caso de ser innecesario).
- Realizar abstracción de algoritmos donde sea oportuno (hay algoritmos que comparten buena parte de la funcionalidad)
En subsistema: Modificación de resultados
- Controller.java
- Quitar el valor por defecto del parámetro votacionId (en el request) y si es null enviar un JSON que avise de eso.
- Simplificar el último for del método respuestas por un for extendido.
- Controlar las excepciones y devolver algo si se produce algún error interno, que ahora solo muestra la traza del error en la consola.
- Anotaciones
- Comprobar la gestión de las respuestas, ahora mismo se están comprobando si el valor es "SI" o "NO" exclusivamente.
- En caso de que se produzca un cambio en el subsistema de verificación, se debe actualizar el uso de la librería verificacion.jar..