Почему не советуют писать await после return в функции?

Question

[1233211]

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

async function getUsers(userId) {
	return await db.users.findOneById(userId);
}

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

async function getUsers(userId) {
	return db.users.findOneById(userId);
}

почему?

Comments

[Interface]

Так спросите у тех, кто не советует. И тут расскажите потом :)

[Kovalsky]

Если findOneById возвращает промис, то async/await там не нужны вообще.

[Interface]

Еще на тему разницы почитайте вот этот пост на стековерфлоу: https://stackoverflow.com/questions/38708550/diffe...

Answer 1

[Somewhere Intech]

Потому что нет try catch конструкции, а это уже deprecated по умолчанию..
Во втором случае вам промис вернётся и уже за функцией можно обрабатывать ошибку. В противном случае у вас двойная обработка(как минимум) получится

Comments

[Interface]

Не могли бы раскрыть мысль? Если db.users.findOneById(userId) бросает ошибку (возвращенный промис реджектится), то в обоих случаях вернется промис который зареджектится. То есть в обоих случаях его можно обработать через .catch() или try/catch (внутри async функции). В чем тогда разница?

а это уже deprecated по умолчанию

Вот это не очень понятно

[Somewhere Intech]

Interface, в nodejs уже нельзя просто так оставить unhandled reject в просисе, код не компилируется. А так вобщем вот статья рассказывает почему так делать не стоит. Вообще, я думаю, что это лишает гибкости в коде - лучше сделать await вне вспомогательной функции

[Interface]

Johnny Lowhunter, о каком unhandled reject вы говорите? В обоих случаях функция вернет промис который зареджектится в будущем. Если его не ловить - он конечно станет unhandled. Но ловить его можно и try/catch и .catch()

[Somewhere Intech]

Interface, согласен, тут я узко не верно мыслил ;)

Answer 2

[SagePtr]

Потому что асинхронная функция возвращает промис в любом случае, конструкция await там после return излишняя

Answer 3

[Robur]

Нет "правильно" и "неправильно". Лично я раньше тоже убирал await, потом наоборот стал добавлять.
Не советуют потому что с точки зрения логики это лишнее слово.

Но, как говорится, есть нюансы. Если копнуть глубже - V8 оптимизирует return с await лучше чем просто return.
А так же конструкция с await дает лучший стектрейс. Еще что-то было в пользу return await, но я не помню детали. В блогах самого V8 или девелоперов его пилящих где-то была целая большая статья на эту тему, с раскладкой по всем деталям.

Поэтому, если вы видите return await не спешите говорит что человек написавший эти строки не в курсе как работает async/await, возможно он знает лучше вашего ;)

Answer 4

[Антон Швец]

return await promise
"разворачивается" в
return promise.then(result => result);
Зачем оно вам надо?

И плюс если в функции нет ни одного await, то async ей тоже не нужен, меньше текста, меньше нагрузки на программиста и компилятор.