Jump to content
  • 0

jQuery each — где создавать обработчики


rash
 Share

Question

Здравствуйте.

 

Вопрос предельно нубский, но попробую не стесняться :)

 

Задача — после загрузки страницы обойти все элементы по селектору (в частности, с заданным классом) и инициализировать для них некоторое поведение.

 

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

 

Первый — обработчики создаются в замыкании для каждого экземпляра:

$('.class').each(function() {	var self = $(this);	var data = 1;	self.on('event1', 'child1', handler1)	    .on('event2', 'child2', handler2)	    .on('event3', 'child3', handler3);	function handler1() {		...	}	function handler2() {		...	}	function handler3() {		...	}});

Здесь при каждой итерации each в замыкании будет создаваться новый комплект обработчиков, из которых будут доступны данные data и сам элемент self просто из скоупа. Это упрощает доступ к данным, их сохранение и обмен между обработчиками.

 

Второй способ — обработчики создаются внутри внешнего замыкания:

(function() {	$('.class').each(function() {		var self = $(this);		var data = 1;		self.on('event1', 'child1', function() { handler1(self, data); })		    .on('event2', 'child2', function() { handler2(self, data); })		    .on('event3', 'child3', function() { handler3(self, data); });	});	function handler1(self, data) {		...	}	function handler2(self, data) {		...	}	function handler3(self, data) {		...	}}());

Здесь обработчики существуют в единственном экземпляре, но чтобы они могли оперировать общими данными, эти данные необходимо передавать в явном виде. Можно, конечно, привязать их к DOM-элементу, но его, тем не менее, передавать придется.

 

Вот и вопрос — стоит ли экономия на количестве обработчиков усложнения доступа к данным? Какой из вариантов выбрали бы вы?

Link to comment
Share on other sites

8 answers to this question

Recommended Posts

  • 0

если подумать то и цикл там не нужен.

что-то типо такого:

$(document).ready(function(){   var data = 'red';   $('.class')     .on('click', '.c_1', data, handler)     .on('click', '.c_2', data, handler)     .on('click', '.c_3', data, handler);});function handler(event) {    var $this = $(this);    var data = event.data;    $this.css('background-color', data);}

Обработчиков можно сделать 3 или больше, но можно и передавать часть функционала через объект data, он попадает в event.data , и ссылку на контекст тоже передавать не нужно.

 

На фидле запилил http://jsfiddle.net/H8hBp/

 

UPD забыл если нужен родитель .class то ведь опять же есть  parent()

Edited by wwt
  • Like 3
Link to comment
Share on other sites

  • 0

Цикл не нужен, просто так получается удобнее, и опять же за счет замыкания на общего родителя, которому делегирую обработчики, проще сохранять данные для каждого экземпляра отдельно (а это и нужно).

 

В вашем случае data получается общей для всех экземпляров.

 

Конечно, данные можно привязать к DOM-элементу через .data, но мне не хочется этого делать уже только потому, что этого можно не делать, воспользовавшись обычной переменной. Или у сохранения в DOM есть какие-то преимущества, которые я упускаю? Пока мне кажется, что это имеет смысл делать, если к данным должен быть доступ из разных блоков или частей документа, которые не смогут воспользоваться общей переменной.

 

Впрочем, ваш подход можно попробовать адаптировать к моему второму варианту. Что-то я упустил из виду, что при назначении обработчика можно данные передавать через event.data. Код немного увеличится, но по памяти должно получиться экономичнее все равно.

 

Свежий взгляд помогает :)

Link to comment
Share on other sites

  • 0

Если данных много то можно воспользоваться Local storage это ещё удобнее чем переменные, данные запоминаются на клиенте и доступ с любой страницы сайта будет. Или уж если переменная то как вариант использовать один глобальный объект для данных и обращаться к нему из любого места скрипта. Мне кажется так лучше чем кучу кода созданного для того же самого.

Link to comment
Share on other sites

  • 0

Штука в том, что данных немного, но они должны быть назависимыми для каждого экземпляра. Local storage ни к чему, они нужны только на время существования страницы и только на этой странице.

 

В общих чертах — «поведение» каждого блока требует хранения его состояния. И глобальный объект тоже ни к чему — они имеют смысл только внутри самого блока, к ним не нужен доступ извне. Но каждый блок должен вести себя независимо, то есть хранить свое собственное состояние отдельно от других.

Link to comment
Share on other sites

  • 0

Штука в том, что данных немного, но они должны быть назависимыми для каждого экземпляра. Local storage ни к чему, они нужны только на время существования страницы и только на этой странице.

 

В общих чертах — «поведение» каждого блока требует хранения его состояния. И глобальный объект тоже ни к чему — они имеют смысл только внутри самого блока, к ним не нужен доступ извне. Но каждый блок должен вести себя независимо, то есть хранить свое собственное состояние отдельно от других.

тогда data не плохой вариант. И получить доступ к данным можно из любого места скрипта. Чем городить огород с переменными и их передачей в обработчик.

Link to comment
Share on other sites

  • 0

Ну вот и я пришел к такому же выводу, просто, вероятно, когда-то бегло читал документацию по .on, и на data не обратил внимания, так и жил в уверенности, что такой возможности нет :)

Link to comment
Share on other sites

  • 0

Задача — что-то типа кастомных контролов. Соответственно, на странице их может встретиться довольно много (заранее неизвестно), и каждый должен вести себя самостоятельно. Все данные, которые от него могут понадобиться другим частям скрипта будут доступны из DOM-представления (все равно все делается на базе стандартных контролов, просто вокруг них навешивается дополнительная функциональность и логика), остаются те же value, onchange, onblur и добавляются некоторые jquery-события.

 

