Как правильно передать значение свойства в слайдер?

Question

[Fehax]

Добрый всем вечер. Моя функция возвращает undefined, соответственно в значении вписывает подобное. В принципе из-за этого весь код и не работает...

Как решить эту задачу?

codepen

Comments

[Саман]

Что именно вы пытаетесь получить? Откуда получить? Можно чуть подробнее

[Fehax]

Саман, в папке изображения у меня, они перечислины как элементы в массиве slides

[Fehax]

Могу гит сделать, чтобы и изображения подтянуть

[Fehax]

проблема в выделенном жёлтым фрагментом.
честно говоря, я даже не понимаю, как это фрагмент работает... как функция получает этот параметр image

[Саман]

Проблема скорее всего в путях.Попробуйте через консоль вывести эти изображения

[Fehax]

Саман, как видим, в этом и проблема, но изображение есть, оно выводится...

Answer 1

[0xD34F]

if (this.frame < 0) {
    this.frame = this.slides.lenght-1;
    this.setBackground(this.slides[this.frame])
}

Если frame неотрицательный, то вызывать setBackground уже не надо?

Что такое lenght? Я такого слова не знаю.

И массивы не знают. Так что получаете undefined. А после вычитания единицы - NaN. Элемента с индексом NaN в массиве нет, так что в setBackground улетает undefined - его вы и видите в стилях.

if (this.frame > this.slides.length) {

По-вашему, у массива может быть элемент с индексом, равным длине?

buttonLeft.addEventListener('click', slider.left());

Функция и результата её вызова - не одно и то же.

Comments

[Fehax]

Я новичок, потому бывает путают методы.. Бывает
Спасибо, что помогли разобраться. Вы крутой! Не откажусь от хорошего совета по текущему вопросу в целом для новичка, буду признателен! Заработал пример

if (this.frame > this.slides.length) {

там должно быть равно. очередная невнимательность..

buttonLeft.addEventListener('click', slider.left());

Была задумка изначально написать на onclick, потом получил кнопки, скопировал, а скобки так и не удалил, невнимателен