Добавляет "На практике" для Promise.all()

[zheleznikov]

Добавил еще один практический пример для Promise.all()

[zheleznikov]

@vitya-ne
Витя, привет.
Извини за долгую задержку.

Отвлекся на рабочие дела и переделать пример оказалось не такой простой задачей. В результате предлагаю упростить пример и убрать отправку формы, потому что в этом примере перемешались Promise.all() и цепочка вызовов.
Теперь в примере, в случае успешного выполнения всех промисов, просто будет доступна кнопка отправки данных.
Это вполне жизненно и не перегружает.
Вот ссылка на страницу на превью: https://content-5989.dev.doka.guide/js/promise-all/

[vitya-ne]

предлагаю также добавить функцию для блокировки кнопки отправки (disableSubmitButton()) в блоке catch. Это необходимо для такого кейса:
Пользователь вводит корректные данные формы, кнопка отправки становится доступной (enabled). Пользователь меняет значение одного из полей формы на невалидное значение.

[zheleznikov]

Согласен, нельзя оставлять состояние кнопки без внимания. Добавил вызов функции, чтобы блокировать кнопку. Мне кажется теперь нормально. Да ведь? :)
Но мне теперь не очень нравится, что в части кода есть комментарии, например, вот здесь:
checkEmail(email), // Проверка, что email свободен

А в блоках then и catch комментариев к вызову функций нет. Мне кажется будет лучше, если будет одинаково. Я бы хотел убрать комментарии примера. Будет выглядеть так:

Promise.all([
  checkEmail(email),
  checkLogin(login), 
  checkPromoCode(promoCode)
])
  .then(() => {
    enableSubmitButton()
  })
  .catch((error) => {
    disableSubmitButton()
    showValidationError(error)
  })

Как думаешь, убрать комментарии из кода?

[vitya-ne]

Коменты в коде на усмотрение автора.
А вообще, лучше описать всё обычным текстом над примером.

[vitya-ne]

Теперь в примере, в случае успешного выполнения всех промисов, просто будет доступна кнопка отправки данных. Это вполне жизненно и не перегружает. Вот ссылка на страницу на превью: https://content-5989.dev.doka.guide/js/promise-all/

Я посмотрел на текущую реализацию немного под другим углом...
Сейчас получается что мы должны выполнять Promise.all() при любом изменении зависимых полей в форме. При этом будут выполняться все три проверки, даже если рузльтат первой вернёт false.
Насколько это оптимальный подход ?
В первоначальном варианте, выполнение всех проверок было оправдано, так как происходило однакратно при попытке отправки формы.

[github-actions]

Превью контента из bbbdad0 опубликовано.

[zheleznikov]

Теперь в примере, в случае успешного выполнения всех промисов, просто будет доступна кнопка отправки данных. Это вполне жизненно и не перегружает. Вот ссылка на страницу на превью: https://content-5989.dev.doka.guide/js/promise-all/

Я посмотрел на текущую реализацию немного под другим углом... Сейчас получается что мы должны выполнять Promise.all() при любом изменении зависимых полей в форме. При этом будут выполняться все три проверки, даже если рузльтат первой вернёт false. Насколько это оптимальный подход ? В первоначальном варианте, выполнение всех проверок было оправдано, так как происходило однакратно при попытке отправки формы.

Ну да, теперь сложно объяснить использование Promise.all() для этой цели. Теперь мне даже не нравится первоначальный вариант, потому что можно было просто сделать валидацию на бэкенде в момент регистрации.
Предлагаю пока что захолдить этот совет. Подумаю, насколько он полезен.
Просто хотелось показать случай, когда нужно получить какую-то информацию из нескольких источников и если вся информация пришла, то какая-то возможность становится доступной.

[vitya-ne]

Теперь в примере, в случае успешного выполнения всех промисов, просто будет доступна кнопка отправки данных. Это вполне жизненно и не перегружает. Вот ссылка на страницу на превью: https://content-5989.dev.doka.guide/js/promise-all/

Я посмотрел на текущую реализацию немного под другим углом... Сейчас получается что мы должны выполнять Promise.all() при любом изменении зависимых полей в форме. При этом будут выполняться все три проверки, даже если рузльтат первой вернёт false. Насколько это оптимальный подход ? В первоначальном варианте, выполнение всех проверок было оправдано, так как происходило однакратно при попытке отправки формы.

Ну да, теперь сложно объяснить использование Promise.all() для этой цели. Теперь мне даже не нравится первоначальный вариант, потому что можно было просто сделать валидацию на бэкенде в момент регистрации. Предлагаю пока что захолдить этот совет. Подумаю, насколько он полезен. Просто хотелось показать случай, когда нужно получить какую-то информацию из нескольких источников и если вся информация пришла, то какая-то возможность становится доступной.

Можно подумать как изменить описание самой формы.
Но я предлагаю сохранить первоначальную конструкцию с немного изменённым флоу:

PromoseAll([check1(), check2(), check3()])
  .then(()=> {
    return true
  })
  .catch((error) => {
    showValidationError(error)
    return false
  })
  .then(valid=> {
    if (valid) {
      return submitRegistration({ email, login, promoCode }))
  })
  .catch((error) => {
    showSubmitError(error)
  })