Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Comentarios entrega #2 #3

Closed
4 tasks done
edjus opened this issue Nov 26, 2019 · 0 comments
Closed
4 tasks done

Comentarios entrega #2 #3

edjus opened this issue Nov 26, 2019 · 0 comments

Comments

@edjus
Copy link

edjus commented Nov 26, 2019

Les dejo un par de comentarios sobre mejoras o correcciones:

  • Eliminar código comentado, por ejemplo en la clase EstadoArma, Jinete, etc, además limpien el resto de código comentado.

  • En las clases correspondientes a las diferentes Distancias, existen números mágicos, reemplazar por cosntantes y agregarlo al ProveedorDeConstantes.

  • Sugerencia: En el método atacar(Unidad unidad, Tablero tablero) de Unidad en lugar de atrapar UnidadDestruidaException y luego quitar la pieza tal vez sea mejor dejar seguir la excepción y atraparlo en un nivel superior Juego o AlgoChess y sea éste el que se encargue de remover la pieza destruida delegando a tablero. Ya que como está ahora da a entender que el atacar hace dos cosas, ataca y si está destruido se encarga de quitarlo del tablero.

  • La clase UnidadAtacante no se usa, es es correcto? Si no se usa removerlo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants