Soy nuevo en nodeJS y quiero mostrar el cuerpo del resultado de mi solicitud al navegador usando express, pero mi código arroja un error como este, he intentado usar array.push () pero no funciona demasiado

ERROR

node:_http_outgoing:576
    throw new ERR_HTTP_HEADERS_SENT('set');
    ^

Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
    at new NodeError (node:internal/errors:371:5)
    at ServerResponse.setHeader (node:_http_outgoing:576:11)
    at ServerResponse.header (C:\xampp\htdocs\#masgalih\tt\node_modules\express\lib\response.js:771:10)
    at ServerResponse.send (C:\xampp\htdocs\#masgalih\tt\node_modules\express\lib\response.js:170:12)
    at ServerResponse.json (C:\xampp\htdocs\#masgalih\tt\node_modules\express\lib\response.js:267:15)
    at ServerResponse.send (C:\xampp\htdocs\#masgalih\tt\node_modules\express\lib\response.js:158:21)
    at Request._callback (C:\xampp\htdocs\#masgalih\tt\index.js:12:24)
    at Request.self.callback (C:\xampp\htdocs\#masgalih\tt\node_modules\request\request.js:185:22)
    at Request.emit (node:events:390:28)
    at Request.<anonymous> (C:\xampp\htdocs\#masgalih\tt\node_modules\request\request.js:1154:10) {
  code: 'ERR_HTTP_HEADERS_SENT'
}

CÓDIGO

const request = require('request')
const express = require('express')
const app = express()

app.get('/getdata', (req, res) => {
    if (req.query.id !== undefined && req.query.id !== '') {
        request('http://localhost/myApi/index.php?id=' + req.query.id, (err, response, body) => {
            return res.send({
                status: 200,
                data: body
            })
        })
    }

    return res.send({
        status: 400,
        msg: 'Parameter invalid'
    })
})

app.listen(2000)
3
Galih Sukristyan Saputra 27 nov. 2021 a las 07:47
2
¿Echas de menos un "más"?
 – 
John
27 nov. 2021 a las 07:56

1 respuesta

La mejor respuesta

El app.get('/getdata'); tiene dos res.send(); diferentes. Esto en sí mismo está bien, pero lo que está sucediendo es que cuando se ejecuta la función en app.get('/getdata');, primero verifica esa instrucción if. Si el if es falso, pasa por alto y todo funciona bien.

Pero, ¿qué pasa si la declaración if es verdadera? Bueno, el código dentro de la declaración se ejecuta y envía un request a la URL especificada y espera una respuesta. Sin embargo, mientras espera, JavaScript sigue ejecutando su código porque JavaScript es asincrónico .

Entonces se ejecuta el segundo res.send(), pero luego se recibe la respuesta al request y, por lo tanto, se envía un segundo (el primero en orden cronológico) res.send(), que Es por eso que el error dice que está tratando de establecer encabezados (la información principal en una solicitud de nodejs) después de enviarla al cliente.

Lo que debería hacer el código es, en cambio, ejecutarse solo en el otro, no en ambos. Podemos lograr esto poniendo el segundo res.send() en un else, lo que significa que solo se ejecuta si la declaración if es falsa. Entonces algo como esto:

const request = require('request')
const express = require('express')
const app = express()

app.get('/getdata', (req, res) => {
    if (req.query.id !== undefined && req.query.id !== '') {
        request('http://localhost/myApi/index.php?id=' + req.query.id, (err, response, body) => {
            return res.send({
                status: 200,
                data: body
            })
        })
    } else {

        return res.send({
            status: 400,
            msg: 'Parameter invalid'
        })
    }
})

app.listen(2000)
2
charlie-map 27 nov. 2021 a las 08:03
Ohh, ya veo, soy un novato en nodejs, creo que el código se ejecuta igual que PHP
 – 
Galih Sukristyan Saputra
27 nov. 2021 a las 08:23