Как сделать код более плоским (промисы)?

Question

[triggerfinger]

Учусь писать на промисах, перенес логику Монгуз + Редис на промисы и есть ощущение что я намудрил (хотя все работает без нареканий). Что-то явно лишнее и вложенность убивает саму иде промисов. Как можно переписать этот код более грамотно, что можно убрать?

client.getAsync('data')
    .then((data) => {
      if (data) {
        return res.status(200).json(JSON.parse(data))
      } else {
        let count
        Article.count({})
          .then((c) => {
            count = c
            return count
          })
          .then(() => {
            Article
              .find({}, '-_id title content slug')
              .sort({_id: -1})
              .limit(limit)
              .skip((page - 1) * limit)
              .then((articles) => {
                let redisData = {
                  count,
                  articles
                }
                client.set('data', JSON.stringify(redisData), 'EX', 20, function (error) {
                  if (error) { throw error }
                })
                return res.status(200).json({
                  count,
                  articles
                })
              })
              .catch((err) => {
                return next(err)
              })
          })
      }
    })
    .catch((err) => {
      return next(err)
    })

(Тостеру надо срочно менять эту подсветку кода, это же ужас просто!)

Comments

[Петр Мухуров]

https://monsterlessons.com/project/lessons/async-a...

[triggerfinger]

pterodaktil, ок, спс за ссылку. А если чисто на промисах, то как быть?

[Петр Мухуров]

ну, поразносить на функции, что полотно убрать)

[Антон]

лучше перепишите на async/await, это сахар для промисов

Answer 1

[Илья Герасимов]

client.getAsync('data')
	.then((data) => {
		if (data)
			return [data.count, data.articles, true];

		return [Article.count({}),
			Article.find({}, '-_id title content slug')
				.sort({_id: -1})
				.limit(limit)
				.skip((page - 1) * limit),
			false]
	})
	.spread((count, articles, fromCache) => {
		if (!fromCache) {
			return new Promise((resolve, reject) => {
				client.set('data', JSON.stringify({count, articles}), "EX", 20, (error) => {
					if (error)
						return reject(error);

					return resolve([count, articles]);
				});
			});
		}

		return [count, articles];
	})
	.spread((count, articles) => {
		return res.status(200)
			.json({
				count,
				articles
			})
	})
	.catch((err) => {
		return next(err);
	});

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

Comments

[triggerfinger]

Спс за версию. А насчет вот этого куста у меня сомнения

.catch((err) => {
                return next(err)
              })

У меня два раза. В промисах вроде бы не надо второй раз было испольовать? Очибка отловится с одним этим блоком?

[Илья Герасимов]

triggerfinger, Отловится последним блоком не важно из какого промиса была брошена, мы можем например отловить её в отдельном блоке уровнем выше, как-то обработать (кинуть в лог или еще что-то специфическое) и бросить дальше. Но так, как нас интересует проброс ошибки дальше, в случае если она есть, то хватит последнего блока.

Например сделать так:

return [Article.count({}).catch(err => {
	console.log('Эй! А где количество?!', err);
	throw err;
}),

В случае возникновения ошибки там, у нас будет лог и она будет передана дальше (в последний блок, где будет передана функции next)

Answer 2

[qwead]

if (data) {
        return res.status(200).json(JSON.parse(data))
      } else {
...
}

Блок else можно опустить -- выигрыш в 1 уровень вложенности