Почему функция работает не так?

Question

[Даниил Сухих]

Есть функция для модальных окон, но он работает только для последнего вызова. В чём проблема?

function popUp(elem, open, close) {
	body = document.querySelector('body');
	close = document.querySelector(close);
	open = document.querySelector(open);
	elem_obj = document.querySelector(elem);

	if (close != '') {
		close.addEventListener('click', function() {
			fadeOut(elem);
			body.style.overflow = 'visible';
		}, false);
	}
	

	if (open != '') {
		open.addEventListener('click', function() {
			fadeIn(elem);
			body.style.overflow = 'hidden';
		}, false);
	}

	body.addEventListener('click', function(e) {
		display = window.getComputedStyle(elem_obj).display;
		console.log(elem_obj); // В КОНСОЛИ ВОЗВРАЩЕТЬСЯ ТОЛЬКО ПОСЛЕДНИЙ ВЫЗВАНЫЙ ЭЛЕМЕНТ, ДВА РАЗА
		if (!e.target.closest(elem + ' > *') && e.target != open && display != 'none') {
			fadeOut(elem);
			body.style.overflow = 'visible';
		}
	}, false);
}

popUp('.call-form__wrapper', '.main-section__slogan__button', '.call-form__close');
popUp('.promo-popup__wrapper', '.promo-info', '.promo-popup__close');

Answer 1

[0xD34F]

Потому что кое-кто не в курсе, что бывает, когда выполняется установка значения необъявленной переменной.

Если переменной нет, она будет искаться во внешних областях видимости. И если ни в одной из внешних областей видимости переменной с таким именем нет, будет создана глобальная переменная. То есть, у вас при первом вызове popUp создаётся глобальная переменная elem_obj, а при последующих вызовах обновляется её значение.

Comments

[Kovalsky]

А еще пишет в блокноте вместо того чтоб IDE пользоваться

[Даниил Сухих]

А когда я пишу переменные тут это не объявление?

function popUp(elem, open, close) {
}

[abberati]

danchikraw, это параметры, а не переменные