Как оптимизировать проверку условий в форме?

Question

[Mishcake]

Здравствуйте. Помогите, пожалуйста, оптимизировать код проверки условий на странице оформления заказа.

Есть страница оформления заказа, сделанная на конструкторе, который позволяет вставить кастомный html-css-js код.

Есть форма на PHP, которая принимает запросы, которые отправляются после первого входа на страницу заказа и возвращает информацию о пользователе из базы.

На основании этой информации делаются проверки.

Например, если вернулось hasDiagnostic === "true" и workedWith === "какая-то строка", то сделать следующие операции:

if (hasDiagnostic === "true") {
      
      if (workedWith !== undefined) {
        $(`option[value="${workedWith}"]`).attr("selected", "selected");
      }

      switch (workedWith) {
        case "Лилия":
          typeConsultation.prop("checked", true);
          typeConsultation.click();
          liliyaConsultation.prop("checked", true);
          liliyaConsultation.click();
          break;
        case "Александра":
          typeConsultation.prop("checked", true);
          typeConsultation.click();
          alexConsultation.prop("checked", true);
          alexConsultation.click();
          break;

Это лишь маленький участок кода, у меня там дальше по 15 таких "кейсов".

Это можно как-то оптимизировать или придётся мириться с огромным количеством кейсов и условий?

Или другой случай, более сложный для меня.

Нужно отслеживать изменение двух параметров, например: выбор имени специалиста и выбор типа услуги. Перед этим, естественно, скрипт запросил у базы данные о пользователе и вернул их.

И дальше начинается пляска:
- (если выбран тип услуги №1 и сотрудник) И (значение из базы === имя сотрудника), то всё ок, ничего не делаем
- (если выбран тип услуги №1 и сотрудник) И (значение из базы !== имя сотрудника), то выводим предупреждение
- (если выбран тип услуги №2 и сотрудник) И (значение из базы === имя сотрудника и другое значение из базы === true), то всё ок, ничего не делаем

И так далее. Вот тут ступор, как это всё учитывать при проверке, если пользователь может свободно переключать типы услуг (там радио-кнопка) и выбирать сотрудника из выпадающего списка?

Делать большой список условий, который будет проверяться при изменении типа услуги? Типа input[id="type"].change() - и внутри погнал проверять все условия, если сменилась радио-кнопка.

И отдельно select[option="Имя Фамилия"].change() - это для КАЖДОГО сотрудника в списке - и тоже аналогичные проверки, только на совпадение с типом услуги?

Вроде простая проверка, по сути, но у меня всё на 200 строк кода выходит. Понимаю, что творю дикую вирмишель из кода, но как оптимизировать - не понимаю :(

Подскажите, пожалуйста, какие-нибудь идеи по оптимизации таких проверок.

Comments

[FanatPHP]

при чем здесь РНР?

[Mishcake]

FanatPHP, кто-то когда-то мне сказал, что такие проверки можно/нужно делать на сервере, поэтому рискнул добавить тег.

[FanatPHP]

Какие "такие"?
На РНР надо проверять входящие данные.
Если же проверки касаются отображения, то РНР не при чем.

[Mishcake]

FanatPHP, окей, подправил, спасибо.

Answer 1

[Одиночка Айс]

1. liliyaConsultation, alexConsultation и прочие чем отличаются? Если все одинаково, акромя выбранного сотрудника, то напишите функцию, которая на вход будет принимать выбранного сотрудника, дабы под каждого одно и тоже не писать.

2. Распишите на листочке сначала все типовые действия, потом найдите однообразность данных/выполняемых условий.

3. Не торопитесь писать сразу весь код, делайте понемногу и проверяйте, например "Выбрать сотрудника", потыкали на всех, ага, все ок, двигаемся дальше.

4. typeConsultation.prop("checked", true); и typeConsultation.click(); более чем уверен, что можно вынести за условия (за скобки, вспоминаем математику).

5. Удачи))

Comments

[Mishcake]

1. liliyaConsultation и alexConsultation - это переменные, в которые я записал значения input чисто для удобства использования. Отличаются только значением:

var liliyaConsultation = $('input[value="424000"]');
var alexConsultation = $('input[value="424001"]');

2. Спасибо за совет, сделаю блок-схему, чтобы было нагляднее)

3. Вот так по частям я и писал все предыдущие условия, всё превратилось в простыню кода. А теперь когда надо проверять условия при смене сотрудника или смене типа услуги, то на ум приходит только .change() и внутри для каждого писать сотню условий...

4. Написать функцию типа chooseConsultation(), внутри которой будет выбираться чек-бокс и совершаться клик и вызывать функцию везде, где нужно эти 2 условия сделать?

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

function checkDiagnosticAndWorker(typeDiagnostic, workedWith) {
  if (
    typeDiagnostic.is(":checked") &&
    $(`option[value="${workedWith}"]`).is(":selected")
  ) {
    console.log(
      "Выбрана диагностика и сотрудник, с которым работал человек, всё ок."
    );
  } else if (workedWith !== undefined) {
    if (typeDiagnostic.is(":checked")) {
      console.log("Выбрана диагностика и человек ни с кем не работал, всё ок.");
    }
  }
}

И я вызываю её при каждом изменении типа услуги И при каждом изменении выпадающего списка:

checkDiagnosticAndWorker(typeDiagnostic, workedWith);

При условии, естественно, что workedWith определена при загрузке страницы.

5. Спасибо! :))

[Одиночка Айс]

checkDiagnosticAndWorker - на этапе разработки пригодится в качестве логирования/дебаггинга

[Mishcake]

Одиночка Айс, можно чуть подробнее, не понял, как пригодится и будет ли полезна в будущем?)

[Одиночка Айс]

Mishcake, когда будет тыкать в ваши инпуты или чего там, в консоли сможете смотреть прошло событие или нет, порой такие вещи помогают. Ну, или на ее основе сделать всю логику.

[Mishcake]

Одиночка Айс, а вот есть на её основе сделать логику - это как я написал выше? Отдельно вынести код выполнения функци и вызывать функцию, когда срабатывает какой-то триггер для проверки, да?

[Одиночка Айс]

Mishcake, да, как вариант