【JavaScript】ループ最初に実行回数は決まっておらず、ループ内にて次のループを実行するかどうかを判定する書き方

Question

前提

JavaScriptにてループ処理を行っています。
ループ1回目で実行回数は決まっておらず「各ループによって次のループをするかどうか」という処理を実装したいのですが、実装方法が分からず困っております。

実現したいこと

以下のような処理をループを用いて実装したいです。
必ず1回はAPIリクエストを送信し、2回目以降は「next_cursor」があったら次のループ、なかった時点でループ終了というイメージです。

該当のソースコード

const userResponse = slackAccessor.requestUserList();  // 実行①(必ず行うリクエスト)
let userInfo = userResponse.members;

// response.response_metadata.next_cursorがあるまで「requestUserList()」を繰り返す
if (userResponse.response_metadata.next_cursor) {  // 実行①で「next_cursor」がなかったら実行しない
  let response1 = slackAccessor.requestUserList(userResponse.response_metadata.next_cursor);  // 実行②(実行①でnext_cursorがあったら)
  let members = response1.members;
  userInfo = userInfo.concat(members)
  if (response1.response_metadata.next_cursor) {
    let response2 = slackAccessor.requestUserList(response1.response_metadata.next_cursor);  // 実行③(実行②でnext_cursorがあったら)
    let members = response2.members;
    userInfo = userInfo.concat(members)
    if (response2.response_metadata.next_cursor) {
      let response3 = slackAccessor.requestUserList(response2.response_metadata.next_cursor);  // 実行③(実行②でnext_cursorがあったら)
      let members = response3.members;
      userInfo = userInfo.concat(members)
      if (response3.response_metadata.next_cursor) {
        let response4 = slackAccessor.requestUserList(response3.response_metadata.next_cursor);  // 実行④(実行③でnext_cursorがあったら)
        let members = response4.members;
        userInfo = userInfo.concat(members)
      }
      // 以下「実行⑤」「実行⑥」... とnext_cursorがなくなるまで繰り返す
    }
  }
}

最後に

よろしくお願いいたします。

Answer 1

JavaScript入門書とか読んでなくて、whileやforの構文を知らないと言うことなんでしょうか？

userInfo以外の変数は後で使わない前提で。

let response = slackAccessor.requestUserList();
let userInfo = response.members;
while(response.response_metadata.next_cursor){
	response = slackAccessor.requestUserList(response.response_metadata.next_cursor);
	userInfo = userInfo.concat(response.members);
}
あるいは、
let response = slackAccessor.requestUserList();
let userInfo = [];
do{
	userInfo = userInfo.concat(response.members);
	response = slackAccessor.requestUserList(response.response_metadata.next_cursor);
}while(response.response_metadata.next_cursor);

夜中なので見間違いがあるかも。

ちゃんと入門しないでプログラムを書くのは却って効率悪いと思いますよ。

Comments

[Trilin]

回答いただきありがとうございます！
whileやforで書いていたのですが上手くいかなかったので、特別な書き方が必要なのかと思い質問させていただきました💦
精進します！

[otn]

> whileやforで書いていたのですが上手くいかなかったので、
そのうまく行かなかったというコードを載せた方が良かったですね。

Answer 2

1回目は実行するということならば、

do {
    ...
} while (response.response_metadata.next_cursor);

とか

while (true) {
    ...
   if (!response.response_metadata.next_cursor) break;
}

といった書き方をします。

Comments

[Trilin]

回答いただきありがとうございます！
do while, またwhile文の中で条件式を使うなど色々やり方あったのですね、、それぞれ試してみます！

Answer 3

無限ループになってしまう場合がないか気をつける必要はありますが、何回目かに必ず next_cursor が無い、またはfalsy な値となっているレスポンスが返されることを前提として、基本的には以下でよいかと思います。

let userInfo = [], cursor;

while (true) {
  const response = slackAccessor.requestUserList(cursor);
  
  userInfo = userInfo.concat(response.members);
  
  cursor = response.response_metadata.next_cursor;  
  if (!cursor) break;
}

参考までに、擬似的な slackAccessor を作成して、上記のロジックを試すデモを作成しました。

• 👉 https://codepen.io/su507/pen/mdLdpqv?editors=0012

このデモでは、

• members はアルファベット一文字の配列です。
• 1回のrequestUserListごとに3個の要素を含む members がレスポンスに含まれます。
• 6回目のrequestUserList が返すレスポンスのresponse_metadata には next_cursor が含まれません。
• 従って7回目のrequestUserList は呼ばれずにループからbreakします。
• 結果として、userInfo は 3個/回 × 6回= 18個 のアルファベットを含む配列になります。

※実際にこのコードを使うときは、ループの上限回数を超えたら break するなど、レスポンスがどうであっても無限ループにはならないような防御的なコード追加が必要かと思います。

Comments

[Trilin]

回答いただきありがとうございます！
また、デモの作成まで詳細にしていただきありがとうございます！！

> レスポンスがどうであっても無限ループにはならないような防御的なコード追加が必要
cursorが""になるときにループ終了させたいイメージでしたが、レスポンスがどうであれ、、というのは非常に勉強になりました！