JonathanGalvanPerez
commented
5 years ago Tambien nos marcó que hay demasiados paquetes. Pero como ya lo entregamos con los diagramas no se si vale la pena. Voy a ver si puedo hacer las funciones getter.
Closed FacuMastri closed 5 years ago
JonathanGalvanPerez
commented
5 years ago Tambien nos marcó que hay demasiados paquetes. Pero como ya lo entregamos con los diagramas no se si vale la pena. Voy a ver si puedo hacer las funciones getter.
diaznicolasandres1
commented
5 years ago Agregue la sintaxis para que todo boton tenga imagen. Habria que ver si se pueden mejorar las que ocupan varios casilleros. Y achicar un poco el mapa y agrandar los botones en todo caso para una mejor visualizacion
Dejo la lista de cosas a mejorar (que no necesariamente están mal) que nos mencionó hoy Tomás:
[x] Cambiar los instanceOf para los botones por, por ejemplo, un getter (fijarse si ya hay algún método que hace esto) de manera que cada colocable (unidad/edificio) devuelva su clase en forma de String.
[x] Relacionado con lo anterior, si se implementa eso se debería sacar los colores de los botones y usar el String devuelto para buscar y abrir imágenes para cada entidad.
[x] Similar al primer item, usar un getter (fijarse si ya hay algún método que hace esto) para devolver el estado de cada entidad (en forma de String) y poder mostrarlo al usuario.
[ ] Cuando se selecciona un casillero con un colocable se debería dar algún tipo de indicio de que ese casillero fue seleccionado (aumentar/disminuir brillo, etc..)