Как сделать один элемент Node массивом?

Question

[Станислав Шабалин]

Добрый день всем.

Вопрос по переменной relatedList, которая содержит массив элементов:
Если в функцию relatedNavUpdate() передать конкретный элемент, то как правильно его преобразовать в массив?

function relatedNavUpdate(el = null) {
		const relatedList = (el == null) ? main.querySelectorAll('.products-wrapper') : el;

		relatedList.forEach( (e) => {
		});
}

Answer 1

[Stockholm Syndrome]

const relatedList = el ? [el] : main.querySelectorAll('.products-wrapper');

Comments

[Станислав Шабалин]

Да, все работает. Буду знать ) Спасибо

[jiggz]

Из-за того, что relatedList при разных el будет разного типа - либо массив, либо NodeList, вам будут недоступны некоторые полезные методы для работы с массивами в случае с NodeList. Потому желательно еще вот так сделать
[...main.querySelectorAll('.products-wrapper')]

[Станислав Шабалин]

Ngorso, по сути вся функция отвечающая за отображение стрелок у списков товаров вот такая:

function relatedNavUpdate(el = null) {

		const relatedList = el ? [el] : main.querySelectorAll('.products-wrapper');
		relatedList.forEach( (e) => {
			var ul = e.querySelector('ul.products'),
				leftArrow = e.querySelector(".arrow-left"),
				rightArrow = e.querySelector(".arrow-right");

			if ( ul && ul.scrollLeft < 10 )
				leftArrow.classList.contains('visible') && leftArrow.classList.remove('visible');
			else
				!leftArrow.classList.contains('visible') && leftArrow.classList.add('visible');

			if ( ul && ul.scrollWidth - ul.scrollLeft - ul.clientWidth < 10 )
				rightArrow.classList.contains('visible') && rightArrow.classList.remove('visible');
			else
				!rightArrow.classList.contains('visible') && rightArrow.classList.add('visible');
		});

	}

Не знаю, надо ли это делать, но логика определенно есть. Могу добавить, если это не в нагрузку и не в холостую

[jiggz]

Станислав Шабалин, ну, если не планируется как-то иначе манипулировать элементами, а нужен просто перебор, то можно и без трансформации в массив. Но если бы вам необходимо было еще использовать методы filter map reduce pop и тд, тогда да. Я всегда по привычке конвертирую для однообразия :)

P.S.
:D :D Только что обнаружил возможность запускать функции прямо в параметрах. Не совсем ваш кейс, но фича забавная.

function relatedNavUpdate(el = main.querySelectorAll('.products-wrapper') {

    el.forEach( (e) => {
      var ul = e.querySelector('ul.products'),
 ...

[Станислав Шабалин]

Ngorso, На будущее если только. Нюанс очень важный и ответ легко не найти, если не профи в нативном JS