No vas mal encaminado en cuanto a la causa del problema, simplemente has descubierto la recursividad. Cuando llamas a menu_Principal
dentro de menu_Principal
lo que haces es una llamada recursiva, de forma que ejecutas de nuevo la función al completo con su propio ciclo while
. La ejecución de la función "padre" queda detenida en ese punto hasta que la llamada a la función "hija" retorne. En el momento que la ejecución de la "hija" retorne la ejecución del "padre" continuará desde el punto donde llamó a la "hija". Esta cadena se repite tantas veces como llamadas recursivas hagas. Por lo tanto, cuando ingresas la opción 5 ejecutas de nuevo la función, al pulsar 6 sales de esta segunda llamada pero vuelves a la inicial.
Para ilustrarlo algo mejor vamos a ver un ejemplo muy simple con algunos prints:
def foo(n):
print("Ejecutada llamada recursiva {}.\n".format(n))
while True:
op = input("Ingrese una opción:\n 1- Nueva llamada recursiva.\n 2- Salir.\n\n>>> ")
if op == '1':
foo(n+1)
elif op == '2':
break
print("Terminando llamada recursiva {}.\n".format(n))
foo(1)
Ejemplo de ejecución:
Ejecutada llamada recursiva 1.
Ingrese una opción:
1- Nueva llamada recursiva.
2- Salir.
>>> 1
Ejecutada llamada recursiva 2.
Ingrese una opción:
1- Nueva llamada recursiva.
2- Salir.
>>> 1
Ejecutada llamada recursiva 3.
Ingrese una opción:
1- Nueva llamada recursiva.
2- Salir.
>>> 2
Terminando llamada recursiva 3.
Ingrese una opción:
1- Nueva llamada recursiva.
2- Salir.
>>> 2
Terminando llamada recursiva 2.
Ingrese una opción:
1- Nueva llamada recursiva.
2- Salir.
>>> 2
Terminando llamada recursiva 1.
Lo más simple para solucionarlo es crear una cadena que contenga el menu y volver a imprimirlo en la opción 5:
def menu_Principal():
menu = ("Bienvenido al Reloj Mundial"
"\n\nEstas son las operaciones que puedes realizar"
"\n\n1. Ver hora."
"\n2. Ver fecha y hora."
"\n3. Ver la hora en New York."
"\n4. Ver la hora en San Francisco"
"\n5. Ver las instrucciones nuevamente."
"\n6. Salir del programa.")
print(menu)
while True:
opcion = input("\nIngrese opción: ")
if opcion == '1':
hora_local()
elif opcion == '2':
hora_fecha_local()
elif opcion == '3':
hora_Ny()
elif opcion == '4':
hora_Sf()
elif opcion == '5':
os.system("pause")
os.system("cls")
print(menu)
elif opcion == '6':
print("\nGracias por usarnos.")
break
else:
print("\nPor favor Ingrese una opción de la lista.")
Nota: No se si es el comportamiento esperado pero debes tener en cuanta que tal y como tienes el código la hora y fecha se toma al ejecutar el script, por lo que las funciones muestran esa hora, no actualizan con el paso del tiempo.Es decir, si ejecuto el script y al pasar 15 minutos ingreso una opción la hora mostrada no es la actual, es la de hace 15 minutos. Si no quieres que esto pase debes llamar a
datime.now
en cada función y no al inicio del script.
Edición
Con respecto a la duda que planteas sobre la línea datetime.datetime.now()
que queda muy larga y como comentas es tedioso escribirla completa cada vez, tienes varias posibilidades, pero lo más "pythónico" creo que es:
# Importar la clase datetime
from datetime import datetime
print(datetime.now())
hora = datetime.now().strftime("%I : %M : %S %p")
# Si quieres reducir aún más puedes crear un "alias"
from datetime import datetime as dt
print(dt.now())
hora = dt.now().strftime("%I : %M : %S %p")