1

Estou fazendo um tratamento de erro que se for passado um id errado pela url ele retorna um erro 400, mas quando eu faço o teste ele retorna um erro 200 mais uma mensagem

"error": "Error", "message": "Cast to ObjectId failed for value \"5d69f4aa827b491c678c50a\" at path \"_id\" for model \"Product\""

Segue o controller:

router.get('/product-view/:id', async (req, res) => {    
    const id = req.params.id    
    try{    
        if(!id) return res.status(404).send({ error: 'Produto não encontrado'})    
        console.log(id);        
        const product = await Product.findById(id);        
        res.send(product);        

    }catch(error){    
       res.send({    
            error: 'Error',
            message: error.message    
       })    
    }       
});
3
  • Product.findById pelo visto o erro é aqui ... você usa qual banco de dados?
    – novic
    Commented 28/09/2019 às 12:42
  • eu uso o mongo DB Commented 28/09/2019 às 12:49
  • Status 200 não é de erro, mas sim de sucesso, tu precisa chamar o método status com o código de retorno. Ex. res.status(400).send({error: 'Error', message: error.message}) Commented 28/09/2019 às 14:02

1 Resposta 1

1

Problema: o id recuperado é um tipo diferente que o método findById entende, no caso esse método precisa de um ObjectId (mongoose.Schema.Types.ObjectId) para buscar a informação no banco , ou um objeto com a seguinte estrutura {_id: id} ou uma string com o nome _id, como está descrito na sua documentação.

Particularmente falando eu prefiro sempre colocar um tipo que no caso é ObjectId, exemplo:

router.get('/product-view/:id', async (req, res) => {    
    const id = req.params.id    
    try{    
        if(!id) return res.status(404).send({ error: 'Produto não encontrado'})    
        const _id = new ObjectId(id);  // instância requerida pelo seu código
        const product = await Product.findById({_id:_id}); // ou simplificando {_id}        
        res.send(product);        

    }catch(error){    
       res.send({    
            error: 'Error',
            message: error.message    
       })    
    }       
});

e esse tipo ObjectId está mongoose.Types.ObjectId que é tipo de dado gravado no seu .

ou solução rápida no seu código:

router.get('/product-view/:id', async (req, res) => {    
    const id = req.params.id    
    try{    
        if(!id) return res.status(404).send({ error: 'Produto não encontrado'})      
        const product = await Product.findById({_id: id});        
        res.send(product);        

    }catch(error){    
       res.send({    
            error: 'Error',
            message: error.message    
       })    
    }       
});

Referente:

Exemplos

2
  • Então usei o primeiro exemplo que voce colocou mas esta dando esse erro : ObjectId is not defined Commented 28/09/2019 às 13:34
  • Você precisa pegar o caminho completo, ai não é erro no meu código, e você precisa colocar a referencia correta, se está difícil faça a segunda opção, as duas estão corretas @EdgarSilva
    – novic
    Commented 28/09/2019 às 13:36

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .