JavaScript: forでAPI通信を行っているが429 Too Many Requestsで全て表示できない

楽天APIを使用しています。
楽天APIは1pageごとに30個分しか取得できないので、
1-100位まで取得するに、ページ分の4回を取得しないといけません。
以下のようにバニラJSでコードを組んでいますが、
429 Too Many Requestsで全ての取得ができません。
何かいい方法はありませんでしょうか。
ご教授願います。

JavaScript
1for (i = 0; i < 4; i++) {
2    let n = i + 1;
3
4    function send() {
5      fetch(
6        `https://楽天API&page=${n}`
7      )
8        .then((response) => {
9          if (response.ok) {
10            return response.text();
11          } else {
12            return alert("通信失敗！");
13          }
14        })
15        .then((response) => {
16          var jsObject = JSON.parse(response);
17          console.log(jsObject);
18        });
19    }
20
21    setTimeout(send(), 1000);
22  }

行動規範の内容に同意します

回答3件

ベストアンサー

setTimeout(send(), 1000);

1秒毎に send() を呼ぶ意図なのかもしれませんが、このコードではforループは一瞬で終わって1秒後に一斉に send() が始まります。

1つのリクエストが終わったあとに1秒待つようにしましょう。

js
1async function getAll() {
2  for (let i = 0; i < 4; i++) {
3    const response = await fetch(`https://楽天API&page=${i + 1}`);
4    if (!response.ok)
5      return;
6    const text = await response.text();
7    const jsObject = JSON.parse(text);
8    console.log(jsObject);
9
10    await new Promise(resolve => { setTimeout(resolve, 1000); });
11  }
12}

投稿2022/07/22 05:26

int32_t

総合スコア20874

CatBot

2022/07/22 06:08

ありがとうございます。asyncとPromiseについて学習し理解を深めます。

行動規範の内容に同意します

setTimeoutの呼び出し方が間違っているのですぐにfuncが呼びされてしまいます。正しくはこうですsetTimeout(send, 1000);
それを直してもsetTimeoutはすぐに制御が戻り結局1秒後に4回連続sendが実行されるのでうまくいきません。
非推奨ですが、一番手っ取り早いのはsetTimeoutの1000を1000*nにすることでしょうか。setTimeout(send, 1000*n);
楽天APIの制限は1秒間に1回までだったと思うのでもっと間隔をあけたほうがいいとは思います。

投稿2022/07/22 05:32