Чем плохо массовое использование async функций?

Question

[andreys75]

Присоединился к проекту, каждая вторая функция описана как async, хотя никакой необходимости в этом нет. Помогите привести доводы почему это надо отрефакторить и убрать ненужные async await.

Comments

[origami1024]

Можно еще привести доводы, что лучше отрефакторить, чтобы каждая первая была async

[Владимир Коротенко]

Работает не трожь. Сами же пишите нормально. Теоретически они снижают скорость, практически нужно использовать профайлер и по результатам уже что то делать.

[Kovalsky]

Возможно это копипаста. Когда у тебя 90% функций в проекте асинхронные ты уже без разбору делаешь асинхронной почти каждую. Например это может быть задел на будущее: конкретно сейчас операции в функции синхронные, но вот уже завтра-послезавтра программист планировал напихать в неё асинхронных процедур

[andreys75]

вот пример одной из таких функций:

async function genFailedMessage(res) {
  let respondMessage = 'Validation Failed\n- Please update the following:\n'
  for (i in res.data.warnings) {
    respondMessage += '    - ' + res.data.warnings[i] + '\n'
  }
  for (i in res.data.errors) {
    respondMessage += '    - ' + res.data.errors[i] + '\n'
  }
  respondMessage += '- Please refer to Catalog Asset Best Practices\n'
  return respondMessage
}

[andreys75]

я уже даже не обращяю внимание на циклы, которые конечно можно заменить ...

[Павел Диденко]

Может быть вы скинете этот код, чтобы понять, действительно ли там не нужен async?

Answer 1

[Robur]

Помогите привести доводы почему это надо отрефакторить и убрать ненужные async await.

То есть у вас самого доводов никаких нет? А зачем тогда вы хотите это переделать?

Для начала стоит спросить ради чего так сделано.
Если это реально кто-то наговнокодил и налепил async не понимая что это и зачем, то довода "это лишний код" должно быть достаточно.
Ну можете еще сказать что это заворачивает все в лишние промисы, делает выполнение дольше на какие-то микросекунды и усложняет отладку и может портить стектрейсы.

Comments

[andreys75]

ну у меня есть доводы что если это не нужный код, то он во первых сбивает с толку, во вторых чуть медленне выполняется... но честно говоря все это пока не тянет на необходимость рефакторинга:) мне просто очень не нравится когда народ повтыкал async просто на всякий случай.... но надо убедить начальство :) вот может кто то может привести убиственные доводы :)

[Алексей Ярков]

andreys75,

убиственные доводы

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

А если серьёзно, то начальству с большой долей вероятности что и как под капотом. Работает? Ну и прекрасно. Код задачу выполняет? Ну и здорово. Вам с коллегами надо это обсудить и зарефакторить попутно делая таски.

[Robur]

andreys75, убеждать надо не начальство а команду.

Answer 2

[Владимир Солнцев]

Вы вот когда рашили, что асинх там не нужен, чем руководствовались? Подумайте об этом, вспомните и будут вам доводы, которые вы ищете.
Конечно, можно просто ждать, когда вопрос увидит телепат, который по аватарке узрит, что там за код и почему он не нужен.