Переменная, нужна ли проверка

@u4en1k · Регистрация: 04.04.2018

Студворк — интернет-сервис помощи студентам

Здравствуйте.
Имеется к примеру такой код

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
var first_name = document.getElementById('first_name').value;
var last_name = document.getElementById('last_name').value;
if(!!last_name){
    if(!!first_name){
     $.ajax({
                ...
         });
    }
    else{
        alert('заполните поле Имя');
    }
}
else{
  alert('заполните поле Фамилия');
}

Если я таким образом произвожу проверку, то код перед отправкой обработчику ajax, проверяет на пустоту, тем самым обработчик получит не пустые значения $_POST. Разумеется, в обработчике тоже имеются различные проверки.
Меня собственно что интересует, правильная ли такая реализация?

@renat_dmitriev · 14.11.2018, 21:09

Поскольку свойство value строковое, правильнее было бы проверять его lastName !== '', хотя результат тот же

@u4en1k · 14.11.2018, 21:25 **[ТС]**

renat_dmitriev, поэтому и проверяю !!. Суть вопроса все та же, я правильно ведь думаю, что так и необходимо делать, да бы нагрузка на код php уменьшалась. Повторюсь, в обработчике тоже есть empty intval mysqli_real_escape_string и разные условия. НО, сам запрос обработчику, обработка php кодом, потом отправка снова ajax, обработка почти этим же самым условием (если data такая-то вернулась, то это делать, если нет, то это), что отдал код php, и тем самым делая такую проверку, много действий удается избежать. Это я понимаю, а сама реализация верная? К примеру у меня 10 переменных, то есть мне так же нужно каждую проверять? Я думал массив собрать, потом проверять, но мне нужно выводить сообщение об ошибках... Как тут быть еще не придумал, ведь имена переменных разные, каждое за свое поле.

@renat_dmitriev · 14.11.2018, 21:46

Да, красивей будет считать все инпуты внутри формы с реквизитом required

HTML5
1
2
3
4
5
6
<form>
   <input name="first_name" data-ru="Имя" required>
   <input name="second_name" data-ru="Отчество" required>
   <input name="last_name" data-ru="Фамилия" required>
</form>
<button onclick=check()>Check</button>

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
function check() {
   var errorsList = Array.from(document.forms[0].getElementsByTagName('input'))
       .reduce((errors, el) => {
           if(el.hasAttribute('required') && !el.value) {
               errors.push(el.dataset.ru);   
           }
          return errors
       }, []);
 
   if(errorsList.length) 
      alert('Не заполнены поля: ' + errorsList.join(', '));
}

@u4en1k · 14.11.2018, 22:06 **[ТС]**

renat_dmitriev, а если я отказался от формы, и input's расположены в div? Тоже самое получается. Понял как нужно, шикарное решение, благодарю. Таким образом, решением, я смогу вообще код сильно оптимизировать

Новые блоги и статьи Все статьи Все блоги /
Валидация и контроль данных табличной части документа перед записью Maks 22.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в КА2. Задача: контроль и валидация данных табличной части документа перед записью с учетом регламента компании. . .	Отчёт о затраченных материалах за определенный период с макетом печатной формы Maks 21.04.2026 Отчёт из решения ниже размещён в конфигурации КА2. Задача: разработка отчёта по затраченным материалам за определённый период, с возможностью вывода печатной формы отчёта с шапкой и подвалом. В. . .	Отчёт о спецтехнике находящейся в ремонте Maks 20.04.2026 Отчёт из решения ниже размещен в конфигурации КА2. Задача: отобразить спецтехнику, которая на данный момент находится в ремонте. Есть нетиповой документ "Заявка на ремонт спецтехники" который. . .	Памятка для бота и "визитка" для читателей "Semantic Universe Layer (Слой семантической вселенной)" Hrethgir 19.04.2026 Сгенерировано для краткого описания по случаю сборки и компиляции скелета серверного приложения. И пусть после этого скажут, что статьи сгенерированные AI - туфта и не интересно. И это не реклама -. . .
Запрет удаления строк ТЧ документа при определённом условии Maks 19.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "Аккумуляторы", разработанного в конфигурации КА2. У данного документа есть ТЧ, в которой в зависимости от прав доступа. . .	Модель заражения группы наркоманов alhaos 17.04.2026 Условия задачи сформулированы тут Суть: - Группа наркоманов из 10 человек. - Только один инфицирован ВИЧ. - Колются одной иглой. - Колются раз в день. - Колются последовательно через. . .	Мысли в слух. Про "навсегда". kumehtar 16.04.2026 Подумалось тут, что наверное очень глупо использовать во всяких своих установках понятие "навсегда". Это очень сильное понятие, и я только начинаю понимать край его смысла, не смотря на то что давно. . .	My Business CRM MaGz GoLd 16.04.2026 Всем привет, недавно возникла потребность создать CRM, для личных нужд. Собственно программа предоставляет из себя базу данных клиентов, в которой можно фиксировать звонки, стадии сделки, а также. . .

@renat_dmitriev 392 / 294 / 121 Регистрация: 26.08.2016 Сообщений: 902
	14.11.2018, 21:09
	Поскольку свойство value строковое, правильнее было бы проверять его lastName !== '', хотя результат тот же 0

@u4en1k 133 / 118 / 34 Регистрация: 04.04.2018 Сообщений: 593
	14.11.2018, 21:25 [ТС]
	renat_dmitriev, поэтому и проверяю !!. Суть вопроса все та же, я правильно ведь думаю, что так и необходимо делать, да бы нагрузка на код php уменьшалась. Повторюсь, в обработчике тоже есть empty intval mysqli_real_escape_string и разные условия. НО, сам запрос обработчику, обработка php кодом, потом отправка снова ajax, обработка почти этим же самым условием (если data такая-то вернулась, то это делать, если нет, то это), что отдал код php, и тем самым делая такую проверку, много действий удается избежать. Это я понимаю, а сама реализация верная? К примеру у меня 10 переменных, то есть мне так же нужно каждую проверять? Я думал массив собрать, потом проверять, но мне нужно выводить сообщение об ошибках... Как тут быть еще не придумал, ведь имена переменных разные, каждое за свое поле. 0

@u4en1k 133 / 118 / 34 Регистрация: 04.04.2018 Сообщений: 593
	14.11.2018, 22:06 [ТС]
	renat_dmitriev, а если я отказался от формы, и input's расположены в div? Тоже самое получается. Понял как нужно, шикарное решение, благодарю. Таким образом, решением, я смогу вообще код сильно оптимизировать 0

Опции темы

Переменная, нужна ли проверка

Решение