Почему происходит бесконечный цикл?

Question

[Rapen]

Есть список инпутов которые проверяются на checked, по ним создается стек и создаются jsx элементы которые будут отрендерены,

Код

. При нажатии все зависает, и ничего не происходит, если убратьi--, то метод handleRemove почему-то удаляет не те todo элементы, которые нужно по-сути

let cb = document.getElementsByClassName('checkBoxes');
		 for(let i=0; i< cb.length; i++){
			 if(cb[i].checked == true){
                              currentState.splice(i,1);
			       i--;
			 }

Comments

[Анатолий Жаров]

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

[Антон Спирин]

Я настоятельно рекомендую вам удалить весь этот код и поступить так, как советовал вам в одном из предыдущих ваших вопросов, а именно посмотреть типовые Todo App на React на github и получше изучить документацию и возможности библиотеки. Только после этого приниматься писать свое приложение. Подход к реализации с учетом возможностей, которые дает вам React, неправильный.

Answer 1

[xmoonlight]

Зачем менять итеративную переменную два раза?
Нужно убавлять - измените условие внутри цикла на обратное.

Comments

[Rapen]

да, потому что на место удаленного ключа встает последующий ключ в массиве, его пропускать не стоит же вроде не?

[xmoonlight]

Rapen, Ну так не меняйте итеративную переменную. Используйте арифметическую операцию БЕЗ присваивания итеративной переменной!

Answer 2

[ixon]

Потому что вы возвращаете итератору его исходное значение. итератор даёт неверные значения из-за зоны видимости. Попробуйте передавать значение итератора в качестве функции.