Как оптимизировать этот JQ?

Question

[Санек]

// sub menu lvl1 first 
  $('.sub-menu_wrapp.first .sub-menu_lvl1 li a').mouseenter(function () {
    $('.sub-menu_wrapp.first .sub-menu_lvl1 li a.active').removeClass("active");
    if ($(this).hasClass("active")) {
      $(this).removeClass("active");
    } else {
      $(this).addClass("active");
    }
  });
  // sub-menu lvl2
  $('.sub-menu_wrapp.first .sub-menu_lvl1 li a').mouseenter(function() {
    $('.sub-menu_wrapp.first .sub-menu_lvl2').removeClass('visible');
    $('.sub-menu_wrapp.first .sub-menu_lvl2.' + $(this).data('visible')).addClass('visible');
  });

  // sub menu lvl1 second 
  $('.sub-menu_wrapp.second .sub-menu_lvl1 li a').mouseenter(function () {
    $('.sub-menu_wrapp.second .sub-menu_lvl1 li a.active').removeClass("active");
    if ($(this).hasClass("active")) {
      $(this).removeClass("active");
    } else {
      $(this).addClass("active");
    }
  });
  // sub-menu lvl2
  $('.sub-menu_wrapp.second .sub-menu_lvl1 li a').mouseenter(function() {
    $('.sub-menu_wrapp.second .sub-menu_lvl2').removeClass('visible');
    $('.sub-menu_wrapp.second .sub-menu_lvl2.' + $(this).data('visible')).addClass('visible');
  });

// sub menu lvl1 third 
  $('.sub-menu_wrapp.third .sub-menu_lvl1 li a').mouseenter(function () {
    $('.sub-menu_wrapp.third .sub-menu_lvl1 li a.active').removeClass("active");
    if ($(this).hasClass("active")) {
      $(this).removeClass("active");
    } else {
      $(this).addClass("active");
    }
  });
  // sub-menu lvl2
  $('.sub-menu_wrapp.third .sub-menu_lvl1 li a').mouseenter(function() {
    $('.sub-menu_wrapp.third .sub-menu_lvl2').removeClass('visible');
    $('.sub-menu_wrapp.third .sub-menu_lvl2.' + $(this).data('visible')).addClass('visible');
  });

// sub menu lvl1 fourth 
  $('.sub-menu_wrapp.fourth .sub-menu_lvl1 li a').mouseenter(function () {
    $('.sub-menu_wrapp.fourth .sub-menu_lvl1 li a.active').removeClass("active");
    if ($(this).hasClass("active")) {
      $(this).removeClass("active");
    } else {
      $(this).addClass("active");
    }
  });
  // sub-menu lvl2
  $('.sub-menu_wrapp.fourth .sub-menu_lvl1 li a').mouseenter(function() {
    $('.sub-menu_wrapp.fourth .sub-menu_lvl2').removeClass('visible');
    $('.sub-menu_wrapp.fourth .sub-menu_lvl2.' + $(this).data('visible')).addClass('visible');
  });

Comments

[Stalker_RED]

Приложите еще и html-код, а лучше соберите пример на jsfiddle.
Скорее всего можно все переписать вообще строк в пять.

сходу могу подсказать что

if ($(this).hasClass("active")) {
      $(this).removeClass("active");
    } else {
      $(this).addClass("active");
    }

можно заменить на $(this).toggleClass("active");

[キム ファイブプラス]

Инкапсуляция, не? У вас в данном случаем вызываются одни и те же методы jquery. Ну так запишите их в объект/функцию. Или что вы тут собрались оптимизировать?

Answer 1

[Станислав Шендаков]

Первое что увидел на замену это:
...
if ($(this).hasClass("active")) {
$(this).removeClass("active");
} else {
$(this).addClass("active");
}
...
можно ( IMHO нужно ) заменить на
...
$(this).toggleClass("active"))
...
Дальше не анализировал