Revision

[smanosalvas]

Observaciones y preguntas:

• Buen trabajo con el readme, se podría incluir alguna información sobre las API del proyecto y como montarlo
• Veo que implementaron los controladores pero no los enlazaron a las rutas y servicios, estaria genial completar esa parte

[alfred951]

Para expandir un poco lo que menciona mi compañero en src/routes/card.js ustedes tienen la siguiente logica:

router.get('/', async (req, res) => {
  try {
    const Cards = await cardservice.getCards()
    res.json(Cards)
  } catch (err) {
    console.error(err)
    res.status(500).send('Internal server Error')
  }
});

Este código realiza tres funciones:

1. Manejar la ruta /
2. Convertir la respuesta al formato Json
3. Manejar los errores con el bloque try/catch

Aunque esto no es incorrecto y la sintaxis es buena, un enrutador no debería tener todas estas responsabilidades.

Si miramos el archivo src/controllers/card.js:

const getCards = async (req, res) => {
    try {
        const cards = await cardService.getCards();
        res.json(cards);
    } catch (error) {
        res.status(500).json({ error: error.message });
    }
};

Vemos que este ya realizaba 2 de las 3 resonsabilidades:

1. Convertir la respuesta al formato Json
2. Manejar los errores con el bloque try/catch

Lo que nos permitira dejar el router mucho mas simple y limpio con una sola funcionalidad:

router.get('/', cardController.getCards);