Поиск проблемных промисов в JavaScript

[pavelsc]

С появлением async/await появились товарищи пишущие await [].forEach(), await promise.then(), async () => await fn(). Как по мне это performance review внеплановое сразу же должно вызывать

[fransua]

А это несет значительные потери производительности разве? Вот, например, await [].forEach(), если массив хоть немного длинный, работает практически так же, как и без await. TypeScript еще и выкатит предупреждение, что await не нужен, как и в 3 случае.

[mayorovp]

Эти моменты — просто показатель, что программист вообще не понимает что делает. А словосочетание performance review относится тут не к коду, а к программисту.

[kreddos]

Возможно я ошибаюсь, но performance review проводится для оценки результативности разработчика, я думаю в таких местах достаточно встречки после просмотра пуллреквеста, где можно будет обсудить, что так делать не совсем корректно.

Возможно меня заминусят

но бывают моменты где иногда это может быть оправдано, пример реализации allSettled где мы юзали promise.then(...).catch(...). Но тут опять же встречка после пр может помочь сторонам высказать свое мение

[neiroman2k]

Сначала был callback hell

Потом придумали bluebird и promise hell

Теперь живем в async/await hell

Может кто из гуру привести примеры

Говнокод -> true code