1

В общем есть такой вот Jquery код

//Добавление нового пункта
$('.SumoSelect').on('click', '.newSumo', function(){
    newopt = $(this).parents(".SumoSelect").find('.search-txt').val();
    //Добавляем новый параметр
    $(this).parents(".SumoSelect").find("select")[0].sumo.add(newopt);
    //Убираем все выбранные
    $(this).parents(".SumoSelect").find("select")[0].sumo.unSelectAll();
    //Выбираем новую опцию
    $(this).parents(".SumoSelect").find("select")[0].sumo.selectItem(newopt);
    //Перезагружаем блок
    $(this).parents(".SumoSelect").find("select")[0].sumo.reload();
});

Как видно я постоянно обращаюсь к одному и тому же месту через $(this).parents(".SumoSelect").find("select").

Вопрос. Как правильно это сократить?

3
  • var el = $(this).parents(".SumoSelect").find("select");
    – u_mulder
    1 мая 2016 в 13:03
  • Если вам дан исчерпывающий ответ, отметьте его как верный (галка напротив выбранного ответа). 1 мая 2016 в 14:10
  • Сам сейчас занимаюсь изучением Wordpress, вот неплохая статья все очень грамотно описано https://habrahabr.ru/post/228523/ 9 окт 2016 в 11:39

1 ответ 1

3

Сохраните повторяющуюся часть в переменную:

//Добавление нового пункта
$('.SumoSelect').on('click', '.newSumo', function(){
    newopt = $(this).parents(".SumoSelect").find('.search-txt').val();

    var sumo = $(this).parents(".SumoSelect").find("select")[0].sumo;

    //Добавляем новый параметр
    sumo.add(newopt);
    //Убираем все выбранные
    sumo.unSelectAll();
    //Выбираем новую опцию
    sumo.selectItem(newopt);
    //Перезагружаем блок
    sumo.reload();
});
1
  • Благодарю. Работает 1 мая 2016 в 13:41

Ваш ответ

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

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