我无法让我的 Promise 正确拒绝一个新的错误消息，从 try/catch 中捕获

Question

我有一个 Promise 方法来解析来自网络的链接。它返回一个我尝试从中访问link 键的对象，但是当这个对象为空时，它会以某种方式跳过if 我必须检查它的长度，从而导致一个可耻的错误。在代码下方。

首先，解析链接的Promise方法：

* parseReporter() {
  const article = new Promise((resolve, reject) => {
    parser.parseURL(`https://www.google.com/alerts/feeds/${this.googleAlertsUrlId}/${this.reporterUrlId}`, (err, parsed) => {
      if (err) {
        reject(new Error(err))
      }

      if (parsed.feed.entries.length === 0) {
        reject(new Error('Nothing to parse'))
      }

      const link = parsed.feed.entries[0].link
      const betterLink = link.substring(42, link.indexOf('&ct='))

      mercury.parse(betterLink).then((data) => {
        resolve(data)
      }).catch((err) => {
        reject(new Error(err))
      })
    })
  })

  return article
}

然后，这里是调用这个parseReporter()的方法：

* _getLastestNews(userReporter) {
  const reportersOperation = new ReportersOperation()

  reportersOperation.googleAlertsUrlId = userReporter.url.split('/')[3]
  reportersOperation.reporterUrlId = userReporter.url.split('/')[4]

  try {
    return yield reportersOperation.parseReporter()
  } catch (e) {
    this.addError(HTTPResponse.STATUS_INTERNAL_SERVER_ERROR, e.message)

    return false
  }
}

当它尝试从parsed.feed.entries[0] 访问link 时导致该错误。我已经注销了length，我确认要工作并显示一个号码，但它坚持跳过它。我对 Promise it try/catch 自己做错了吗？

Answer 1

reject 不会从像 return 这样的函数“停止”或“返回”

因此，您的代码正在检查错误情况，但继续进行，就好像数据正常

通过在调用reject 之前添加return，您将阻止这种情况的发生

仅显示更改的代码区域：

      // snip
      if (err) {
        return reject(new Error(err))
      }

      if (parsed.feed.entries.length === 0) {
        return reject(new Error('Nothing to parse'))
      }

      const link = parsed.feed.entries[0].link
      const betterLink = link.substring(42, link.indexOf('&ct='))
      //snip

Answer 2

此外，Jaramonda 建议，当你有一个承诺并且你确实在两条路径中解决和拒绝时，你也在使用反模式。你可以更有效地做到这一点：

  resolve(mercury.parse(betterLink));

但是，你真正应该做的是你应该承诺parser.parseURL()，这样你就可以使用承诺来编写所有的控制流逻辑。这更加万无一失，并创建了一个可重用的接口，该接口使用您可以在其他地方使用的 Promise：

// make promisified version of parser.parseURL()
parser.parseURLP = function (url) {
    return new Promise((resolve, reject) => {
        parser.parseURL(url, (err, parsed) => {
            if (err) return reject(new Error(err));
            resolve(parsed);
        });
    });
};

function parseReporter() {
    return parser.parseURL(`https://www.google.com/alerts/feeds/${this.googleAlertsUrlId}/${this.reporterUrlId}`).then(parsed => {
        if (parsed.feed.entries.length === 0) {
            throw new Error('Nothing to parse');
        }

        const link = parsed.feed.entries[0].link
        const betterLink = link.substring(42, link.indexOf('&ct='))

        return mercury.parse(betterLink).catch(err => {
            // wrap error in Error object
            throw new Error(err);
        })
    })
}