Где мне найти силы, чтобы ловко справиться с каруселью на нативном джаваскрипте?

Question

[astmaks]

Всем привет! Бьюсь смертным боем вот уже сутки с первым проектом на нативном джаваскрипте. Есть 3 независимых слайдера, но работает нормально только 1, остальные два - сплошные туцнеядцы: всем своим видом указываю на то, чтобы я их оставил в покое. Буду безумно благодарен, если кто-нибудь сможет найти у себя немного свободного времени для этой заразы!! Спасибо огромное)

https://plnkr.co/edit/WcAymNgTY7Wb61lVQErr?p=preview

Comments

[Kovalsky]

У меня ни один не работает

[astmaks]

Никита Полевой: видимо из-за одинаковых "картинок" сложилось такое впечатление (в левом верхнем углу цифры меняются)

[Kovalsky]

astmaks: работает только первый слайдер, остальные как-то неявно опираются на текущее в первом значение. Простите, я код тщательно не инспектировал, но очевидно у вас либо переменная вылетела из своей области, либо вы сами по ошибке используете не те переменные. Обособьте скрипт слайдера в отдельную функцию, например. Я уверен, этого будет достаточно для того чтобы все слайдеры работали независимо.

Answer 1

[Константин Громов]

counter внутри showSlides является глобально переменной, ссылающейся на первый слайдер. Чтобы исправить передавайте соответствующий counter третьим аргументом: https://plnkr.co/edit/hM1mUp9ickx0oTwTDb7i?p=preview.

И как минимум пересмотрите своё решение в плане расширяемости. Сейчас каждый новый слайд требует тонну копипасты. Хотя должен инициализироваться с одной строки в js и контента слайдов в html.

Comments

[astmaks]

Спасибо за Вашу помощь! Может быть могли бы намекнуть, где можно подглядеть удачный пример расширяемости?

[Константин Громов]

astmaks: https://learn.javascript.ru/widgets