Как заблокировать submit при нахождении стоп-слов?

Question

[YakutD]

Вроде простая задача,но у меня ни в какую не получается.
Есть список стоп-слов, храню их в массиве. Есть input в который пользователь вводит текст. И есть submit который должен быть disabled если пользователь введет хотя бы одно стоп слово.
Вот мой код:

$(document).ready(function(){
  
   $('.myinput').bind("keyup mousemove", function(){
         
       var str = $(".myinput").val();//Выбираем ввод пользователя
       
        var arr = ["лягушка","березка","черепица"];// массив стоп слов
    
    //перебор массива стоп-слов
       
 for (var i = 0; i< arr.length; i++){
            // alert(arr[i]);
    if (str.indexOf(arr[i]) != -1)
        {
        $('.mysubmit').prop('disabled',true);
        }
else {
        $('.mysubmit').prop('disabled',false);
    }
       }
   });

});//end ready

Если я по деревянному пробую вписать каждое стоп-слово в else if то все работает,но это же долго,тем более предполагается наличие куда большего списка стоп-слов.Что я делаю не так?И есть ли какие-либо jquery плагины для валидации формы по вводимым словам?

Comments

[Сергей delphinpro]

1.
Условие неверное.

Еслия вбиваю «лягушка» то на первой итерации цикла кнопка заблокируется, а на второй разблокируется, потому что второго слова «березка» в инпуте нет. А первое есть, и кнопка разблокирована ))
Более правильный алгоритм:

var stopExists = false;
for (var i = 0; i< arr.length; i++){
    // alert(arr[i]);
    if (str.indexOf(arr[i]) != -1) {
        stopExists = true;
        break; // одно слово нашли, остальные можно не искать
    }
}

$('.mysubmit').prop('disabled', stopExists);

2.
Набор событий попахивает говнецом. Просто используйте одно событие input. Оно вызывается при любом изменении значения (ручной ввод, вставка из буфера, заполнение скриптом)

3.
ФОРМАТИРУЙТЕ КОД!
Форматируйте, м***ь в***у код, когда вы выкладываете его на всеобщее обозрение!
Выложить неотформатированный код — это то же самое, что и свои обосранные подштанники вывесить на балкон на первом этаже.

Answer 1

[0xD34F]

Цикл не нужен:

$('.mysubmit').prop('disabled', arr.indexOf(str) !== -1);

UPD. А-а-а, чёрт возьми - у вас поиск слов по тексту, а не проверка соответствия текста одному из слов... Тогда так:

$('.mysubmit').prop('disabled', arr.some(n => str.indexOf(n) !== -1));

Comments

[YakutD]

Спасибо,вы спасли мой крохотный мозг от разрушения)

[Сергей delphinpro]

Не будет работать =))
https://codepen.io/anon/pen/PJQZLP?editors=1010
попробуйте вбить «лягушка зеленая»

[YakutD]

Сергей, да,действительно(

[Сергей delphinpro]

Так лучше ))
Тоже цикл, только скрыто и красиво.

[YakutD]

Сергей, я вот только не пойму что n значит

[Сергей delphinpro]

YakutD, Элемент массива
Это стрелочная функция, можно переписать в более привычном виде:

arr.some(function(n) {
    return str.indexOf(n) !== -1
})