Где ошибка в Javascript?

Question

[Влад]

Посмотрите вывод консоли с отображением выполняемых событий в Яндекс.Метрике https://karedesign.ee/ru/product/box-zebra?_ym_debug=1

Если нажать "В корзину" и нажать на кнопку "Продолжить", то запускается обработчик события:
$("#telefon-submit").click(function(event){

Далее происходит проверка введённых данных:

if(/^(?=.*[0-9])[0-9 \(\)\+-]+$/i.test(document.getElementById('phone').value))
 submitForm();
else
 $("#phone").css("border", "2px solid red");

Если в поле ничего не вводить или ввести цифры, то почему одновременно срабатывают оба условия TRUE и False?
То есть и submitForm(); срабатывает и Yandex Metrika получает событие
yaCounter45870459.reachGoal('add-telefon');
и одновременно поля выделяются красным?
$("#phone").css("border", "2px solid red");

Вот кусок кода, о котором идет речь:

<script>
        $("#telefon-submit").click(function(event){
             // cancels the form submission
             event.preventDefault();
             if(/^(?=.*[0-9])[0-9 \(\)\+-]+$/i.test(document.getElementById('phone').value))
                submitForm();
             else
                $("#phone").css("border", "2px solid red");
        });
        function submitForm(){
             // Переменные с данными из формы
             var phone = $("#phone").val();

             $.ajax({
             type: "POST",
             url: "sendmail.php",
             data: "phone="+ phone + "&product="+ location.href,
             success : function(text){
             if(text == "success"){
             formSuccess();
             }
             }
             });
        }
        function formSuccess(){
             yaCounter45870459.reachGoal('add-telefon'); //Yandex Metrika
        }
        </script>

Comments

[Дмитрий]

if(/^(?=.*[0-9])[0-9 \(\)\+-]+$/i.test(document.getElementById('phone').value)){
                submitForm();
            }
             else{
                $("#phone").css("border", "2px solid red");
             }

[Влад]

slo_nik, не помогло(

Answer 1

[Павел Корнилов]

Конечно, поскольку /^(?=.*)/ находит любое начало строки, что бы за ним не следовало.
Если сделать /^(?=.+)/ - пустое значение уже не отловится.

Comments

[Влад]

ввёл

if(/^(?=.+)/i.test(document.getElementById('phone').value)){

всё равно выполняется и TRUE и False условия... посмотрите.. я не понимаю почему так..

[Влад]

Условие TRUE не выполняется!!

Я удалил функцию formSuccess(), но yaCounter45870459.reachGoal('add-telefon'); это событие все равно срабатывает...
Можно как-то определить откуда оно выполняется??
В tagmanager его нет.

[Павел Корнилов]

Ерунда, только что в консоли проверил:

/^(?=.+)/i.test('+7978')
true
/^(?=.+)/i.test('')
false

[Влад]

Аа жесть! Нашел проблему.
У меня ранее этот скрипт выполнялся через Tagmanager. Потом я его удалил оттуда, но внесённые изменения не нажал "Подтвердить", поэтому у меня до сих пор висела старая версия Tagmanager с этим скриптом - при этом самого тега с этим скриптом уже нигде не было - он был удален, но не ПОДТВЕРЖДЕН!

проблема решена - спасибо!

[Павел Корнилов]

Влад, как все сложно =)
На здоровье!

Answer 2

[ой всё]

так через sources в dev tools поставьте брейкпойнт в начале функции и посмотрите по шагам как и что считается, в хроме удобнее всего

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

слёту не разберусь в вашем регексе, он проверяет на пустое поле?

Comments

[Влад]

вроде регекс этот должен проверять на наличие любых записей типа
12345678
([phone]
[phone]

то есть только цифры, пробелы, скобки, знак плюса и чтоб не пусто было.
может неправильно?

[ой всё]

может проще сделать явную проверку на пустоту?