Как минимизировать нагрузку от JQuery проверок setInterval?

Question

[dima_maketov]

Добрый день!
Подскажите как оптимизировать этот код? Написал этот код в качестве проверок для формы и верхнего меню. Но кажется, что он очень убог и имеет сильную нагрузку на устройство(особенно смартфоны). Как минимизировать энергозатраты на проверки или минимизировать их?

setInterval(
			function(){
        if($('#main__background .phone').val() != '' && $('#main__background .phone').val().search('_') == -1){
          $('.success-get-top').prop('disabled', false);
        }else{
          $('.success-get-top').prop('disabled', true);
        }
        if($(window).width() <= 415){
          if($(window).scrollTop() == 0){
            if($('.bot').hasClass('hide')){
              $('.bot').removeClass('hide');
            }
            if($('#main__background').hasClass('up')){
              $('#main__background').removeClass('up');
            }
          }
          else if($(window).scrollTop() != 60){
            $('.bot').addClass('hide');
            $('#header').addClass('up');
            $('#main__background').addClass('up');
          }
			}
    }, 350);

Answer 1

[Петр]

Для начала хотябы уберите двойные запросы по селектору.

if($('.bot').hasClass('hide')){
   $('.bot').removeClass('hide');
}

Достаточно
$('.bot').removeClass('hide');

Это

if($('#main__background .phone').val() != '' && $('#main__background .phone').val().search('_') == -1)

Замените на

var phvl = $('#main__background .phone').val();
if (phvl != '' && phvl.search('_') == -1)

Это

var w = $(window);
if(w.scrollTop() != 60)

Comments

[dima_maketov]

Спасибо.
Переменную phvl локально прописать или глобально?

[Петр]

dima_maketov: Ее нужно использовать тогда, когда у вас будет необходимость в значении данного элемента.

Answer 2

[Stepanya]

Перепишите без использования setInterval.
Это вообще выносит в onResize

if($(window).width() <= 415){
if($(window).scrollTop() == 0){
if($('.bot').hasClass('hide')){
$('.bot').removeClass('hide');
}
if($('#main__background').hasClass('up')){
$('#main__background').removeClass('up');
}
}
else if($(window).scrollTop() != 60){
$('.bot').addClass('hide');
$('#header').addClass('up');
$('#main__background').addClass('up');
}