obj1-unahur-2024s1 / TPGameIntegrador-pixelados

TPGameIntegrador-pixelados created by GitHub Classroom
0 stars 0 forks source link

correccion #1

Open lspigariol opened 1 week ago

lspigariol commented 1 week ago

el juego quedo simpatico, pero su codigo es muy procedural

generalizar niveles y cantidad de letras de las palabras, evitando repetir logica, evitando preguntar que nivel es. ganaste y perdiste son parecidos. modelar con objetos (tal vaz subclases de Fondo) una clase Palabra / Letra podria ayudar La logica de las letras es similar, no repetir logica cada nivel podria conocers sus palabras, su musica, etc.

AlexisGiordaUnahur commented 1 week ago

cambios hechos: -los niveles pasaron a ser objetos -se reemplazaron los métodos ganaste y perdiste por un objeto pantallaFinal
-se corrigió la lógica de posición de las letras correctas usando palabraActiva.indexOf(letra)*2+3 -se eliminó el objeto sonidos, ahora cada nivel conoce su canción. -no se pudo evitar el if que pregunta por el nivel y la cantidad de aciertos, se aceptan sugerencias para corregir eso (method hay_EnPalabra(letra))

AlexisGiordaUnahur commented 1 week ago

Más cambios: -se mejoró la lógica de hay_EnPalabra(letra) para no preguntar por el nivel y cantidad de aciertos. Ahora pregunta por lo errores pero hasta ahí llega mi creatividad, no encontré otra forma