Есть ли смысл удалять обработчик событий document после закрытия попапа?

Question

[cepgej]

Представим, что есть веб-страница, на которой иногда выскакивает попап. Он должен закрываться при нажатии Esc и при клике на непопапную область страницы. Попап появляется/закрывается при помощи функций jQuery show()/hide(). На document вешаются обработчики:

$(document).on("keydown", function(e) {
	if (e.keyCode == 27) {
		$("#popup").closePopup();
	}
});
$(document).on("click", function() {
	$("#popup").closePopup();
});

Есть ли смысл после закрытия попапа выполнять

$(document).off("keydown");
$(document).off("click");

т.е. удалять обработчики, чтоб у элемента #popup впустую не вызывался hide()?

Answer 1

[Евгений Петров]

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

Даже если ставить/снимать, делать нужно примерно так

var popup = $('#popup');

function openPopup() {
	...
	$(document)
		.on('keydown', closePopup)
		.on('click', closePopup);
}

function closePopup() {
	popup.closePopup();
	$(document)
		.off('keydown', closePopup)
		.off('click', closePopup);
}

Comments

[cepgej]

Наличие попапа так проверять?
$("#okpopup").is(":visible")

[Евгений Петров]

@cepgej именно так. Я же правильно понимаю, что вы просто скрываете попап?

[cepgej]

Да!

[Евгений Петров]

@cepgej можно тогда и так

var popup = $('#popup');

$(document)
	.on('keydown', closePopup)
	.on('click', closePopup);

function closePopup() {
	if (popup.is(':visible')) {
		popup.closePopup();
	}
}

[Евгений Петров]

@cepgej и? Тишина? Вопрос-то решён?

[cepgej]

Да, благодарю.