Почему цикл внутри функции даже не запускается?

Question

[Максим]

Привет! Потихоньку изучаю JS. Не могу понять, почему этот код даже не запускается.
Это будущий акордеон. Не отрабатывает код который перед открытием одного айтема бежит по всем айтемам акордеона и все закрывает.

function acordeonOpen() {
  //Этот цикл даже не зпускается, почему?
  for (let i = 0; i < acordeonItems[i].length; i++) {
    acordeonItems[i]
      .querySelector('.acordeon__item-body')
      .style
      .height = '0px';
  }
  //хотя в то же время этот код запускается
  this
    .parentNode
    .querySelector('.acordeon__item-body')
    .style
    .height = this.parentNode.querySelector('.acordeon__item-body').getAttribute('data-acordeon-height');
}

Вот кодпен.
Спасибо!

Comments

[Сергей Сергей]

i < acordeonItems.length

[Максим]

Сергей Сергей, Да, вы правильно подметили. Но это всёравно не помогает. Видимо где-то ещё накосячил)

[Сергей Сергей]

нормально запускается.

[Максим]

Сергей Сергей, да, я просто "очень внимательный") Спасибо большое)

Answer 1

[Кира]

i < acordeonItems.length

acordeonItems[i] это li

Comments

[Максим]

Кира Да, вы правильно подметили. Но это всёравно не помогает. Видимо где-то ещё накосячил)

[Кира]

Максим, у вас сейчас в codepen acordeonItems[i].length, должно быть acordeonItems.length, тогда вроде работает как надо.

[Максим]

Кира, ёёмаёёё. Спасибо большое :)

Answer 2

[Владимир Проскурин]

Нужно смотреть, что у вас в acordeonItems. Как вы его получаете? Может это не массив совсем.
Смотрите пример, где acordeonItems это список элементов. jsfiddle.net/eocjkx3v все работает. (ну, если еще изменить acordeonItems[i] на acordeonItems)

Comments

[Максим]

Владимир Проскурин Спасибо за уделённое время :)
Нашли ошибку, по невнимательности неправильно написал условие цикла :)