tfloxolodeiro
commented
4 years ago Tambien hablamos sobre mostrar los puntos, pueden pueden empezar implementandolo de una forma simple con un game.say() durante o al final del juego, y después si tienen tiempo pueden hacer algún indicador mas complejo. También hablamos sobre nuevas ideas a agregar después de terminar las correcciones, como agregar buffs y debuffs para el carpincho/jaguarete como ser invulnerable.
¡Buenas! Hoy se van a encontrar con Tom, quien les responderá dudas y les contará las cosas a corregir. Mientras dejo por acá algunas correcciones, Tom agregará o explicará mejor:
Antes que nada, bien, gente, por tener lo que planteamos la clase pasada :+1:
Veo que gastaron un buen rato en la lógica de perseguir, en el juego quedó bastante bien, felicitaciones. Se puede jugar y está copado.
Pero ojo, su tp tiene varias fallas, muchísima pero muchísima repetición de lógica (:crying_cat_face: :scream_cat: :crying_cat_face: :scream_cat: ). Una forma de darse cuenta es: en Linux, Wollok distingue mayúsculas, así que algunas de sus imágenes no me andaban. Adivinen cuántas "carpincho50.png" tuve que cambiar en el código por "Carpincho50.png" ... ¿Cuántas debería haber cambiado?... O sea, no me importa lo de las mayúsculas, no hace falta que lo cambien, pero sí hay que cambiar la repetición, porque ante un pequeño bug hay que tocar en 1000 lugares.
En general, repiten código porque a su código le faltan abstracciones. Por ejemplo, varias de las repeticiones se pueden arreglar haciendo una superclase, o creando un objeto al que delegarle responsabilidades.
[x] El mate, el mate de oro, y el alfajor (me emocioné al jugar, les juro :laughing: ) tienen exactamente el mismo comportamiento, lo único en lo que difieren es la imagen y los puntos que aportan.
[x] Las teclas de los tres modos son exactamente iguales, cambian dos boludeces. Hay muchísima repetición. Sería genial no tener que repetir esos 4 onPressDo 4 veces. Si configurar teclas siempre significa hacer lo mismo con las teclas wasd, ¿Por qué no abstraerlo y escribir esos cuatro renglones en un sólo lugar?
Péguenle un vistazo al objeto dirección que mostraron ivo y el rasta la clase que contaron Wollok Game.
Si se fijan en la bitácora hay pautas para la entrega siguiente:
[x] Tests: ¿Qué parte del sistema les parecería interesante testear para verificar que funcione como se espera y que no se rompa a futuro? Deberían tener algunos tests.
[x] Diagramas: ¿Qué parte del diseño de la solución les copó y les gustaría hacer algún diagrama para contar?
[x] Documento: si quieren pueden ir chusmeando las preguntas que hay en las pautas y ver si tienen alguna duda.
Hasta ahí lo que pude revisar yo, sigan revisando con Tom :grin: