Map не дожидается выполнения await?

Question

[waltaki]

Здравствуйте.

await Promise.all(
  resourcesList.map(
    async (e: ResourceBodyEntities): Promise<ResourceBodyEntities> => {
      console.log(e.id, "-");
      e.versions = await page.getResource(e);
      console.log(e.id, "+");
      return e;
    }
  )
);

1 -
2 -
3 -
2 +
3 +
1 +

Как быть? map не дожидается выполнения await, и летит дальше.

Comments

[vetsmen]

У тебя Promise.all параллельно выполняет все промисы, которые возвращает map, ответ вполне нормальный. Сначала сработает первый console.log у всех промисов, а второй уже в зависимости от скорости выполнения вложенного await.

[Роман]

Погуглил за вас

ЗЫ:

resourcesList.map(
    async ...

Тут не верное понимание работы map. Сам по себе map не обращает внимание на то является ли функция-аргумент синхронной или асинхронной и не ждет выполнения возвращаемого ей промиса. Как сделать это правильно смотрите по ссылке выше.

ЗЫЗЫ: Вот обобщенное описание того, что происходит в вашем коде:
map последовательно для каждого item из resourcesList запускает функцию-обработчик, которая возвращает промис и запускает на выполнение функцию-аргумент промиса, которая в свою очередь выводит "-" и доходя до await page.getResource(e); встает на "паузу", ожидая результата выполнения запроса page.getResource, передавая тем самым управление следующей задаче в eventloop, которая в вашем случае является функцией-обработчиком для следующей итерации resourcesList. Таким образом map проскакивает все итемы в resourcesList возвращая массив промисов для Promise.all (и выведя в консоль кучу "-"). Далее начинают подтягиваться ожидающие выполнения запросы page.getResource и функции стоящие на паузе размораживаются, выводя в консоль "+". При этом следует понимать что порядок снятия с паузы может не соответствовать порядку следования элементов в resourcesList, порядок будет зависеть только от момента завершения запросов page.getResource

Answer 1

[Владимир]

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