MS-Marcks / python-course-integrator-project

integrator project pyton course
0 stars 0 forks source link

Proyecto Integrador - Módulo 2 #1

Open EzequielUS opened 2 years ago

EzequielUS commented 2 years ago

Hola Gerson,

Espero que estes muy bien. Por lo visto en el repositorio, ya tenes dominio de los contenidos dictados en el curso. Por lo pronto te voy a dar una devolución con respecto a la entrega del módulo dos.

Esta muy bien, sin embargo, siento que te complicas un poco en esta parte:

elif option == "2":
            name = ""
            course = ""
            cancel = False

            while True:
                name = input("Ingrese el nombre del estudiante (para cancelar la insercion presione:-1): ")
                if name == "-1":
                    cancel = True
                    break
                elif(name != ""):
                    break
                print("\n\t Error: no puede dejar el nombre vacio\n")

            while True and not cancel:
                try:
                    course = int(input("Ingrese la cantidad de cursos (para cancelar la insercion presione:-1): "))
                    if course == -1:
                        cancel = True
                        break
                    elif course > 0:
                        break
                    print("\n\t Error: los cursos deben de ser numero enteros positivos\n")
                except:
                    print("\n\tError: los cursos deben de ser un numero entero\n")

            if not  cancel:
                student.append([name,course])

Tomando en cuenta que se deberia poder agregar un solo alumno por oportunidad, podriamos simplificar un poco el código y mejorar la legibilidad.

elif option == "2":
            nombre = input("Ingrese un nombre: ")
            while nombre == "":
                nombre = input("Ingrese un nombre: ")

             course = input("Ingrese la cantidad de cursos: : ")
             while not course.isdecimal():
                 course = input("Ingrese la cantidad de cursos: : ")

            student.append([name,course])

Sin embargo, la resolución esta muy bien!

Como veo que te manejas bien con los contenidos del curso, te voy a sugerir que investigues otras cuestiones vinculadas al desarrollo de software. Cconceptos mas abstractos, como legibilidad, expresividad, principios de programación, buenas practicas y mas adelante patrones de diseño.

MS-Marcks commented 2 years ago

buen dia, en el caso del codigo lo implemente asi para evitar que el usuario corrompiera la informacion y no haya datos sin sentidos.

agregando una validación por si el usuario ya no quiere seguir agregando que tenga un metodo de escape.

Se muy bien que programar en consola resulta con mas codigos porque se tiene que tomar mas casos que cuando uno trabaja con interfaz grafica, por ello valido todas las opciones posibles para que no falle o no este propenso a fallos las aplicaciones.

Tomare su consejo de seguir mejorando, muchas gracias