Tu principal problema puede ser que self.contactos = []
luego ya irás guardando cosas en la lista.
Tu ejercicio puede quedar algo así:
class Agenda:
def __init__(self):
self.listin = []
def add_contacto(self):
"""Añadir Contacto"""
contacto = {'Nombre': '', 'Telefono': '', 'Email': ''}
for campo in contacto:
contacto[campo] = input( f'{campo}: ')
op = input('Cargar contacto (s/n):\t')
if op == 's':
self.listin.append(contacto)
print('Contacto guardado')
def lista_contactos(self):
""" Lista contactos"""
for contacto in self.listin:
for campo in contacto:
print(contacto[campo], end = '\t')
print()
ag = Agenda()
for _ in range(3):
ag.add_contacto()
ag.lista_contactos()
En el __init__
simplemente creas una lista.
En la función add_contacto
puedes crear el diccionario con los datos que quieres de cada contacto.
Puedes ir recorriéndolos con un for
, así podrás ampliar los datos que quieres simplemente añadiéndolos al diccionario.
Para leer los datos, igual que para cargarlos, no necesitas saber los datos que hay en el diccionario:
for contacto in lista:
for campo in contacto:
print(contacto[campo]
Intentando hacer ese ejercicio ampliándolo un poco (tras varias versiones) me quedó algo parecido a esto:
@dataclass
class Agenda:
listin: list = field(default_factory = list, init = False)
contacto: dict = field(default_factory = dict, init = False)
nombre: str = ''
telef: str = ''
email: str = ''
nom_fich: str = 'agenda.json'
def __post_init__(self):
self.contacto = {'nombre': '', 'telef': '', 'email': ''}
@staticmethod
def __crea_menu(texto):
titulo = '[ - - - - - - - - - - ]\n' * 2 + \
texto + '\n[ - - - - - - - - - - ]' * 2
return titulo
def add_contacto( self ):
print(self.__crea_menu('Añadir Contacto'))
for dato in self.contacto:
self.contacto[dato] = input(dato + ':\t')
print(self.contacto)
if 's' == input('Guardar contacto (s/n)').lower():
self.listin.append(self.contacto)
def lista_contactos( self ):
print(self.__crea_menu('Lista Contactos'))
for cont in self.listin:
for dato in cont:
print(cont[dato], end = '\t')
print()
print(f'Hay {len(self.listin)} contactos')
self.contactos
a[]
. Tal como lo tienes contendría al principio un diccionario vacío que es el que te está dando problemas. Por otro lado para iterar por un diccionario puedes usarfor key in contacto
. No es buena práctica especificar "manualmente" las claves. Las sacas con unfor
. Incluso puedes sacar clave y valor directamente confor key, value in contacto.items(): print(f'{key}: {value}')