Почему тут повторно используется Promise.all?

Question

[Rapen]

Есть вот такой код c промисами почему в предпоследнем then мы опять используем promise.all, если просто вернуть map, т.е массив этих же самих промисов то undefined, почему?

let names = ['iliakan', 'remy', 'jeresig'];

let requests = names.map(name => fetch(`https://api.github.com/users/${name}`));

Promise.all(requests)
  .then(responses => {
    // все промисы успешно завершены
    for(let response of responses) {
      alert(`${response.url}: ${response.status}`); // покажет 200 для каждой ссылки
    }

    return responses;
  })
  // преобразовать массив ответов response в response.json(),
  // чтобы прочитать содержимое каждого
  .then(responses => Promise.all(responses.map(r => r.json()))) // Вот тут!
  // все JSON-ответы обработаны, users - массив с результатами
  .then(users => users.forEach(user => alert(user.name)));

Answer 1

[Дмитрий]

Потому что метод json() возвращает промис, а не результат, соответственно применение к нему user.name не будет иметь смысла, так как у промиса никакого name нет, он будет только у результата, когда мы оборачиваем массив промисов в Promise.all, то функция возвращает промис, а значит в следующий then попадает уже результат вычисления, то есть данные юзеров.

Comments

[Rapen]

Понял, а responses это массив запросов, которые являются api, прямо как в ajax со своими status, url?

Answer 2

[Евгений]

А что без лапши не написать?

const fetch = require('node-fetch');
let names = ['iliakan', 'remy', 'jeresig'];
let requests = names.map(name => fetch(`https://api.github.com/users/${name}`).then(response => response.json()));

(async () => {
  const users = await Promise.all(requests);
  users.forEach(user => {
    console.log(user.name);
  })
})();

Comments

[Дмитрий]

Данные возвращаются в json, не будет у вашего user никакого name, пока вы к результату не примените json

[Евгений]

Дмитрий, а если запустить и проверить?
Ну и на 3 строчку внимательнее посмотреть надо.

[Дмитрий]

Евгений, Писать надо так, чтобы она была не километрового размера и не уходила за пределы экрана, тогда я с большим удовольствием буду на нее смотреть =)

[Евгений]

Дмитрий, за линейку в webstorm не выходит

[Дмитрий]

Евгений, Мне плохо виден ваш вэб-сторм отсюда, я по старинке на сайте смотрел. У меня кончается ровно на
fetch(`https://api.github.com/users/${name}`)

Так да, код абсолютно корректный.

[Даша Циклаури]

Дмитрий, еще когда точек больше одной в цепочке вызовов, то каждый вызов на новой строке форматируют, поэтому тут больше вопросов к форматированию

[Евгений]

Даша Циклаури, кто сказал что точку переносят на другую строку?
Если заглянуть в google code style
https://google.github.io/styleguide/jsguide.html#f...
там можно увидеть такую сноску, что к точке правило не относится, потому что это не оператор
"When a line is broken at an operator the break comes after the symbol. (Note that this is not the same practice used in Google style for Java.)
This does not apply to the dot (.), which is not actually an operator."
Взять туже библиотеку для ассертов chai, у меня бы крыша поехала писать вот так
beverages.should
.have
.property('tea')
.with
.lengthOf(3)

[Даша Циклаури]

Евгений, честно не знаю кто этот стайлгайд использует, кроме ,может быть, самого Гугла. Обычно ставишь Преттир, вперед и с песней.

Крыша, возможно, писать и поехала, но в одну строчку она едет когда смотришь диффы - добавилась еще одна функция в цепочке и ты ее ищешь

[Rapen]

Ну с асинхронной функцией то понятно все, просто не учел тот момент что .json() возвращает чертов промис, и тут ломаю голову, почему же у меня undefined :)