На сколько правильно делать вложенность promise?

Question

[nurdus]

Добрый день.
На сколько "правильно" писать promise вложенным (пример "утрированный"):

let Redis     = require('ioredis');
let client = new Redis();

client.pop('games:wait')
  .then((token) => {
    client.hgetall('games:wait:' + token)
      .then((gameInfo) => {
        let player1 = gameInfo.player1;
        client.hgetall('account:' + player1)
          .then(('accountInfo') => {
            // ...
          })
          .catch(() => {
            // ...
          });
      })
      .catch(() => {
        // ...
      });
    })
    .catch(() => {
      // ...
    });

Comments

[Сергей delphinpro]

Из пустого в порожнее...
Говнокодъ.

[Aves]

их незачем делать вложенными

client.pop('games:wait')
  .then((token) => client.hgetall('games:wait:' + token))
  .then(({gameInfo: player1}) => client.hgetall('account:' + player1)
  .then((accountInfo) => {
    // ...
  })
  .catch(() => {
    // ...
  });

[nurdus]

Aves, Спасибо.

[nurdus]

Сергей, поэтому и спросил, чтобы не писать "говнокод".

[eRKa]

Сергей, грубиян, учись с уважением относиться к вопрошающим. Если ты крутой перец, то поправь человека, на то он и есть сервис Тостера.

[Сергей delphinpro]

eRKa, Это не грубость. Это оценка кода. Человек спросил «На сколько правильно», я ответил на сколько.

Answer 1

[Дима Паутов]

client.pop('games:wait')
client.hgetall('games:wait:' + token)
client.hgetall('account:' + player1)

Эти строки уже возвращают Promise, исходя из вашего примера.

Поэтому лучше писать так:

client.pop('games:wait')
  .then((token) => client.hgetall('games:wait:' + token))
  .then(({gameInfo: player1}) => client.hgetall('account:' + player1)
  .then((accountInfo) => console.log(accountInfo))
  .catch((error) => console.error(error));

Такой код выглядит просто и чудесно. Иначе вы пишите стилем "collback hell", вкладывая промисы друг в друга, убивая весь их смысл. Он были как раз придуманы ради того, что бы сделать ваш код в вопросе, таким, который я привёл в примере.

Answer 2

[Александр Таратин]

node7+

(async function (){

const token = await client.pop('games:wait');
const {player1} = await client.hgetall('games:wait:' + token);
const accountInfo = await client.hgetall('account:' + player1);

})().catch(e){ console.error(e); }

js async await

Comments

[nurdus]

Переписал на async/await :)

Answer 3

[Константин Китманов]

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