Feedback Manu #2
Description
Felicitaciones, muy buen trabajo. Te dejo algunas observaciones de mejora para que las tengas en cuenta.
- La lista de apis con las que trabajas te recomiendo que las pongas en un archivo de variables globales, ejemplo un .env o mejor aún en una carpeta config y un archivo apiEndpoints.
- Felicitaciones por trabajar con modulos, pero esta estructura no hace falta. ES6 tiene una propuesta que hace más ágil el trabajo con módulos.
- La lógica de la selección aleatoria de la broma la metería en el servicio de jokes.
- Las funcionalidades del clima las debes llevar a un servicio único para esto.
- Las respuestas a cambio de ir en la carpeta utils, las deberías meter en la carpeta config y dentro puede haber una caprtea reponses y dentro el archivo weatherResponses (emogi me gusta, pero es un nombre que determina una forma muy específica de mostrar las cosas ¿y si mañana no queremos dar emojis si no imagenes? lo re-enfocar al menos por ahora en el nombre del archivo.
- Te recomiendo que trabajes mejor en la estructura y robustez del error handler.
- Tienes todo muy bien separado, haz implementado muy bien el concepto de single resposibility para ser tu primera vez, te recomiendo que hagas test, pues con la estructura que tienes puedes probar que cada cosa funcione.
- Ya hemos aprendido qué es cada cosa en la UI, lo del DOM y Eventos los llevaremos ahora a una carpeta llamada UI.
- Jokes y Weather dentro de una carpeta services.
- la carpeta types, mejor llámala interfaces.
- Felicitaciones por la forma en la que enfocaste el proyecto. Está todo muy limpio
- Es importante aprender a hacer análisis funcional a traves de diagramas de flujo, te recomiendo algunas herramientas que te pueden ayudar: exalidraw, draw.io, figma, hoja y boli, Mermaid.