@kivinus1
учусь

Как сделать перебор NodeList?

После нажатия на кнопку класс меняется у 1,3,5ого элементов. После второго нажатия добавляются 2 и 6. После третьего только меняется элемент под номером 4. Почему не меняются все элементы по первому нажатию?
Пожалуйста, скажите где ошибка. Я много раз перепроверил
JS
var bn = document.getElementById("button1");
	bn.addEventListener("click",changeColor);

	var str = document.getElementsByClassName('test');
	console.log(str);

	function changeColor(){
	for(var i=0;i<str.length;i+=1){
	   str[i].className=("color1");}
	}

HTML
<div class="test">
			<p>change my color1</p>
		</div>
		<div class="test">
			<p>change my color2</p>
		</div>
		<div class="test">
			<p>change my color3</p>
		</div>
		<div class="test">
			<p>change my color4</p>
		</div>
		<div class="test">
			<p>change my color5</p>
		</div>
		<div class="test">
			<p>change my color6</p>
		</div>
  • Вопрос задан
  • 81 просмотр
Решения вопроса 3
Seasle
@Seasle Куратор тега JavaScript
Помог - отметь решением \( ゚ヮ゚)/
let btn = document.querySelector('#button1');
let elements = document.querySelectorAll('.test');

btn.addEventListener('click', changeColor);

function changeColor() {
	for (let element of elements) {
		elements.classList.toggle('color1');
	}
}
Ответ написан
Fragster
@Fragster
помогло? отметь решением!
getElementsBy* возвращает динамическую коллекцию, поиск происходит каждый раз при обращении к её своствам. Поскольку сама коллекция изменяется при назначении класса (пропадает первый элемент), то приувеличении i обработка идет через один.
Нужно либо не изменять i: https://jsfiddle.net/07br1g2j/ или воспользоваться циклом while https://jsfiddle.net/07br1g2j/1/
либо добавлять класс вместо переписывания: https://jsfiddle.net/534pn0by/10/
либо преобразовать nodelist в массив перед обработкой https://jsfiddle.net/534pn0by/11/
либо воспользоваться методом, который возвращает "статическую коллекцию" https://jsfiddle.net/xrypz7v1/
Ответ написан
@dimoff66
Кратко о себе: Я есть
Потому что возвращается коллекция, из которой динамически удаляются данные когда элемент больше не соответствует выставленному отбору. Зафиксируйте значения в массиве, тогда будет работать.

var str = Array.from(document.getElementsByClassName('test'))
Ответ написан
Пригласить эксперта
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы