Как объединить функции / слушатели в родительской функции - PullRequest
0 голосов
/ 29 мая 2018
 How would I go about combining these scripts under one function/listener so that they do not interfere with one another.  The submit-button fadein/fadeout function should only fire if all inputs are valid or at least one input is invalid.

Так как коды стоят прямо сейчас, когда они запускают fadein / fadeout для кнопки review-submit, они ломаются после первого запуска любого из слушателей, переключающихся на основе отдельных входных значений вместо рассмотрения состояний всехиз входных данных.

 I am unsure of how to create a universal function that validates each input without using things such as 

$('#review-username')

Я также не уверен, с чего начать выяснять, как применять конкретные эффекты проверки к конкретным входам (электронная почта против имени пользователя против пароля).Каждый сценарий, который я написал ниже, работает сам по себе, проблема в том, что когда все сценарии находятся на одной странице, эффекты fadein / fadeout кнопки обзора-отправки прерываются.

Сценарий проверки 1

$('#review-username').on('input', function() {
           $('#review-username').addClass('review-invalid');     

        if (!/^[A-Za-z0-9]*$/g.test(this.value)) {       
          this.focus();
          return false; 
          this.setCustomValidity('Invalid Characters, Please Use Only AlphaNumeric Characters');
                $('#review-username').removeClass('review-valid'); 
           $('#review-username').addClass('review-invalid');        
        }

          if(this.value.length >= 5) {
            $('#review-submit-button').fadeIn(2250);   
        this.setCustomValidity('');
         $('#review-username').addClass('review-valid');
          $('#review-username').removeClass('review-invalid'); 
          $('#review-submit-button').removeClass('preventclick');
        }


     if(this.value.length < 5) {
$('#review-submit-button').fadeOut(1250);     
                $('#review-username').removeClass('review-valid'); 
           $('#review-username').addClass('review-invalid');
          $('#review-submit-button').addClass('preventclick');
        }


                  if(this.value.length > 16) {
                  $('#review-submit-button').fadeOut(1250);     
                $('#review-username').removeClass('review-valid'); 
           $('#review-username').addClass('review-invalid');
          $('#review-submit-button').addClass('preventclick');
        }

});

Сценарий проверки 2

$('#review-jobtitle').on('keyup', function(k) {
    var val = $(this).val();
   if (val.match(/[^a-zA-Z\s]/g)) {
       $(this).val(val.replace(/[^a-zA-Z\s]/g, ''));
   }
});


$('#review-jobtitle').on('input', function() {   
          $('#review-jobtitle').addClass('review-valid');  
 $('#review-jobtitle').addClass('invalid');
     if (!/^[a-zA-Z\s]*$/g.test(this.value)) {
         $('#review-jobtitle').removeClass('review-invalid');
        $('#review-jobtitle').addClass('review-valid');
        }else{           

     if(this.value.length >= 3) {
$('#review-submit-button').fadeIn(2250);
  $('#review-jobtitle').removeClass('review-invalid');
 $('#review-jobtitle').addClass('review-valid');
            $('#review-submit-button').removeClass('preventclick');
          }else{
                    $('#review-jobtitle').removeClass('review-valid');
          $('#review-jobtitle').addClass('review-invalid');
          }
        }

     if(this.value.length < 3) {
$('#review-submit-button').fadeOut(1250);
                 $('#review-jobtitle').removeClass('review-valid');
 $('#review-jobtitle').addClass('review-invalid');
           $('#review-submit-button').addClass('preventclick');
        }else{
          $('#jreview-obtitle').removeClass('review-invalid');
 $('#review-jobtitle').addClass('review-valid');
        }

         if(this.value.length > 32) {
         $('#review-submit-button').fadeOut(1250);
                 $('#review-jobtitle').removeClass('review-valid');
 $('#review-jobtitle').addClass('review-invalid');
           $('#review-submit-button').addClass('preventclick');
        }
});

Сценарий проверки 3

$('#review-email').on('input', function() {
  $('#review-email').addClass('review-invalid');     
     if(this.value.length >= 6) {

    var mailformat = /^\w+([\.-]?\w+)*@\w+([\.-]?\w+)*(\.\w{2,3})+$/;  
if(this.value.match(mailformat))  

 {  
$('#review-submit-button').fadeIn(2250);
  $('#review-email').removeClass('review-invalid');
 $('#review-email').addClass('review-valid');

 $('#review-submit-button').removeClass('preventclick');

}else{
$('#review-submit-button').fadeOut(1250);
  $('#review-email').removeClass('review-valid');
 $('#review-email').addClass('review-invalid');

 $('#review-submit-button').addClass('preventclick');
}

        }

     if(this.value.length < 6) {
$('#review-submit-button').fadeOut(1250);
  $('#review-email').removeClass('review-valid');
 $('#review-email').addClass('invalid');

 $('#review-submit-button').addClass('preventclick');
        }
           if(this.value.length > 64) {
           $('#review-submit-button').fadeOut(1250);
  $('#review-email').removeClass('review-valid');
 $('#review-email').addClass('review-invalid');

 $('#review-submit-button').addClass('preventclick');
        }
});

1 Ответ

0 голосов
/ 29 мая 2018

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

function determineButtonState(){
    var bothInputsAreValid = // logic for validation
    if(bothInputsAreValid) // handle case for valid inputs
    else // handle case for invalid inputs
}

$('#review-email').on('input', function() {
    ...
    determineButtonState()
})

В общем, это хороший подход.Рефакторинг общих частей кода в единый, многократно используемый источник.Я не собираюсь чистить весь ваш код и рассказывать вам точно, что нужно сделать, но этот подход должен дать вам то, что вам нужно.

...