如何使用 Promise 和 node.js 正确检查和记录 http 状态代码？

Question

我是 JavaScript 新手，对 node.js 框架非常陌生，几天前才开始使用它。如果我的代码是荒谬的，我深表歉意，承诺和回调的整个想法仍然深入人心。话虽如此，我的问题是以下我试图弄清楚对网站的某些请求是否成功或导致错误的范围他们的状态码响应。我正在处理一系列网站，到目前为止我所做的工作如下，但是我在安装了 node.js 的本地计算机上得到了TypeError: Cannot read property 'then' of undefined，但不知道为什么。

const sample = [
    'http://www.google.com/',
    'http://www.spotify.com/us/',
    'http://twitter.com/',
    'http://google.com/nothing'
]

const http = require('http')

const getStatusCodeResult = (website) => {

    http.get(website, (res) => {
        return new Promise((resolve, reject) => {
            setTimeout(() => {
                let statusCode = res.statusCode
                error = statusCode >= 400 && statusCode <= 500 ? `error: ${website}`: null
                if (error) {
                    reject(error)
                    
                } else if (statusCode >= 200 && statusCode <= 300) {
                    resolve(`Success: ${website}`)
                }
            }, 0)
        })
    })
}
// LOOP PROMISES
const getAllStatusCodeResult = (websites) => {
    websites.forEach((website) => {
        getStatusCodeResult(website)
            .then((result) => {
                console.log(result)
            })
            .catch(error => {
                console.log('error', error)
            })
    })
}
getAllStatusCodeResult(sample)

理想情况下，我希望将结果打印为下面的示例，但现在我只是使用console.log 来确定代码是否有效。

   // Example Printout
   {
      success: ['https://www.google.com/', 'https://www.spotify.com/us/', 
      'https://twitter.com /' ],
      error: [''http://google.com/nothing']
   } 

Answer 1

你混淆了前两行。为您获取返回值的 new Promise 包装器需要位于外部，而 http.get 调用应位于其执行器回调中。你也不需要那个超时：

function getStatusCodeResult(website) {
    return new Promise((resolve, reject) => {
        http.get(website, (res) => {
            let statusCode = res.statusCode,
                error = statusCode >= 400 && statusCode <= 500 ? `error: ${website}`: null
            if (error) {
                reject(error)
            } else if (statusCode >= 200 && statusCode <= 300) {
                resolve(`Success: ${website}`)
            }
        })
    })
}

Answer 2

使用util.promisify()，您可以将http.get()转换为基于promise的异步方法，但首先需要做一些准备，因为它不遵循callback(error, response) { ... }的约定：

const http = require('http')
const { promisify } = require('util')

// define a custom promisified version of `http.get()`
http.get[promisify.custom] = (options) => new Promise(resolve => {
  http.get(options, resolve)
});

// convert callback to promise
const httpGet = promisify(http.get)

async function getStatusCodeResult(website) {
  const res = await httpGet(website)
  const status = res.statusCode
  const message = `${http.STATUS_CODES[status]}: ${website}`

  if (status >= 400) {
    throw message
  } else {
    return message
  }
}

此外，您可以使用http.STATUS_CODES 为每个可能的statusCode 获取适当的消息，而不是返回模糊的Error 或Success。