Estoy aprendiendo Python
y he estado intentando hacer el siguiente ejercicio.
Escribir un programa que permita gestionar la base de datos de clientes de una empresa.
Los clientes se guardarán en un diccionario en el que la clave de cada cliente será su NIF y el valor será otro diccionario con los datos del cliente (Nombre, dirección, teléfono, correo, preferente), donde preferente tendrá el valor True
si se trata de un cliente preferente.
El programa debe preguntar al usuario por una opción del siguiente menú:
(1) Añadir cliente
(2) Eliminar cliente
(3) Mostrar cliente
(4) Listar todos los clientes
(5) Listar clientes preferentes
(6) Terminar
En función de la opción elegida el programa tendrá que hacer lo siguiente:
- Preguntar los datos del cliente, crear un diccionario con los datos y añadirlo a la base de datos.
- Preguntar por el NIF del cliente y eliminar sus datos de la base de datos.
- Preguntar por el NIF del cliente y mostrar sus datos.
- Mostrar lista de todos los clientes de la base datos con su NIF y nombre.
- Mostrar la lista de clientes preferentes de la base de datos con su NIF y nombre.
- Terminar el programa.
Para hacer mi código más legible y por lo tanto mas limpio, he decidido que cada acción que realice el programa corresponda con una función y luego llamar a éstas cuando sea necesario.
Y quedó tal que así;
clientes = dict()
def joinClientes(diccionario):
c1 = str(input('Introduzca el nif con el siguiente formato (########-letra):'))
nombre = str(input('Introduzca un nombre:'))
direccion = str(input('Introduzca una dirección;'))
telefono = int(input('Introduzca un teléfono:'))
correo = input('Introduzca un correo')
preferente = False
p = str(input('Indique si es un cliente preferente(si/no)')); p = p.lower()
if p == 'SI':
preferente = True
v1 = dict(nombre = nombre,direccion=direccion,telefono=telefono,correo=correo,preferente=preferente,)
diccionario[c1]=v1
def eliminarCliente(diccionario):
x = int(input('Introduzca el nif del cliente a eliminar'))
diccionario.pop(x,0)
def mostrarCliente(diccionario):
x = int(input('Indique el nif del cliente que desea mostrar:'))
if x not in diccionario:
print('¡El nif del cliente no existe!')
else:
print(diccionario[x])
def mostrarPreferentes(diccionario):
for x in diccionario:
if diccionario[x]['preferente'] == True:
print(x,diccionario[x]['nombre'])
#Ejecución del programa.
while True:
print('1-Añadir cliente| 2- Eliminar cliente | 3-mostrar cliente | 4-listar todos los clientes | 5 listar clientes preferentes | 6 Terminar')
option = int(input('Indique con números la operación que desea realizar:'))
if option == 1:
joinClientes(clientes)
elif option == 2:
eliminarCliente(clientes)#no funciona
elif option == 3:
mostrarCliente(clientes)
elif option == 4:
for x in clientes:
print(x,clientes[x]['nombre'])
elif option == 5:
mostrarPreferentes(clientes)
elif option == 6:
print('fin programa')
break
La función joinClientes() actúa sobre la variable(diccionario) añadiendo un nuevo cliente. Pero las funciones mostrarCliente() | eliminarCliente() | mostrarPreferentes(). No optengo de ellas ningún comportamiento al llamarlas, salvo que haya pre-cargado los datos en la variable antes de entrar en el bloque while True:
Este comportamiento me tiene confundido ya que si funciona la función joinClientes() el resto de funciones deberían modificar o actuar sobre la variable(diccionario) perfectamente. Si alguien pudiese ayudarme lo agradecería muchísimo.
Saludos.
joinclientes
debe devolver el diccionario. Sino pierdes lo que hayas modificado. Este es un caso en el que seria mejor que usaras clases. De esta forma puedes tener el diccionario como atributo, acceder a el como self.diccionario y asi tus functiones (metodos) no tienen que devolver nada. Echale un vistazo a un tutorial de clases e implementa tu codigo como una clase