Правильно ли я описываю очередность выполнения кода JS?

Question

[Алексей Хаткевич]

Написал код сам не понимая как , когда писал всё понимал, вот стоило прочитать его, ничего не понял, суть такова, чтобы передавать значение x в другую функцию чтобы вызвать скролл на определенную высоту.

var tap = document.getElementsByClassName('tap');
for( let i = 0 ; i < tap.length ; i++){
		console.log('i = ' + i);
		tap[i].parentNode.addEventListener('click' , function(e){
			var self = e.target;
			var selfTap = tap[i];
				if(selfTap == self){
					var x = i;
					scroll(x);
	}});};

1)Я объявил переменную i и в цикле for, переменная i будет иметь 2 значения i=0 и i=1?
2)После создается 'якобы' 2 слушателя события, в котором tap[0] и tap[1]?
Если это не так, то объясните пожалуйста и не бейте палкой пожалуйста за код, если есть косяки в коде, то подскажите в каком направлении двигаться :)

Comments

[SagePtr]

Если не понимаете собственный код - возьмите и перепишите его понятным языком

[Василий Банников]

Ели я правильно понял ваше намерение, то вам нужно что-то такое:

const taps = document.getElementsByClassName('tap'); // берём все элементы  с классом tap
for (const tap of taps) { // Для каждого из них
    tap.parentNode.addEventListener('click', e => e.target.scrollIntoView()); // назначаем слушатель события click, который проскроллит к нему.
}

[Алексей Хаткевич]

SagePtr, Спасибо

[Алексей Хаткевич]

Василий Банников, Что-то он ругается мне на 'taps':)

[Василий Банников]

Алексей Хаткевич, а как ругается то?

[Алексей Хаткевич]

Василий Банников, Простите меня, я только недавно начал изучать :)
Uncaught ReferenceError: taps is not defined at window.onload

[Василий Банников]

Алексей Хаткевич, это по тому что копипастить не надо )

[Алексей Хаткевич]

Василий Банников, :)))))))

[Василий Банников]

Ну и зачем вы вопрос в бессмыслицу превратили?

Answer 1

[Tendor]

Зачем эта проверка?
if(selfTap == self){
var x = i;
scroll(x);
}
Что зачем функции scroll передавать х?

Comments

[Алексей Хаткевич]

function scroll(x) {
var win = ($(window).height() + 1) * ++x;
$('html, body').animate({scrollTop: win}, 800);
return false;
};

[Василий Банников]

Алексей Хаткевич, неадекватный код какой-то.

[Алексей Хаткевич]

Василий Банников, Какой придумал :)

[Василий Банников]

Алексей Хаткевич, Что вообще эта строка делать должна?
($(window).height() + 1) * ++x;

[Алексей Хаткевич]

Василий Банников, ($(window).height(), тут я получаю значение высоты экрана пользователя.
+1 написал, потому что при скролле, сверху оставался 1px черной полоской.
++x, я передаю значение с другой функции, так как таких копок на странице много, и каждая кнопка должна быть на +1 больше другой, чтобы скрол срабатывал на высоту экрана. Если без X, то кнопка след кнопка не работает :(

Answer 2

[Алексей Хаткевич]

Вы правы, эта проверка тут не нужна, спасибо :)
(Х) - нужна для этой функции
function scroll(x) {
var win = ($(window).height() + 1) * ++x;
$('html, body').animate({scrollTop: win}, 800);
return false;
};