keze87 / TP2

0 stars 1 forks source link

Update importante (leer) #34

Open guido-z opened 7 years ago

guido-z commented 7 years ago

Ya empecé a integrar la batalla en la vista, hasta ahora se pueden usar ataques y recibir daño. Tvue que implementar algunas excepciones nuevas para mostrar mensajes sobre algunos eventos importantes (un algoMon se queda dormido, un algoMon dormido no puede atacar, etc) en la consola, y por eso se rompieron algunos tests, pero no es nada grave, solo hay que agregar algunos try/catch.

Lo que voy a necesitar es que testeen el juego desde la interfaz así hacemos más rápido, y me avisen sobre cualquier error o incoherencia que aparezca.

ldtg commented 7 years ago

Es una boludez pero yo probe con dos algomones iguales en el equipo y no se entendia quien tenia el turno deberia tener un lugar donde diga quien tiene el turno, disculpa que no pueda hacer mucho ahora pero mañana trato de hacer algo de la interfaz.

guido-z commented 7 years ago

Eso es una cosa que hay que agregar, también un indicador en los botones para elegir algoMones para saber que están seleccionados.

El 18 de noviembre de 2016, 22:24, Luis Tejerinanotifications@github.com escribió:

Es una boludez pero yo probe con dos algomones iguales en el equipo y no se entendia quien tenia el turno deberia tener un lugar donde diga quien tiene el turno, disculpa que no pueda hacer mucho ahora pero mañana trato de hacer algo de la interfaz.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/keze87/TP2/issues/34#issuecomment-261682682, or mute the thread https://github.com/notifications/unsubscribe-auth/ASdG44yOdB67JDUOZ88tyhBcrLfB5b_7ks5q_k_KgaJpZM4K3FE0 .

wjma commented 7 years ago

Che el test 10 de AlgomonTest falla porque hay una falla en la cantidad de ataques: AtaqueBuilder builder = new AtaqueBuilder(NombreAtaque.ATAQUERAPIDO, 10, /*16*/2, new SinEfecto());

Ahi en vez de ese 2 iria el 16

guido-z commented 7 years ago

Ya sé, le bajé los usos máximos para probar unas cosas más rápido

2016-11-18 22:34 GMT-03:00 wjma notifications@github.com:

Che el test 10 de AlgomonTest falla porque hay una falla en la cantidad de ataques: AtaqueBuilder builder = new AtaqueBuilder(NombreAtaque.ATAQUERAPIDO, 10, /16/2, new SinEfecto());

Ahi en vez de ese 2 iria el 16

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/keze87/TP2/issues/34#issuecomment-261683514, or mute the thread https://github.com/notifications/unsubscribe-auth/ASdG49MEwuC3CNbzKmtfU9JnW6aITd80ks5q_lIvgaJpZM4K3FE0 .

guido-z commented 7 years ago

Es más, hasta dentro de un rato van a fallar unos cuantos a causa de algunas excepciones nuevas que agregué, pero ya me estoy ocupando.

El 18 de noviembre de 2016, 22:36, Guido Zaglulguido.zaglul@gmail.com escribió:

Ya sé, le bajé los usos máximos para probar unas cosas más rápido

2016-11-18 22:34 GMT-03:00 wjma notifications@github.com:

Che el test 10 de AlgomonTest falla porque hay una falla en la cantidad de ataques: AtaqueBuilder builder = new AtaqueBuilder(NombreAtaque.ATAQUERAPIDO, 10, /16/2, new SinEfecto());

Ahi en vez de ese 2 iria el 16

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/keze87/TP2/issues/34#issuecomment-261683514, or mute the thread https://github.com/notifications/unsubscribe-auth/ASdG49MEwuC3CNbzKmtfU9JnW6aITd80ks5q_lIvgaJpZM4K3FE0 .