Promise 既解决又拒绝

Question

好像我的 Promise 同时返回 true 和 false。控制台正在返回 "undefined"，然后在其正下方返回 "something went wrong"。数据在这些下面返回，表明它实际上并没有在等待 Promise。

调用的函数如下：

module.exports = (url) => {
  return new Promise((resolve, reject) => {
    axios({
      method: 'get',
      url: url
    })
      .then(response => {
        const html = response.data
        const $ = cheerio.load(html)
        const songtable = $('.chart-list__elements > li')
        const topsongs = []
        songtable.each(function () {
          const rank = $(this).find('.chart-element__rank__number').text()
          if (rank == 11) return false;
          const name = $(this).find('.chart-element__information__song').text()
          const artist = $(this).find('.chart-element__information__artist').text()

          topsongs.push({
            rank,
            name,
            artist
          })
        })
        resolve()
        return topsongs;
      })
      .catch(reject("something went wrong"))
    })
}

来自来电者：

componentDidMount() {
    const top_songs = topsongs('https://www.billboard.com/charts/hot-100')
    .then(console.log(top_songs))
    .catch(err => console.log(err))
  }

谢谢，我是 Promises 的新手，几乎尝试了所有方法。尽管有异步 axios() 调用，但我有一个 Promise 的原因是它没有被异步执行并返回未定义的数据。

Answer 1

.catch(reject("something went wrong"))

您需要将函数传递给 catch。

您正在立即调用reject并传递其return值。

您还使用了嵌套承诺反模式。

axiosreturn是一个承诺。没有必要再创建一个。

module.exports = (url) =>
  axios({
    method: "get",
    url: url,
  })
    .then((response) => {
      const html = response.data;
      const $ = cheerio.load(html);
      const songtable = $(".chart-list__elements > li");
      const topsongs = [];
      songtable.each(function () {
        const rank = $(this).find(".chart-element__rank__number").text();
        if (rank == 11) return false;
        const name = $(this).find(".chart-element__information__song").text();
        const artist = $(this)
          .find(".chart-element__information__artist")
          .text();
        topsongs.push({
          rank,
          name,
          artist,
        });
      });
      return topsongs;
    })
    .catch(() => {throw "something went wrong"});

（用通用的 "something went wrong" 替换抛出的错误似乎没有帮助。如果没有 catch 调用你可能会更好）

Answer 2

你已经有一个承诺，就return吧。

  return axios({
      method: 'get',
      url: url
    })
      .then(response => {
        const html = response.data
        const $ = cheerio.load(html)
        const songtable = $('.chart-list__elements > li')
        const topsongs = []
        songtable.each(function () {
          const rank = $(this).find('.chart-element__rank__number').text()
          if (rank == 11) return false;
          const name = $(this).find('.chart-element__information__song').text()
          const artist = $(this).find('.chart-element__information__artist').text()

          topsongs.push({
            rank,
            name,
            artist
          })
        })
        return topsongs;
      })

并且仅针对 "syntactic sugar"，async/await 使所有内容更易于阅读：

module.exports = async (url) => {
   const { data } = await axios({method:'get',url});
   const $ = cheerio.load(data);

   ...

   return topsongs;
}

Promise 既解决又拒绝

Promise both resolves and rejects

javascript

promise

reactjs

es6-promise