Как запустить коллбек после асинхронных функций?

Question

[Mike]

Друзья, подскажите пожалуйста, как запустить коллбек функцию после того как полностью соберется объект root. Слышал, что нужно использовать Promises, но ума не приложу, как это сделать, когда у меня вложенные асинхронные функции. Вот кусок кода, где собирается объект root:

let regexp_title = /<title>(.*?)<\/title>/i
let regexp_chapter = /<meta chapter="(.*?)" order="([0-9]+)">/i
let root = {
  name: 'Global',
  file: null,
  children: []
};

fs.readdir(dataFolder, (err, fileNames) => {
  for (var i = 0, len = fileNames.length; i < len; i++) {
    if(path.extname(fileNames[i]) === '.html'){
      fs.readFile(dataFolder+fileNames[i], (err, data) => {
        let dataChapter = regexp_chapter.exec(data)[1];
        if(dataChapter.indexOf('\\') !== -1){
          let chapters = dataChapter.split('\\');
          let parentLeaf = null;
          for(var j=0, jlen = chapters.length; j < jlen; j++){
            if(parentLeaf === null) {
              if(!objTraverse.findFirst(root, 'children', {name:chapters[j]})) {
                root['children'].push({
                  name:chapters[j],
                  children:[],
                  file:null
                });
              }
            } else {
              if(!objTraverse.findFirst(root, 'children', {name:chapters[j]})) {
                if(objRef = objTraverse.findFirst(root, 'children', {name:parentLeaf})) {
                  objRef['children'].push({
                    name:chapters[j],
                    children:[],
                    file:null
                  });
                }
              }
            }
            parentLeaf = chapters[j];
          }
        }
      })
    }
  }
})

Answer 1

[someart]

Вот это ваше называется callback hell'ом. Поэтому все таки стоить выделить время и почитать о промисах. Вот тут очень годно расписано все.

Comments

[Mike]

Как добавить 1 promise для readdir в принципе понятно, а вот как вложенный сделать к тому же там readFile в цикле сидит, не догоняю...

[Виталий]

dillix: https://nodejs.org/api/fs.html#fs_fs_readfilesync_... ну почитайте вы перед работой доку а...

[someart]

Есть promises chaining, в той статье все описано

[Mike]

Виталий: я вот первый раз с нодой столкнулся, пишу десктопное приложение на Electron, с JS раньше работал, и честно говоря не сразу понял, что грабли у меня были из-за асинхронности... Простые примеры нагуглил, но вот в асинхоне еще и цикл с асинхронной функцией: видимо надо какой-то пул сделать из promise + какую библиотеку лучше взять?

[someart]

Можно просто обойтись обычными промисами, в ES6 они довольно хороши, библиотек не нужно

[Mike]

Переписал код используя нативные промисы, попутно отрефакторил. Посмотрите правильно ли? И еще вопрос, можно ли теперь const tree вынести за пределы промиса, чтобы к ней имели доступ другие функции?

let root = {
  name: 'Global',
  file: null,
  children: []
};

function ipGetTree(folder) {
  return new Promise(function(resolve, reject) {
    fs.readdir(dataFolder, (err, files) => {
      files = files.filter(function(file) {
          return path.extname(file) === '.html';
      });

      var count = files.length;
      files.forEach(function(file) {
        fs.readFile(dataFolder+file, 'utf-8', function(err, contents) {
          if(contents.indexOf('meta content="_index"') !== -1){
            // Front pages

          } else {
            // All other pages
            let dataChapter = regexp_chapter.exec(contents)[1];
            if(dataChapter.indexOf('\\') !== -1) {
              let chapters = dataChapter.split('\\');
              let parentLeaf = null;
              for(var i=0, ilen = chapters.length; i < ilen; i++) {
                if(parentLeaf === null) {
                  if(!objTraverse.findFirst(root, 'children', {name:chapters[i]})) {
                    root['children'].push({
                      name:chapters[i],
                      children:[],
                      file:dataChapter
                    });
                  }
                } else {
                  if(!objTraverse.findFirst(root, 'children', {name:chapters[i]})) {
                    if(objRef = objTraverse.findFirst(root, 'children', {name:parentLeaf})) {
                      objRef['children'].push({
                        name:chapters[i],
                        children:[],
                        file:dataChapter
                      });
                    }
                  }
                }
                parentLeaf = chapters[i];
              }
            }
          }

          count--;
          if (count === 0) {
            resolve(root);
          }
        });
      });
    });
  });
}

ipGetTree(dataFolder).then(function(root) {
  const tree = require('electron-tree-view')({
    root,
    container: document.querySelector('.sidebar'),
    children: c => c.children,
    label: c => c.name
  })
});

Answer 2

[de1m]

В данном случае они не очень сильно помогут, там будут такие же вложения, может быть немого более читаемо.
Я похожее делал через async/await, но там тоже надо было дождаться выполнения предыдущей команды, обработать результат и передать дальше. Тоже что-то такое получается, посмотрите вот тут с 180 строки