[Sugestão] Não é necessario converter os Ids de req.params de String para Number

module.exports = (objetoParams) => { for (let propriedade in objetoParams) { if (/Id|id/.test(propriedade)) { objetoParams[propriedade] = Number(objetoParams[propriedade]); } } return objetoParams; };

Oi, Luis! Tudo bem?

Você levantou um ponto interessante sobre a conversão de IDs de string para número e a regex utilizada. Realmente, em muitos casos, o JavaScript consegue lidar bem com strings que contêm apenas números, especialmente quando estamos apenas passando esses valores para o Sequelize, que faz a conversão internamente.

Sobre o regex, você está certo ao apontar que ele pode causar problemas se houver propriedades como "ideia". Uma maneira de melhorar a regex para garantir que ela só converta propriedades que terminam com "Id" ou "id" seria usar o símbolo $, que indica o final da string, a regex ficaria assim:

module.exports = (objetoParams) => {
  for (let propriedade in objetoParams) {
    if (/Id$|id$/.test(propriedade)) {
      objetoParams[propriedade] = Number(objetoParams[propriedade]);
    }
  }
  return objetoParams;
};

Com essa alteração, apenas propriedades que terminam com "Id" ou "id" serão convertidas, evitando problemas com propriedades como "ideia".

Obrigada por ter compartilhado esse ponto com a nossa comunidade! Espero ter contribuído!

Um forte abraço e bons estudos!