Pure JavaScript Carousel, OOP. Как выбрать получить текущий слайд?

Question

[Unknown]

Привет!
Осваиваю ООП на примере слайдера.
Я завис с получением текущего слайда с помощью метода ForEach.
Код на codepan.

Slider.prototype.currentSlide = function() {
...
};

там будет вот такой блок

Этот метод должен получать номер текущего слайда.
Всем спасибо за помощь!

Answer 1

[Дмитрий Лузанов]

Ну во-первых, нужно перенести определение слайдов и номер текущего слайда в сам прототип. Получать номер текущего слайда нет смысла. Лучше наоборот. Хранить номер и по надобности получать сам слайд. К тому же для поставленной вами задачи нет никакого материала. У вас нет текущего слайда.

(function Slider() {
  function Slider(slides, startIndex = 0) {
    this.slides = slides
    this.currentSlideIndex = startIndex
  }
  
  Slider.prototype.getCurrentSlide = function() {
    return this.slides[this.currentSlideIndex]
  }
  
  return Slider
})()

const slider = new Slider(document.querySelectorAll(".slider-item"))

slider.getCurrentSlide()

Comments

[Unknown]

А почему вы обернули

function Slider(slides, startIndex = 0) {
    this.slides = slides
    this.currentSlideIndex = startIndex
  }

в самовызывающуюся функцию и назвали ее также как и сам клас

[Дмитрий Лузанов]

Richard, для замыкания. Так есть возможность использовать "приватные" переменные, к которым получить доступ из вне невозможно.

(function Slider() {
  const private = 'any'
  Slider.staticValue = 'any'
  Slider.prototype.instanceValue = 'any'

  function Slider(slides, startIndex = 0) {
    this.slides = slides
    this.currentSlideIndex = startIndex
  }
  
  Slider.prototype.getCurrentSlide = function() {
    return this.slides[this.currentSlideIndex]
  }
  
  return Slider
})()

[Unknown]

Predve4niy, ааа, вы бы рекомендовали делать это всегда?

[Дмитрий Лузанов]

Richard, Вам не придется использовать такой подход к "классам" в реальных проектах, т.к. сейчас используют ES6. В этой спецификации есть обертка над классами. А в плане обучения нет причин не познакомиться.

[Unknown]

Predve4niy, Спасибо!