Async: «Превышен максимальный размер стека вызовов» с использованием async.forEachLimit

forEachLimit выглядит неправильно.

started += 1 running += 1

следует переместить перед вызовом итератора. arr[started] также следует заменить на arr[started - 1]

Примечание. Вы столкнетесь с этой ошибкой только в том случае, если код внутри итератора iterator вызывает обратный вызов в том же такте.

ПРЕКРАТИТЬ вызов функций SYNC в async.js

Я столкнулся с этой проблемой, когда у меня было условное return callback(null); в моем итераторе. Это означало, что мне нужно было сделать более умную вещь, а именно отфильтровать массив с помощью Array.filter перед применением async.forEach , так что это было действительно полезно...

Если вы _действительно_ считаете, что должны, вы можете обойти это, используя process.nextTick(function() { callback(); }) .

@jacobrask Это сработало хорошо. Спасибо

Выполнение callback(null); синхронно. Вам нужно вызывать его асинхронно, аля process.nextTick(callback) или process.nextTick(function() { callback(null); }); . Как сказал @bobrik , прекратите вызывать функции синхронизации в асинхронном режиме :)

Сказать «прекратить вызов функций синхронизации в асинхронном режиме» не хватает смысла — он не работает, и только в forEachLimit это кажется проблемой.

async.forEachLimit(['a','b'], 1, function(item, callback){
  if(something) {
    doStuff(function(){ callback(); });
  } else {
    callback();
  }
});

Хотя я могу довольно легко защититься от этого очень конкретного примера (как упоминает @jacobrask с помощью фильтрации), что, если doStuff() (который может находиться в библиотеке вне моего контроля) решит вызвать мой обратный вызов без process.nextTick()?

Я добавил некоторый код для вызова async.nextTick, если он обнаруживает синхронный итератор, который в этих случаях должен избегать переполнения стека. А если серьезно, ПРЕКРАТИТЕ ВЫЗОВАТЬ ФУНКЦИИ СИНХРОНИЗАЦИИ В ASYNC! ... вместо этого сделайте свои функции последовательно синхронными или последовательно асинхронными;)

Это также произошло для меня с async.eachSeries .

Также происходит в mapLimit , и да, моя функция синхронизируется, потому что это фиктивная функция. Затем я оборачиваю его в setImmediate , проблема решена.

Случись со мной на async.forEachOfLimit

Столкнулся с той же проблемой с async.waterfall .

async.queue тоже, конечно.

Я выполнял операцию синхронизации. Я исправил это, изменив с:

callback();

к

setTimeout(callback, 0);

Это случилось со мной, когда к обещанию был привязан улов. Что-то типа:
async.forEach(items, (item:any, callback:Function) => { someAsyncPromise(item).then(() => { callback(); }).catch((err) => { //console error or something here callback(); }) });

catch должен быть выбран перед асинхронным битом в обещании, потому что эта ошибка возникает только тогда, когда в обещании возникает исключение.

`

Я понимаю, что мы должны прекратить вызывать функции синхронизации в асинхронном режиме.

Но если вызов функции синхронизации callback() внутри асинхронного тела неверен, то почему документация по асинхронности показывает это в примерах?

Может ли кто-нибудь помочь мне понять, как правильно работать с асинхронным модулем?

Вот пример непосредственно из асинхронной документации:

// assuming openFiles is an array of file names
async.each(openFiles, function(file, callback) {

    // Perform operation on file here.
    console.log('Processing file ' + file);

    if( file.length > 32 ) {
      console.log('This file name is too long');
      callback('File name too long');
    } else {
      // Do work to process file here
      console.log('File processed');
      callback();
    }
}, function(err) {
    // if any of the file processing produced an error, err would equal that error
    if( err ) {
      // One of the iterations produced an error.
      // All processing will now stop.
      console.log('A file failed to process');
    } else {
      console.log('All files have been processed successfully');
    }
});

https://caolan.github.io/async/docs.html#each

Спасибо!