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

Update importante (leer) #34

Open
guido-z opened this issue Nov 19, 2016 · 5 comments
Open

Update importante (leer) #34

guido-z opened this issue Nov 19, 2016 · 5 comments

Comments

@guido-z
Copy link
Collaborator

guido-z commented Nov 19, 2016

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
Copy link
Collaborator

ldtg commented Nov 19, 2016

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
Copy link
Collaborator Author

guido-z commented Nov 19, 2016

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
#34 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/ASdG44yOdB67JDUOZ88tyhBcrLfB5b_7ks5q_k_KgaJpZM4K3FE0
.

@wjma
Copy link
Collaborator

wjma commented Nov 19, 2016

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
Copy link
Collaborator Author

guido-z commented Nov 19, 2016

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
#34 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/ASdG49MEwuC3CNbzKmtfU9JnW6aITd80ks5q_lIvgaJpZM4K3FE0
.

@guido-z
Copy link
Collaborator Author

guido-z commented Nov 19, 2016

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
#34 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/ASdG49MEwuC3CNbzKmtfU9JnW6aITd80ks5q_lIvgaJpZM4K3FE0
.

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

3 participants