MarcelaGitHub22 / Aguero-Python

0 stars 0 forks source link

Revisión final #2

Open LucianoEspe opened 3 months ago

LucianoEspe commented 3 months ago

Hola Marcela, estuve revisando tus códigos: El de la biblioteca tiene algunos detallitos que corregí y me funcionó. El menú comprobaba las opciones 1, 2, 3, ... pero en realidad el input estaba devolviendo el valor en formato string, entonces modifiqué a "1", "2", ... Por otro lado, las bibliotecas se creaban dentro del bucle while, entonces en cada vuelta se estaban reseteando y se borraban los libros en su interior. El ejercicio del tamagotchi tiene una particularidad, siempre está muerto 😂. Es por un tema de comprobación, quizás en verificar estado deberías retornar el propio estado, True si está vivo y False si está muerto. Otra cosa a comprobar son los valores mínimos y máximos para no tener por ejemplo Hambre: -10 Lo demás todo excelente, felicitaciones.

MarcelaGitHub22 commented 3 months ago

Gracias Luciano. No pude estar la última clase, ya que estuve de viaje. Muchas gracias por todo

El sáb, 1 de jun de 2024, 23:47, Luciano Esperlazza < @.***> escribió:

Hola Marcela, estuve revisando tus códigos: El de la biblioteca tiene algunos detallitos que corregí y me funcionó. El menú comprobaba las opciones 1, 2, 3, ... pero en realidad el input estaba devolviendo el valor en formato string, entonces modifiqué a "1", "2", ... Por otro lado, las bibliotecas se creaban dentro del bucle while, entonces en cada vuelta se estaban reseteando y se borraban los libros en su interior. El ejercicio del tamagotchi tiene una particularidad, siempre está muerto 😂. Es por un tema de comprobación, quizás en verificar estado deberías retornar el propio estado, True si está vivo y False si está muerto. Otra cosa a comprobar son los valores mínimos y máximos para no tener por ejemplo Hambre: -10 Lo demás todo excelente, felicitaciones.

— Reply to this email directly, view it on GitHub https://github.com/MarcelaGitHub22/Aguero-Python/issues/2, or unsubscribe https://github.com/notifications/unsubscribe-auth/BG6GFQOE672YA6EM3OQ4T7TZFKBTZAVCNFSM6AAAAABIUVF37WVHI2DSMVQWIX3LMV43ASLTON2WKOZSGMZDSNBTG4ZTGMI . You are receiving this because you are subscribed to this thread.Message ID: @.***>