Taller 2 -- Definición de cambios a realizar

De Wiki de EGC
Revisión del 21:38 5 nov 2015 de Rafriosan (discusión | contribuciones) (Página creada con «== Taller 2 -- Definición de cambios a realizar == '''General''' - Modificar el nombre de las clases para que el código sea más claro: :Por ejemplo: Respuesta y Answer...»)
(dif) ← Revisión anterior | Revisión actual (dif) | Revisión siguiente → (dif)
Saltar a: navegación, buscar

Taller 2 -- Definición de cambios a realizar

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"...

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..