Почему переменная приобретает не правильное значение?

Question

[DenKuzmin17]

Если нажать, например
<div class="prev" data-id="1">2</div>, а потом
<div class="slider-btn next">&gt;</div>
слайдер сдвигается на последнее изображение, вместо следующего. Почему-то "currentImg" приобретает значение "maxImages - 1", хотя указано "Math.min".
Если просто нажимать
<div class="slider-btn next">&gt;</div>
не трогая ".prev", всё работает как надо.

Как сделать, чтобы после нажатия ".prev", а затем ".next" "currentImg" приобретал значение следующего слайда, а не последнего?

Answer 1

[0xD34F]

В currentImg лежит строка. Соответственно, и при сложении тоже строка получается. То есть, номер слайда, к которому вы пытаетесь перейти, вместо 2 оказывается 11.

В качестве заплатки на скорую руку можно преобразовывать значение к числу:

this.currentImg = Math.min(+this.currentImg + 1, this.maxImages - 1);

Ну а по-хорошему - вам следует разобраться, почему там строка вместо числа, и исправить это.

Comments

[DenKuzmin17]

Понял. Спасибо большое. После нажатия ".prev" "currentImg" становится равным "dataset.id". Видимо "dataset.id" это строка. Буде исправлять. Благодарю за развёрнутый ответ.

[0xD34F]

DenKuzmin17, так точно. В dataset хранятся строки. Преобразовывайте к числу, когда достаёте оттуда номер слайда.

[DenKuzmin17]

поменял:

this.sliderPoint.on('click', e => {
      this.currentImg = e.target.dataset.id;
      this.scrollImages(this.currentImg, this.speed);
    });

на

this.sliderPoint.on('click', e => {
      this.currentImg = parseInt(e.target.dataset.id);
      this.scrollImages(this.currentImg, this.speed);
    });

работает правильно.