agustisanchez/trivia

Legacy Code Retreat - Trivia Game codebase

trivia

Legacy Code Retreat - Trivia Game codebase

• Permitir más jugadores (actualmente limitados a 6).
• Limpieza del código

Requiere Java 8

trivia's Issues

`currentCategory()` no es realmente random

Game.currentCategory() no es realmente random sino que viene determinado por la posición del jugador.

Esto crea el problema añadido de depender de un parámetro definido en otra clase (Player.MAX_PLACES).

¿Debería ser Game.currentCategory() random e independiente de la posición?

String currentCategory(){
   ...
   return categories.get(random.nextInt(categories.size());
}

Permitir más jugadores

Permitir más jugadores.
Actualmente está limitado a 6 debido a limitaciones del diseño.

Introducir principios SOLID y otras buenas prácticas OOP.

Eliminar código muerto

Eliminar código que no se usa.

`remove(playerName)` da IndexOutOfBoundsException

remove(playerName) no elimina el jugador especificado sino el último introducido

Limpieza de código

Eliminar código que no se usa.

Si número de jugadores elevado, el juego falla porque se queda sin preguntas

Si número de jugadores elevado, el juego falla porque se queda sin preguntas.

¿Como tratar este caso?

Refactorizar estructura proyecto

• Se eliminan los .class del directorio de fuentes
• Se mueven la carpeta Metrics y los archivos Eclipse un nivel más arriba para dejar los fuentes limpios
• Se crea una carpeta classes y se define en .gitignore