async/await が上手く動作しない

Question

やっていること

requestとcheerioを使ってWebサイトから特定のデータを抽出するコードを書いています。

困っていること

理想の出力は下記の通りです。

start
url1
url2
url3
[url1, url2, url3]

実際の出力は下記のようになってしまいます。

start
[]
url1
url2
url3

awaitを待って配列を出力したい所で、先にconsole.log(urls)が呼ばれてしまっているようです。

コード

export const someFunction = async () => {
  console.log('start');
  let urls: string[] = [];
  try {
    await request.get(
      "https://example.com",
      async (error, response, data) => {
        const $ = await cheerio.load(data);
        $(".some-class").each((index, item) => {
          const url = $(item).attr("href");
          console.log(url);
          urls.push(url);
        });
      }
    );
  } catch (e) {
    console.log(e);
  }
  console.log(urls);
};

eachを使っている部分が影響しているのでしょうか。
色々と調査してみましたが原因が分からず、考えられる理由があれば教えていただけないでしょうか。
よろしくお願いいたします。

Answer 1

これでどうやろか

export const someFunction = async () => {
  console.log('start');
  let urls: string[] = [];
  try {
    await Promise((res, rej) => {
      request.get(
        "https://example.com",
        async (error, response, data) => {
          const $ = await cheerio.load(data);

        
          $(".some-class").each((index, item) => {
            const url = $(item).attr("href");
            console.log(url);
            urls.push(url);
          });

          res()
        }
      );  
    })
  } catch (e) {
    console.log(e);
  }
  console.log(urls);
};

Answer 2

原因はawait request.getの外側に
console.log(urls)が記述されているからです。

以下のコードのようにawait request内に記述すれば
想定どおりに動作します。

const request = require('request')
const cheerio = require('cheerio')
const url = 'https://eiga.com/now/all/rank/'

const someFunction = async() => {
    console.log('start')
    const urls = []
    await request(url,
        async (e, response, body) => {
        try {
            const $ = cheerio.load(body)
            $('.unitBox_s strong > a').each((i, elem) => {
                const url = $(elem).attr('href')
                urls[i] = url
                console.log(url);
            })
            console.log(urls)
         } catch (e) {
             console.error(e)
         }
    })
}
someFunction()

Answer 3

とりあえずresolveかrejectを返すようにしてみてはどうでしょうか
awaitの例を参照

Comments

[hentaiman]

cheerioを調べてみたけど、awaitとかasyncとかを書かなければデフォルトでsyncっぽい動きをしてくれる作りになってるように見えます