можно еще коллбеков накидать

asyncForEach колбек и использует :)

function (err, result) добавить где-то и будет ?

Божественно

Это лучший код эвар!

asyncForEach колбек и использует :)

whoopsies

asyncForEach колбек и использует :)

Ну тогда он идеален

причём не просто колбек, асинхронный колбек!

10 из 10

Ну ладно, совсем загнобили )

причём не просто колбек, асинхронный колбек!

?

Ну ладно, совсем загнобили )

Та нормально, нессы

Давайте хоть перепишем код парню, чтоб не просто ругаться :)

тут это называется критика(с)

разочек

можно, но со скриншота переписывать не комильфо

Ну тогда подождём самого кода

Давайте хоть перепишем код парню, чтоб не просто ругаться :)

Зачем? Можно же просто орнуть всем вместе

Зачем? Можно же просто орнуть всем вместе

У меня есть guilty pleasure

Брать чужие куски говнокода, делать конфетку и обтекать с того, как хорошо получилось

Брать чужие куски говнокода, делать конфетку и обтекать с того, как хорошо получилось

Я первый шаг пропускаю обычно)

Я первый шаг пропускаю обычно)

Можно ещё вместо чужих брать свои, но тогда это банальный рефакторинг :)

Знаешь там, взять старый проект и все промисы переписать на асинки

ммм, йамми

фап фап

Хы, асинки ради асинков :)

тот момент, когда человек не понимает сути промисов и асинков

.then(async это конечно божественно

тот момент, когда человек не понимает сути промисов и асинков

хайп-дривен девелопмент чо :)

Дело в том что получение записи асинхронно, отправка тоже асинхронна и удаление асинхронно

http://jsfiddle.net/#&togetherjs=DQrBg7Oe3S

Покажите как правильно

Я буду только признателен

Ты не поверишь, но это только так работает

.then(async это конечно божественно

И чем .then(async плох? Например, здесь: getPromise() .catch(tap(console.error)) .then(async x => { const a = await doFirst(x) const b = await doSecond(a) return String(b) })

И чем .then(async плох? Например, здесь: getPromise() .catch(tap(console.error)) .then(async x => { const a = await doFirst(x) const b = await doSecond(a) return String(b) })

бля ну например тем, что код менее очевиден

во вторых там он вообще не нужен

И чем .then(async плох? Например, здесь: getPromise() .catch(tap(console.error)) .then(async x => { const a = await doFirst(x) const b = await doSecond(a) return String(b) })

да и за такой код в ПР я бы по рукам бил

во вторых там он вообще не нужен

ну именно в том случае да

И чем .then(async плох? Например, здесь: getPromise() .catch(tap(console.error)) .then(async x => { const a = await doFirst(x) const b = await doSecond(a) return String(b) })

А какой смысл если это очень красиво и без асинка будет

И чем .then(async плох? Например, здесь: getPromise() .catch(tap(console.error)) .then(async x => { const a = await doFirst(x) const b = await doSecond(a) return String(b) })

ахренеть вот это конструкция

да и за такой код в ПР я бы по рукам бил

покажи как правильно

А какой смысл если это очень красиво и без асинка будет

это пример, в async функции можно сделать и более сложную логику

А почему просто все на асинке не построить

зачем там .then вообще тогда

А почему просто все на асинке не построить

с try/catch?

напиши что ты получить хочешь

с try/catch?

Да

с try/catch?

да

да

это будет менее красиво

бля

напиши что ты получить хочешь

по очередная обработка(функция createMessage), отправка(sendemail) и удаление записи(при помощи рест запроса)

ну перепиши на .then

не надо делать говно из смеси

разбей на функции и т.д.

Это будет более красиво, но если тебе так не нравится трайкеч - можно сделать await somePromise().catch(handleError) await otherPromise().catch(handleError)

Так можно и разные хендлеры нацепить чтоб более вариативно обрабатывать

почему ты сделал и обычный коллбек и await?

Мы ожидаем пока не отправится сообщение

так ты мог после отправки и обработать сообщение

а не внутри коллбека

response = await smtpTransport.sendMail(createMessage()); ...

разбей на функции и т.д.

async function process (x) { const a = await doFirst(x) const b = await doSecond(a) return String(b) } getPromise() .catch(tap(console.error)) .then(process) Можно .then(async вынести в другую функцию, да, согласен

ну и error это try/catch в данном случае

здесь у всех функций максимально кривое api

хм, сейчас попробую

но опять же, если getRecord или sendMailMessage выбросит ошибку, цикл прервется

и здесь цикл будет последовательным

Люди, видимо, не понимают, для чего сделали асинки

И пихают их везде, говоря, что так лучше

ну и это можно как-то так переделать, сильно не увлекался ``` async function sendMailMessage(message, uri) { const smtpTransport = mailer.createTransport(authMail); try { const sendResponse = await smtpTransport.sendMail(createMessage(message)); console.log("Message sent"); } catch (error) { console.error(error); } const deleteResponse = await requestify.delete(uri, authRest); deleteResponse.getBody(); console.log("Delete in Service Manager message") smtpTransport.close(); } ``` это если "дословно"

и асинки и явные промисы имеют минусы и плюсы их можно использовать в одном проекте но не надо смешивать их в пределах одной функции

не надо юзать везде let, если не меняешь значение переменной

не надо юзать везде let, если не меняешь значение переменной

так Let в области видимости блока только доступен же

так Let в области видимости блока только доступен же

const тоже

так Let в области видимости блока только доступен же

а const для кого сделали?

бля, хватит изучать кусок жс, а

то недоизучили async/await теперь уж не знают о let/const

может @js_noobs_ru?

Вот как хорошо было в ES5

var

Бля

может @js_noobs_ru?

.

Вот как хорошо было в ES5

И без промисов, коллбек хелл

ребят, есть ли бест практис для масштабируемого REST API на node.js без express\fastify...etc?

Чё с орм на ноде?

Созрело для продакшна?

ребят, есть ли бест практис для масштабируемого REST API на node.js без express\fastify...etc?

connect

Чё с орм на ноде?

sequelize

ребят, есть ли бест практис для масштабируемого REST API на node.js без express\fastify...etc?

koa2 + createrest

ок, без фреймворков, на чистом http.createServer))

ок, без фреймворков, на чистом http.createServer))

createrest + что хочешь