发出 Axios GET 请求时出现无限循环 - 循环不会关闭答案

【问题标题】：Infinite loop when making Axios GET Request - Loop Won't Close发出 Axios GET 请求时出现无限循环 - 循环不会关闭
【发布时间】：2020-01-30 12:00:46
【问题描述】：

在使用 API 和 Promise 方面，我是一个相当大的新手 - 所以我很难解决这个问题。

目标是向 API 发出 GET 请求并返回这些结果。但是，由于 API 将数据分散到多个页面，因此我将遍历 API 页面，直到返回 0 个列表。

我能够成功地将列表记录到控制台，但是，尽管将 listingsExist 设置为 false，但 loopThroughListings 中的 while 循环似乎永远不会关闭。

我哪里做错了？

const axios = require('axios');

// Loop through multiple pages of listings in the API
function loopThroughListings() {
  let listingsExist = true;

  // Loop through listings until they don't exist
  while(listingsExist) {

    getListing().then(function(listing) {

      if(listing.length < 1 ) {
        // if listings don't exist, stop the loop
        // THIS IS WHERE THE ISSUE IS
        console.log("No Listings");
        listingsExist = false;
      } else {
        // if listings do exist, log them to console
        console.log(listing);
      }

    });

  }
}

// Return listing data from API
async function getListing(page) {
  try {
    const response = await axios.get(`https://mlb19.theshownation.com/apis/listings.json?type=Stadium&page=1}`);
    return response.data.listings;
  } catch (error) {
    console.error(error);
  }
}

loopThroughListings();

【问题讨论】：

getListing() 是异步的，因此 axios.get 永远不会完成执行，因为您实际上有一个 while(true){} 循环

标签： javascript node.js promise axios

【解决方案1】：

你需要loopThroughListings 成为async 然后你可以await getListing()

像这样：

const axios = require('axios');

// Loop through multiple pages of listings in the API
async function loopThroughListings() {
    let listingsExist = true;

    // Loop through listings until they don't exist
    while (listingsExist) {
        const listing = await getListing();
        if (listing.length < 1) {
            // if listings don't exist, stop the loop
            // THIS IS WHERE THE ISSUE IS
            console.log("No Listings");
            listingsExist = false;
        } else {
            // if listings do exist, log them to console
            console.log(listing);
        }
    }
}

// Return listing data from API
async function getListing(page) {
    try {
        const response = await axios.get(`https://mlb19.theshownation.com/apis/listings.json?type=Stadium&page=1}`);
        return response.data.listings;
    } catch (error) {
        console.error(error);
    }
}

loopThroughListings();

就个人而言，因为 while 循环当然会至少运行一次，所以我会改用 do/while - 对我来说，它使代码流更加明显

    do {
        const listing = await getListing();
        if (listing.length < 1) {
            // if listings don't exist, stop the loop
            // THIS IS WHERE THE ISSUE IS
            console.log("No Listings");
            listingsExist = false;
        } else {
            // if listings do exist, log them to console
            console.log(listing);
        }
    } while(listingsExist);

但这只是一个见仁见智的问题

【讨论】：