Respondiendo primero a tu pregunta:
¿Todas la líneas de código deben estar dentro de la función menu()?
Para comenzar, hay que definir bien cual es el comportamiento que esperamos de nuestro programa. Luego dividir en piezas funcionales ese comportamiento y, aunque parezca que no tiene importancia, colocar nombres adecuados a nuestras funciones.
Si una función debe realizar tal o cual tarea, debiera quedar claro solo con ver su nombre. Por esa misma razón, siempre trato de asignar a las funciones nombres que comienzan por un verbo en infinitivo. Eso añade mucha claridad.
Mira, sin ver el código, lo difícil que sería decir qué es lo que tiene que hacer la función menu()
.
En cambio, si la la función se llamase mostrarMenu()
o imprimirMenu()
, podríamos decir que el código luce razonablemente completo.
Pero si la función se llamara operarMenu()
diría que hace falta el resto del código, además de otras cosas.
Ahora, vamos primero a la definición del comportamiento esperado.
Yo, como usuario, cuando se me presenta un menú, espero poder elegir una opción seguida de otra y otra, hasta que elijo salir del menú (tu menú parece ir en la misma línea, de otra forma no tendría una opción para salir.
Luego, dividamos esto en partes.
Podríamos tener las siguientes funciones,
- Una que se encargue de imprimir el menú en pantalla
imprimirMenu()
,
- Otra que se encargue de interactuar con el usuario para que ingrese una opción y retorne solo hasta que el usuario ingresó una opción válida. Esta se llamará
pedirOpcion()
. Retornará solo si el usuario ha ingresado un número entre 1 y 6, y retornará este valor como un número entero. Para escribir esta función, te invito a leer esta respuesta, que te dará los principios de cómo validar lo ingresado por el usuario.
- Otra que se encarga de integrar todo, es decir, llama a una función para imprimir el menú, luego a la función para pedir el valor seleccionado al usuario, luego a las que ejecutan las acciones seleccionadas y repetir todas estas operaciones hasta que el usuario elige salir. Esta se llamará
operarMenuPrincipal()
No voy a escribir cada función por ti, pero si la principal, basada en tu código, agregando un ciclo para que se repita hasta que el usuario quiera salir.
def operarMenuPrincipal():
opcion = 0
while (opcion != 6)
imprimirMenu()
opcion = pedirOpcion()
if opcion == 1:
funcion_Conectar_ssh(["ssh_device_1" , "10.164.25.1"] , cmd)
elif opcion == 2:
funcion_Conectar_ssh(["ssh_device_2" , "10.181.2.1"] , cmd)
elif opcion == 3:
funcion_Conectar_ssh(["ssh_device_3" , "10.90.123.1"] , cmd)
elif opcion == 4:
funcion_Conectar_ssh(["ssh_device_4" , "10.90.123.41"] , cmd)
elif opcion == 5:
funcion_Conectar_ssh(["ssh_device_5" , "10.182.22.1"] , cmd)
elif opcion == 6:
print("Bye.....")