Closed LtDeno closed 5 months ago
Correcciones acatadas y solucionadas. Reentrega lista para corrección.
Buenas! Como estan?
El estado del TP1 se encuentra en Reentrega
Todas las correcciones fueron hechas pero hubo problemas al implementar la configuración de la grilla, se lanzan excepciones al configurar de una manera especifica.
Corecciones
Establezcan un máximo de filas y columnas para la grilla. Al querer generar una grilla muy grande como de 1000x10, se lanza una sucesion de RuntimeException
. O si creo una de 10000x1000 se lanza una sucession de OutOfMemoryError
.
Hay métodos de clases que no le definieron el tipo de método. Establezcan si son públicos o privados
Fecha limite de reentrega: Domingo 19/05
Disculpen la tardanza. Arranque la semana engripado. Cualquier duda me consultan por slack o me abren un issue por acá, como les sea mas cómodo.
Saludos!
Correcciones acatadas y solucionadas. Reentrega lista para corrección.
Buenas! Como estan?
El TP1 esta Aprobado
Nota Final: 10 (Diez)
Correcciones hechas con exito.
Espero que sigan bieen!! Saludos!!
Buenisimo! Muchas gracias, Dante.
Buenas! Como andan?
El estado del TP1 se encuentra en Reentrega
Aplican bien el polimorfismo y la herencia. Les quedo bastante linda la interfaz de usuarioy bastante usable, salvo por algunos defectos que encontre.
Corecciones
De la interfaz grafica:
Como usuario no puedo configurar la grilla.
Telepot Sofely debe tener un limite de 1 uso por nivel.
Una vez que fueron eliminado los robots, se debe pasar de nivel al instante (o de forma mas comoda posible). Para el usuario es muy tedioso tener que mover al jugado dos veces para pasar de nivel. Podrian usar un
Label
diciendo: 'has pasado de nivel' con un boton de OK, por ejemplo, sin tener que hacer algo extra para que aparezca dicho mensaje.Cuando se inicia el juego y se cierra la ventana el sistema sigue abierto esperando intrucciones, el sistema deberia terminar una vez que se cierra la ventana.
De codigo:
Saquen los 'System.out.println'. Estan imprimiendo cosas en pantalla, como por ejemplo en el metodo
readConfigFile
de la claseConfigurator
.El metodo
assignEvents
de la claseGameController
les quedo demasiado largo. Separen funcionalidades. Dividan responsabilidades entre las clases.Fecha limite de reentrega: Viernes 10/05 La verdad que no son cambios tan grandes. Por eso les doy una semana nomas
Cualquier duda me consultan por slack o me abren un issue por aca, como les sea mas comodo.
Saludos!