Совсем по-хорошему, наверное, надо плагин написать для этого. Но сейчас не планирую.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Answer this question...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

  • Similar Content

    • By jksnf
      Не работает transition при появлений модального окна:
      <a class="btn">Modal</a> <div class="modal"> <div class="overlay"> <div class="content"> <a class="close">X</a> <h1>Title</h1> <p>Lorem ipsum dolor sit amet consectetur adipisicing elit. Sunt repellat asperiores rerum animi officiis minus corporis iure voluptate accusantium, ut optio ratione iusto nemo delectus. Quasi illum libero dolorum neque?</p> </div> </div> </div> //scss .modal .overlay{ position: fixed; top: 0;left: 0; width: 100%;height: 100%; background: rgba($color: #000000, $alpha: .8); z-index: 1; display: none; } .modal .content{ width: 500px;height: 200px; position: absolute; top: 50%;left: 50%; transform: translate(-50%, -50%); background-color: #fff; z-index: 2; text-align: center; display: none; a{ line-height: 30px; font-weight: bold; color: #fff; top: -50px;; position: absolute; right: -50px; font-size: 40px; padding: 10px; } h1{ margin-bottom: 20px; } } .modal.active .overlay{ display: block; } .modal.active .content{ display: block; } //Jquery $(document).ready(function(){ $(".btn, .close").click(function() { $(".modal").toggleClass("active") }) }) Я понимаю что на display не работает transition. но и нельзя ставить opacity т.к перекрывает контент сзади него изначально.

      Что делать?
    • By IsayR
      Всем привет! Подскажите, пожалуйста, как правильно сделать раскрывающиеся блоки со скрытым содержанием внутри? У меня в принципе все работает, блок открывается и закрывается, НО если я нажимаю на блок, а не на стрелочку, она у меня остается в таком же положении(а она должна у меня меняться, когда блок открывается), и получается куча мала, и выглядит это не очень презентабельно. Подскажите, пожалуйста!
      <script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script> <div class="wrapper-div"> <div class="div"> <div class="window-one"> <h3 class="title"> Заголовок 1 </h3> <div class="arrow-test"></div> </div> <div class="window-two"> <div class="text">Текст 1</div> </div> </div> </div> <div class="wrapper-div"> <div class="div"> <div class="window-one"> <h3 class="title"> Заголовок 2 </h3> <div class="arrow-test"></div> </div> <div class="window-two"> <div class="text">Текст 2</div> </div> </div> </div> <div class="wrapper-div"> <div class="div"> <div class="window-one"> <h3 class="title"> Заголовок 3 </h3> <div class="arrow-test"></div> </div> <div class="window-two"> <div class="text">Текст 3</div> </div> </div> </div> .div{ border: 1px solid #3A444E; border-radius: 10px; width: 300px; height: 64px; overflow: hidden; } .div.open { height: auto; background: #000; color: #fff; } .window-one { display: flex; align-items: center; justify-content: space-between; } .title { margin-bottom: 20px; } .arrow-test { content: ''; background: url('http://cdn.onlinewebfonts.com/svg/img_387044.png') no-repeat center center / 100%; width: 30px; height: 30px; cursor: pointer; } .arrow-test.open { content: ''; background: url('https://upload.wikimedia.org/wikipedia/commons/thumb/d/dd/Eo_circle_yellow_arrow-up.svg/1024px-Eo_circle_yellow_arrow-up.svg.png') no-repeat center center / 100%; width: 30px; height: 30px; cursor: pointer; } $('.div, .arrow-test').click( function(){ $(this).toggleClass('open')});  
    • By Алеся
      Всем привет! Хотела бы представить свою работу новичка на обсуждение для получения конструктивной критики и/или советов, взглядов что можно улучшить или изменить, общее впечатление, конечно, тоже интересует.   https://alesia-k.github.io/Tajam-template/
      Обратите, пожалуйста, внимание на:
      - блок "MEET OUR AMAZING TEAM". Немного не нравится как смещаются item при наведении мыши, как будто немного дергаются. Так ли по-вашему это должно работать или такое поведение блока нужно изменить?
      - инпут "subscribe" в футторе. При клике на инпут, немного смещается и сам инпут, и рядом стоящая кнопка. Было бы классно, если бы подкинули идею как сделать, чтобы такого не было.
      - и почему-то при клике на <a href="tel:(+62)2122243333">(+62)21-2224 3333</a>   не идет процесса соединения.
      Заранее спасибо!
    • By Only091
      Помогите пожалуйста, не получается сделать постраничную навигацию. Делал все по урокам. в Итоге получилось сделать два разных каталога один с фильтрами другой с постраничной навигацией. И теперь я пытаюсь объединить два каталога. Но не получается. Сами файлы урока в архике каталог. Буду очень благодарен если мне помогут! catalog.phpcatalogDB.js
      каталог.7z
    • By Alexand
      Здравствуйте дорогие специалисты програмирования.
      Взываю вас о помощи!? Прощу Вас уделите мне своё безценное время. Я который раз выполняю тестовое задание для трудоустройства, а мне постоянно отказывают. Я уже в замешательстве, что же я делаю не так. Я просил их, дать мне хоть какую-то критику, единственное что мне отвечают СЛАБО. Но что именно слабо не говорят!
      Прошу Вас рассмотреть мою из последних тестовую работу и укозать мне ВСЕ мои прогрехи, по полной строгости. Единственная надежда на Вашу благосклонность и доброту.
      Вот ссылка на гитхаб - https:/github.com/ShusevA/ARI-AJAX-Application 
      Требоватие к этой работе https://docs.google.com/document/d/10gLPHDqGNMDSeSQbYfG8RVtXR3XQOrchSBmd50OzJ_s/edit
      С уважением к Вам, за ранее спасибо.
×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. See more about our Guidelines and Privacy Policy