Closed AnadeLuna closed 3 years ago
Hola Ana! 🙋🏻♂️
Te coy contando:
El código 👩💻:
En general: No has hecho los docstrings. Acostúmbrate a poner docstring diciendo qué hace cada función y qué recibe y qué devuelve; sobre todo por hacerte un favor a tu yo del futuro. Los hashtags no aparecen si haces un:
#Este comentario no saldría
def suma(a+b)
'''Recibe dos numeros
Devuelve: la suma'''
return a+b
#al hacer esto:
suma.__doc__
También súper buena modularización de las funciones: en api.py las funciones mínimas con los decoradores que llamen a funciones definidas en otros sitios, como mongoconnection 👏
Respecto a la parametrización:
def delete_user(name,age,sex,lookingfor):
dic = {"Name":f"{name}",
"Age":f"{age}",
"Sex":f"{sex}",
"Looking for":f"{lookingfor}"}
return users.remove(dic)
#Funtion to delete a message.
def delete_message(name,messa,match):
dic_mes = {"Name":f"{name}",
"Message":f"{messa}",
"Match":f"{match}"}
return message.remove(dic_mes)
si las dos funciones lo que hacen es eliminar cosas y depende de la colección de donde se eliminen, puedes parametrizar la función para que en función de alrgumento haga una cosa o haga otra. Al final tener funciones muy parecidas puede ser señal de que hace falta refactorizar.
Como decíamos ayer, puedes incluir el sentiment analysis dentro de un endpoint. Es cuestión de modularizar las funciones y poner un decorador más. Puedes incluso hacer que, en función de lo que pidas, devuelvas una gráfica. También puedes hacerlo por grupos, por usuarios, por usuarios en el tiempo (antes o después de un evento), por palabras más usadas, etc.
Enhorabuena Ana! Has creado una base de datos, has creado enpdpoints de post, de get y has hecho un sentiment analysis. A este proyecto siempre se le pueden añadir más cosas: hacer más visualización, meter más datos, puedes deployearlo (para eso necesitas pasar la base de datos de Mongo Compass a Mongo Atlas), pero el proyecto está genial. En unas semanas habéis conseguido nutrir una base de datos a través de una aplicación vuestra y habéis sacado insights. Y esto es sólo el principio, esto lo puedes escalar de muchas formas.
A tope Ana!! 💪 🚀
https://github.com/AnadeLuna/Proyect-Api