Как проверить что ответ предыдущих функций именно if (true)?

Question

[Deadorik]

Делаю проверку формы заказа. Есть несколько проверок полей и есть отдельная функция которая ответы сравнивает. Если 3 проверки успешны (ветки if), то форма отправляется.

Как сказать что ответы предыдущих функций должны быть if? Обычный true не помогает, т.к функции все равно выполняются. Пробовал return true в ветках функций проверок, но код тоже работает некорректно.

// Проверка номера телефона
  function numberTest(input) {
    const numberValidate =
      /^(\+7|7|8)?[\s\-]?\(?[489][0-9]{2}\)?[\s\-]?[0-9]{3}[\s\-]?[0-9]{2}[\s\-]?[0-9]{2}$/; // Регулярка для проверки телефона
    const phoneRegexp = input.value.match(numberValidate); // Проверяю что ввел пользователь с регуляркой

    if (phoneRegexp) {
      console.log("Номер подходит");
      removeError(input); // Если номер подходит, то удали ошибку и передай конкретное поле
    } else {
      console.log("Номер не подходит");
      addError(input); // Если номер не подходит, то добавь ошибку и передай конкретное поле
    }
  }

// Общая проверка
  function testAll() {
    if (numberTest && emailTest && adressTest) {
      console.log("Все проверки успешны");
    } else {
      console.log("Поля проверку не прошли");
    }
  }

Answer 1

[Александр]

- if (numberTest && emailTest && adressTest) {
+ if (numberTest() && emailTest() && adressTest()) {

ну и передать аргументы не забудьте.

Comments

[Elvis]

Добавлю что каждая функция должна возвращать true или false в зависимости от того прошла проверка или нет. то есть нужно добавить return true и return false в соответствующие места

[Александр]

HelpSophie, в профиле описано.

[HelpSophie]

Александр, супер, спасибо :)

[Deadorik]

Александр,

Не получается.. теперь ошибка "Uncaught (in promise) TypeError: Cannot read properties of undefined". Думаю что эта функция теперь выполняется раньше чем проверки, не понимаю почему так.. где-то не так написал..

async function testValidate() {
    const inputReq = document.querySelectorAll("._req"); // Получаю все обязательные поля
    for (let input of inputReq) {
      //Если нахожу телефон запускаю проверку телефона
      if (input.classList.contains("phone_num")) {
        numberTest(input); // Передаю дальше поле с телефоном
      }
    }
    testAll();
  }

  // Проверка номера телефона
  function numberTest(input) {
    const numberValidate = /^(\+7|7|8)?[\s\-]?\(?[489][0-9]{2}\)?[\s\-]?[0-9]{3}[\s\-]?[0-9]{2}[\s\-]?[0-9]{2}$/;
    const phoneRegexp = input.value.match(numberValidate);

    if (phoneRegexp) {
      console.log("Номер подходит");
      removeError(input); // Если номер подходит, то удали ошибку и передай конретное поле
      return true;
    } else {
      console.log("Номер не подходит");
      addError(input); // Если номер не подходит, то добавь ошибку и передай конретное поле
      return false;
    }
  }
// Все проверки
function testAll() {
    if (numberTest() && emailTest()) {
      console.log("Все проверки успешны");
    } else {
      console.log("Поля проверку не прошли");
    }
  }

[Deadorik]

Разобрался в чем дело.. в таком случае (if function( ) ) функция вызывается повторно, а у меня уже были вызовы через callback.

Решил не усложнять.. просто поставил счетчик который увеличивается в случае проверок true и проверять его значение.