Estoy haciendo mi propia versión de un programa que he visto en un tutorial, para poner en práctica lo que he aprendido.
El programa consta de dos clases por ahora. Una clase Nota, encargada de almacenar la información: el mensaje, la fecha de creación, tags para búsquedas, etc. Y otra clase cuaderno, encargada de manejar todo lo referente a la clase Nota.
El problema que tengo es con los métodos modificar y mostrar notas, que me está resultando imposible hacerlo funcionar.
Os muestro el código:
import datetime
""" Importa la clase datetime, que permite, entre otras cosas, dar el atributo fecha de "hoy". """
class Nota:
""" Contendrá el contenido de cada nota. """
def __init__(self, mensaje, tags=""):
""" Inicializará cada vez que se instancie un objeto de la clase nota. Dará los valores pertinentes del contenido de la nota, los tags y la fecha de creación. """
self.mensaje = mensaje
self.tags = tags
self.fecha_creacion = datetime.datetime.now()
def search(self, criterio):
""" Devuelve true o false según si la nota tiene el criterio dado. """
return criterio in self.mensaje or criterio in self.tags
class Cuaderno:
def __init__(self):
""" Inicializa la variable id de la instancia cuaderno y al diccionario que contendrá el par _id : Nota()."""
self._id = 0
self.notas = {}
def mostrar_nota(self, notas=None):
""" Muestra todas las notas o las notas que se le pasen según el filtro. """
if not notas:
notas = list(self.notas)
for nota in self.notas:
if nota in notas:
""" print(f"{nota}: {self.notas[nota].values().tags}\n{self.notas[nota].values().mensaje}") """
def crear_nota(self, mensaje, tags=""):
""" Crea una nota y le añade un identificador. El contenido de la nota es obligatorio, pero los tags no lo son. """
self._id += 1
self.notas[self._id] = Nota(mensaje, tags)
print("Se ha creado una nueva nota.")
def eliminar_nota(self, id):
""" Elimina una nota con un id especificado. Si la nota no existe, muestra un mensaje. """
if self.comprobar_nota(id):
self.notas.pop(id)
print("Nota eleminada.")
def modificar_nota(self, id, mensaje="", tags=""):
""" Modifica el tag o el memo de una nota según si se ha pasado o no el valor. """
if self.comprobar_nota(id):
""" Comprueba que el id de verdad exista """
if mensaje:
self.notas[id].mensaje = mensaje
print("Mensaje modificado")
if tags:
self.notas[id].tags = tags
print("tags modificados")
def comprobar_nota(self, id):
""" Método que servirá para comprobar que el id dado exista. """
if id in self.notas:
return True
else:
print("Nota no encontrada")
return False
def buscar_nota(self, criterio):
return {nota for nota in self.notas if nota.values().search(criterio)}
Bueno, siguiendo el hilo del problema, yo intento que se muestre el contenido de self.notas. Se puede mostrar con todas las notas o con un filtro. Para pasar el filtro, asigno a una variable el resultado de comprobar_nota, clase que a su vez llama a search para comprobar si la nota indicada cumple el criterio. Luego, uso el método mostrar nota con la variable que almacena las notas que cumplían los criterios de evaluación.
Pero al ser mi self.lista un diccionario, se me está complicando bastante.
¿Alguna idea?
Muchas gracias.
Tras mucha ayuda de FJsevilla, he conseguido hacer el programa. Sinceramente, he aprendido muchísimo gracias a ti.
Dejo el código finalizado, por si alguien quiere verlo en algún futuro.
import datetime
""" Importa la clase datetime, que permite, entre otras cosas, dar el atributo fecha de "hoy". """
class Nota:
""" Contendrá el contenido de cada nota. """
def __init__(self, mensaje, tags=""):
""" Inicializará cada vez que se instancie un objeto de la clase nota. Dará los valores pertinentes del contenido de la nota, los tags y la fecha de creación. """
self.mensaje = mensaje
self.tags = tags
self.fecha_creacion = datetime.datetime.now()
def search(self, criterio):
""" Devuelve true o false según si la nota tiene el criterio dado. """
return criterio in self.mensaje or criterio in self.tags
class Cuaderno:
def __init__(self):
""" Inicializa la variable id de la instancia cuaderno y al diccionario que contendrá el par _id : Nota()."""
self._id = 0
self.notas = {}
def mostrar_nota(self, notas=None):
""" Muestra todas las notas o las notas que se le pasen según el filtro. """
if not notas:
notas = self.notas
for id_, valor in notas.items():
print(f"{id_}: {valor.tags}\n{valor.mensaje}")
def crear_nota(self, mensaje, tags=""):
""" Crea una nota y le añade un identificador. El contenido de la nota es obligatorio, pero los tags no lo son. """
self._id += 1
self.notas[self._id] = Nota(mensaje, tags)
print("Se ha creado una nueva nota.")
def eliminar_nota(self, id):
""" Elimina una nota con un id especificado. Si la nota no existe, muestra un mensaje. """
if self.comprobar_nota(id):
self.notas.pop(id)
print("Nota eleminada.")
def modificar_nota(self, id, mensaje="", tags=""):
""" Modifica el tag o el memo de una nota según si se ha pasado o no el valor. """
if self.comprobar_nota(id):
""" Comprueba que el id de verdad exista """
if mensaje:
self.notas[id].mensaje = mensaje
print("Mensaje modificado")
if tags:
self.notas[id].tags = tags
print("tags modificados")
def comprobar_nota(self, id):
""" Método que servirá para comprobar que el id dado exista. """
if id in self.notas:
return True
else:
print("Nota no encontrada")
return False
def buscar_nota(self, criterio):
return {id_: nota for id_, nota in self.notas.items() if nota.search(criterio)}
{id: nota for id, nota in self.notas.items() if nota.search(criterio)}
no obstante creo que deberías separar los criterios ahora mismo buscas criterio en el propio mensaje y en las etiquetas sin discriminar ¿mensaje y tags que tipo de dato son?¿Cadenas ambos? Las etiquetas deberían ser una lista o mejor aún un conjunto.self.notas
, cuyos valores son instancias deNota
. En tu código iteras sobreself.notas
, lo que es lo mismo que iterar sobre sus claves, por lo quenota.values()
es intentar usar el métodovalues
sobre laid
, un entero... en todo caso, puedes iterar sobre los objetos Nota usandoself.notas.values()
. Por otro lado, mencionas problemas conmodificar_nota
, también, pero no ve nada mal en ella en principio ¿cuál es el problema concreto que tienes con dicho método?