Documentación de Game: ¿qué pasa al configurar dos veces?

asanzo commented 3 years ago

Me gustaría que la doc detalle el comportamiento más específicamente con estos métodos (o sea, especificar un poco más el contrato):

Key.onPressDo(...) ¿Qué pasa si se llama dos veces? ¿La segunda tapa a la primera? ¿Cómo hago para desactivar un onPressDo(...)? ¿Quizás debería existir un removePressedActions()?
game.onCollideDo(visual,action) ¿Qué pasa si se llama dos veces con el mismo visual? ¿La segunda tapa a la primera? ¿Cómo hago para desactivar colisiones sino? (Idem whenCollideDo)
game.onTick(ms, name, action) ¿Qué pasa si se llama dos veces con el mismo nombre? (ver siguiente)
game.removeTickEvent(name) ¿Qué pasa si existen varios onTick con el mismo nombre? ¿cuál remueve? (el @PalumboN me había dicho que removía uno solo, y que no había garantías de que fuera el último) (bueno, me gustaría ver eso en la doc).

En particular, con el onTick pasa mucho que reusan el mismo nombre (x ejemplo para disparar muchos disparos y que cada uno se vaya moviendo).

Creo que me hice una mamushka de issues acá.

lspigariol commented 1 month ago

Me parece util lo de remover y remplazar el onPressDo

asanzo commented 1 month ago

Por lo que ví en la implementación de onPressDo, llamar dos veces con la misma tecla encola hooks.

Si yo hago k.onPressDo(gritar) y k.onPressDo(saltar), luego cuando aprieto la k hace ambas.

Este issue es para documentar, quizás además en otro podemos pensar un k.removePressEvents() o algo así como dice Lucas.

asanzo commented 1 month ago

O que el comportamiento de onOressDo "pise" el anterior en lugar de encolae

PalumboN commented 1 month ago

No me parece mal que haga ambas cosas. Y +160 a poder controlar las actions de las teclas usando las keys.

En particular, con el onTick pasa mucho que reusan el mismo nombre

Eso es porque no usan los objetos Tick. La api oficial debería ser esa (actualizar la doc dónde haga falta).

Sobre las colisiones banco documentar más en detalle y hasta decir las "buenas prácticas" para que los juegos no anden lento?

Igual hay que abrir otro issue para discutir más en profundidad y darle una nueva iteración a eso.

asanzo commented 1 month ago

Me gusta

asanzo commented 1 month ago

Quiero hablar de esto el sábado

asanzo commented 1 month ago

Charlamos hoy en el hackaton:

@npasserini dice que si quiero mantener el feature de encolar comportamientos, y poderlos borrar, onPressDo podría devolver el id (lo cual no rompería hacia atrás). Y tener el método removePressEvent(id) o algo así.
- @lspigariol dice que es un bug (no un feature) que el onPressDo encole comportamientos.
- @CeciC24, @FedericoEncinazSayago y @facu777 están implementando para jugar el cambio para que onPressDo reemplace.

CeciC24 commented 1 month ago

Comenzamos a arreglarlo en la branch onPressDo-replaces

Falta:

[ ] Cambiar el wollokdoc de onPressDo para que cuente que reemplaza la acción anterior
[x] Hacer test

uqbar-project / wollok-language