Si quieres que el archivo se pida de inicio antes de mostrar el menú, simplemente sacalo del menú y coloca el código al inicio del mismo. Puedes dejar abierto el archivo durante la ejecución o reabrirlo cuando sea necesario, pero usando el nombre ingresado al inicio.
No obstante unas cuantas observaciones:
Usas POO pero tus clases son meros contenedores de código o funciones, conceptualmente ni las creas ni las usas como clases. Python es un lenguaje multiparadigma, es más no te obliga a encapsular tu código en clases o funciones si no quieres. Más que forzar una implementación mediante POO, te recomendaría que dividideras el código conceptualmente en funciones sin más.
Te recomiendo no usar nombres con mayúsculas para funciones, atributos, variables, métodos o módulos. Todos ellos por convención se nombran en minúscula y usando _
como separador de palabras. El CamelCase se reserva para nombres de clases exclusivamente y TODO_EN_MAYÚSCULAS para variables que actúan como constantes.
Nunca te dejes archivos abiertos sin cerrar de forma explícita, el GC terminará haciendo el trabajo pero no sabemos cuándo di como. Es algo que no se debe dejar al recolector de basura. Si puedes mejor usar with
y el administrador de contexto para ello, el archivo se cerrará de forma segura incluso si hay excepciones de por medio.
Cuando trabajes con entradas de usuario, te recomiendo siempre validar la entrada, no cuesta mucho trabajo y siempre es mejor mostrar un mensaje presentable al usuario que no que le salte una excepción.
En principio, usa siempre ciclos for
cundo conozcas el número de iteraciones a realizar previamente, son más eficientes y más legibles y "pitónicos" generalmente.
No uses nunca readlines
si no necesitas tener en una lista todas las líneas del fichero simultáneamente. Esto solo es necesario en algunas operaciones como ordenamientos. Usa simplemente un for
in para iterar sobre el fichero, así no cargas el archivo al completo en memoria.
La línea float(Num)
realmente no hace nada, convierte Num
en float
y retorna el objeto, como no lo asocias a ninguna variable lo más que hace es producir una excepción si laa cadena no es algo convertible a float
Si quieres convertir Num
debes reasignar: Num = float(Num)
.
Evita operaciones innecesarias y que muchas veces causan más problemas encima. Por ejemplo, no necesitas convertir la opción a entero, primero es más ineficiente que operar con la cadena directamente y además puedes causar un ValueError
si ingresa algo que no sea un entero. Deja la variable Menu
como str
y compara con if Menu == "1":
.
exit
y quit
debe usarse en principio solo en una cesión del interprete interactivo y nunca en producción. Si quieres usarlo, usa sys.exit()
en su lugar.
Después de todo el tostón que te he dado te dejo el código teniendo en cuenta lo anterior:
import sys
def menu():
while True:
print("Elige una opcion")
print("\t1 - Ingresar Datos")
print("\t2 - Leer Archivo")
print("\t3 - Salir")
opción = input("Selecciona una opcion: ")
if opción in ("1", "2", "3"):
return opción
input(("ERROR!!!\n"
"No has pulsado ninguna opción correcta\n"
"pulsa ENTER para continuar\n")
)
def ingresar_datos(ruta_archivo):
while True:
try:
cantidad = int(
input(f"Introduzca la cantidad de valores a ingresar: ")
)
except ValueError:
print("Debe ingresar un entero válido")
else:
if cantidad > 0:
break
print("Debe ingresar un entero positivo mayor que 0")
with open(ruta_archivo, "a", encoding="UTF-8") as archivo:
for n in range(1, cantidad + 1):
while True:
try:
num = float(input(f"Ingresa el {n}º número: "))
except ValueError:
print("Debe ingresar un número válido")
else:
archivo.write(f"{num}\n")
break
def leer_archivo(ruta_archivo):
with open(ruta_archivo, "r", encoding="UTF-8") as archivo:
print("-" * 74)
for linea in archivo:
print(linea.rstrip())
print("-" * 74)
def main():
ruta_archivo = input("Ingrese el nombre del archivo: ")
with open(ruta_archivo, "a", encoding="UTF-8"):
pass
while True:
opción = menu()
if opción == '1':
ingresar_datos(ruta_archivo)
elif opción == '2':
leer_archivo(ruta_archivo)
elif opción == '3':
sys.exit(0)
if __name__ == "__main__":
main()