O seu "problema" consiste do fato de que esta requisição feita pelo módulo ser assíncrona ou seja: o pedido será realizado e posteriormente você obterá uma resposta, seja ela o corpo da solicitação ou um erro que ocorreu no pedido.
Você está declarando uma variável sem um conteúdo (response
) na esperança de atribuir um valor a ela quando a resposta vier porém, esta acessando (usando) esta variável antes disto acontecer (console.log
) ... sempre será undefined
Você deve utilizar uma função de callback para trabalhar sobre este valor, exemplo:
const request = require("request");
function getWeather(callback) {
request(url, function (error, response, body) {
callback(body)
})
}
getWeather(function(responseBody) {
console.log(responseBody)
})
Por padrão este módulo não trabalha com Promise()
más, assim como descrito pelo próprio módulo você pode utilizar os pacotes adjacentes caso queira trabalhar sobre "Promessas" veja os links dos pacotes adjacentes:
https://www.npmjs.com/package/request#promises--asyncawait
Caso não queira utilizar pacotes extras saiba que podes fazer isso de maneira nativa utilizando o módulo nativo utils e "promissificando" o request:
const util = require('util')
const request = require("request");
const requestPromise = util.promisify(request);
function getWeather(url) {
return requestPromise(url).then(response => {
if ( response.statusCode === 200 ) {
return response.body
}
return Promise.reject(response.statusCode)
})
}
getWeather('https://www.google.com').then(body => {
console.log(body)
}).catch(e => console.log(e))
Como nem todos os possíveis códigos de status são um erro na requisição você deve tratá-los conforme realmente os queira, o exemplo acima apenas retornará o corpo da requisição caso o status seja igual a 200 e rejeitará todos os demais.
Promise()
por natureza são funções "assíncronas" mas você pode usar await para trabalhar sobre um comportamento "síncrono" desde que esteja dentro de uma função async:
const util = require('util')
const request = require("request");
const requestPromise = util.promisify(request);
async function done() {
let response = await requestPromise('https://www.google.com')
console.log(response.body)
}
done()
No exemplo acima não há garantias de que a resposta terá uma propriedade "body" e não poderá fazer nenhum tratamento no caso de erro/falha além de que todo o código irá "esperar" a conclusão do pedido e um servidor pode demorar muito a responder.
Uma abordagem "melhor" seria tratar as possíveis falhas na requisição em um bloco .catch()
:
const util = require('util')
const request = require("request");
const requestPromise = util.promisify(request);
async function done() {
async function getWeather(url) {
return requestPromise(url).then(response => {
if ( response.statusCode === 200 ) {
return response.body
}
return Promise.reject(response.statusCode)
}).catch(err => {
return err
})
}
console.log(await getWeather('https://www.google.com'))
}
done()
leitura:
Mozilla Docs: async/await
console.log
dentro do callback. Não é recomendado tentar remover uma variável de dentro do escopo de uma chamada callback. Você pode entender melhor nessa resposta.