then() wykonuje się zanim funkcja na której został wykonany skończy działanie js

Question

Witam, mam taką funkcję:

async function work(count){
     console.log(count)
    if (count == mainPages.length) {
      process.exit();
    }else {
      await sitesCheck(mainPages[count]).then(work(count++));
      console.log('wyswietlam sie zanim funkcja sitesCheck sie wykona :( )')
    }
  }

Problem polega na tym, że instrukcja then i console.log() poniżej wykonuje się zanim funkcja sitesCheck kończy działanie. Jak temu zapobiec?

funkcja sitesCheck:

async function sitesCheck(page){
    var site = Site;
    site.name = page
    //console.log(site.name)
    request(site.name, async function(err, response, html){
      if (!err) {
        var $ = cheerio.load(html);
        var href = $("a").each(function(){
          var a = $(this).attr('href');
          site.subPages.push(a);
        });
        //console.log(site)
        //console.log('------------------')
      var link;

      for (const item of site.subPages) {
        //console.log(item)
        if ( !(item.includes('http://') || item.includes('https://')) ) {
          //console.log('ert')
          if ( !item.includes('www')) {
            if (!item.includes('.')) {
              if ( !(item.includes('(') || item.includes(')') || item.includes(';') || item.includes(',') || item.includes('#')) ) {
                link = site.name + item;
              }else {
                continue;
              }
            }else {
              if (item.includes('.html') || item.includes('.php') || item.includes('.ejs')) {
                link = site.name + item;
              }else {
                link = 'www.' + item;
              }
            }
          }
        }else {
          link = item;
        }

        //console.log(link);

        await takeContent(link).then(val => checkWords(val, page, link))
      }
      }
      //console.log('witam was')
      return true;
    });
}

 

Answer 1

request(site.name, async function(err, response, html){

Na oko, powinieneś await-ować na request(), zamiast przekazywać callback.

W dodatku z tego co widzę request() natywnie nie wspiera promise'ów, trzeba użyć narzędzia do owinięcia go:

https://github.com/request/request#promises--asyncawait

Comments

zastosowałem twoje wskazówki jednak efekt jest ten sam. Zaktualizowany kod sitesCheck:

async function sitesCheck(page){
    var site = Site;
    site.name = page
    //console.log(site.name)
    //var rp = require('request-promise');
    await rp(site.name, function(err, response, html){
      if (!err) {
        var $ = cheerio.load(html);
        var href = $("a").each(function(){
          var a = $(this).attr('href');
          site.subPages.push(a);
        });
        //console.log(site)
        //console.log('------------------')
      var link;
 
      for (const item of site.subPages) {
        //console.log(item)
        if ( !(item.includes('http://') || item.includes('https://')) ) {
          //console.log('ert')
          if ( !item.includes('www')) {
            if (!item.includes('.')) {
              if ( !(item.includes('(') || item.includes(')') || item.includes(';') || item.includes(',') || item.includes('#')) ) {
                link = site.name + item;
              }else {
                continue;
              }
            }else {
              if (item.includes('.html') || item.includes('.php') || item.includes('.ejs')) {
                link = site.name + item;
              }else {
                link = 'www.' + item;
              }
            }
          }
        }else {
          link = item;
        }
 
        //console.log(link);
 
        await takeContent(link).then(val => checkWords(val, page, link))
      }
      }
      //console.log('witam was')
      return true;
    });
}

 

---

Powtórzę sugestię, await zamiast callbacka. Zawartość callbacka może być po prostu kodem pod awaitem.

---

ale muszę jakoś odebrać dane z requesta. Jak zrobić to bez callbacka ?

---

Pewnie tak:

let [err, response, html] = await rp(site.name);

 

---

nie działa takie przypisanie ;/

---

A, spojrzałem na dokumentację i z tego co widzę to po prostu

let html = await rp(site.name);

 

---

dzięki, działa :D