Вырыв из каждой петли

Question

Простите за задание этого вопроса. Я прошел довольно много решений, но для меня это не работает.

У меня есть около 5 полей ввода, и если он находит незаконный символ в любом из полей ввода (если кто-то входит в один и пытается сохранить), он должен предупредить сообщение.

До сих пор, когда он находит один, он делает предупреждение, но затем он идет и сохраняет его.

Я пытаюсь вырваться из цикла each(), return false не работает.

Лучшее решение было бы также оценено.

Fiddle here

// Illegal Character Check 
$('.illegal-check').click(function() { 
    var illegal = /[&<>]/; 
    $(".scoring input:text").each(function() { 
     var inptStr =$(this).val(); 
     if (illegal.test(inptStr)===true) { 
      alert('Your Scoring field(s) contain these & < > invalid characters. Please remove them and try again.'); 
      return false; 
     } 
    });//each 
});//click 

Answer 1

$('.illegal-check').click(function (e) { 
    ... 
    if (illegal.test(inptStr)===true) { 
     ... 
     e.preventDefault(); 
     return; 
    } 
} 

EDIT: Причина, почему ваши данные по-прежнему получать сохраняются в том, что даже если вы нарушаете из петли с помощью «вернуться ложным;» ваше представление данных все еще продолжается. Вам также нужно отменить его. Для этого вы используете аргумент события «e», переданный обработчику событий, вызывая его функцию «preventDefault».

Answer 2

Можно также добавить переменную, чтобы увидеть, был ли нелегальный значение было найдено like this:

// Illegal Character Check 
$('.illegal-check').click(function() { 
    var illegal = /[&<>]/; 
    var found = false; 
    $(".scoring input:text").each(function() { 
    var inptStr =$(this).val(); 
    if (illegal.test(inptStr)===true) { 
     alert('Your Scoring field(s) contain these & < > invalid characters. Please remove them and try again.'); 
     found = true; 
    } 
    }); 
    if(!found){ 
    alert("Nothing has been found") 
    }else{ 
    alert("Illegal char has been found"); 
    } 
}); 

Answer 3

Лично я хотел бы добавить обработчик представить в форме вместо события щелчка по ячейке таблицы.

$('form').on('submit', function(e) { 
    // ... 
    if (illegal.test(inptStr)===true) { 
     alert('...'); 
     e.preventDefault(); 
     return false; 
    } 
    // ... 

Здесь событие передается функции, и любая ошибка предотвратит запуск этого события из фактического представления.

Answer 4

JQuery-х $.fn.each, похожий на $.each, сломается из петли с return false и переходите к следующей итерации с любой не falsy return. It's documented on the API site.

По всем вопросам, эта часть вашего кода должна быть в порядке.

Answer 5

Вот обновленная скрипка. Проверьте наличие ошибок в отдельной функции

$('.illegal-check').click(function() { 
    if(checkIllegal()) evt.preventDefault(); 
}); 

function checkIllegal(){ 
    var illegal = /[&<>]/; 
    var isIllegal = false; 
    $(".scoring input:text").each(function() { 
     if (illegal.test($(this).val()) && !isIllegal) { 
     alert('Your Scoring field(s) contain these & < > invalid characters. Please remove them and try again.'); 
     isIllegal = true; 
     return false; 
     } 
    }); 
    return isIllegal; 
} 

http://jsfiddle.net/T2vT9/7/

Answer 6

return false утверждение не выбиться из цикла $(...).each(...). Ваша проблема в том, что она по-прежнему продолжается и сохраняет ее. Вы также должны остановить событие от выполнения его действия по умолчанию, используя e.preventDefault(). (примечание: это было также упомянуто в помойке кода по m1kael, которые не добавляют объяснения)

Эта сумма на этот код:.

// Illegal Character Check 
$('.illegal-check').click(function (e) { 
    var illegal = /[&<>]/; 
    $(".scoring input:text").each(function() { // loop over elements in form 
     var inptStr = $(this).val(); 
     if (illegal.test(inptStr)===true) { 
      alert('...'); // show alert 
      e.preventDefault(); // prevent default action on click (stop submit) 
      return false; // break out of elements loop, no more alerts needed 
     } 
    });//each 
});//click