0

подскажите пожалуйста как избавиться от дублирующего кода в jq-плагине на странице

проблема в том, что метод step() содержит 2 похожих участка кода:

$(elem).closest('fieldset').addClass('hide').next().removeClass('hide');

$(elem).closest('fieldset').addClass('hide').prev().removeClass('hide');

хотелось бы чтобы аргумент className использовался непосредственно в цепочке методов. соответственно при этом количество кода сократилось бы в 2 раза. возможно ли такое?

2 ответа 2

1
var $fieldset = $(elem).closest('fieldset').addClass('hide');
$fieldset.next().add($fieldset.prev()).removeClass('hide');

Можно так. Кода меньше не стало, но так правильней.

$(elem).closest('fieldset').addClass('hide')
.next().removeClass('hide')
.end().prev().removeClass('hide');

Или так.

0

Разве что так:

var obj = $(elem).closest('fieldset').addClass('hide');
if(className == 'next'){
   obj.next().removeClass('hide');
}
else{
   obj.prev().removeClass('hide');
}

можно, конечно, вытащить и функцию по className, но ИМХО не стоит тут изощряться - смысл?

4
  • Если имелось ввиду это то можно сделать так: $(elem).closest('fieldset').addClass('hide')[className]().removeClass('hide');
    – toliklunev
    15 мая 2013 в 17:53
  • ну да, ну да, javascript он такой - гибкий. Тока надо будет проверить, что [className] не undefined, по хорошему
    – asianirish
    15 мая 2013 в 18:00
  • спасибо. вот это решение понравилось $(elem).closest('fieldset').addClass('hide')[[className]]().removeClass('hide'); а как называется это явление? почитать собираюсь на эту тему..
    – cyklop77
    16 мая 2013 в 4:39
  • className этож просто переменная, второе решение всяко правильнее
    – zb'
    16 мая 2013 в 4:43

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.