Кнопка "назад" в слайдере

[hitori]

Приветствую, я не знаю если у кого-то будет желание сидеть и разбираться в моем *****коде, но я буду очень признателен за любую помощь. Сам слайдер реализован на css, jquery там просто нужен для автоматического переключения слайдов. Кнопка вперед работает нормально, но я не могу понять почему не работает кнопка "назад" ведь логически там вроде все верно, компиляторы тоже не выдают ошибок (вперед\назад это эти 2 стрелочки по бокам картинки, которые перелистывают слайд).

Слайдер я аккуратно выдернул и поместил сюда - http://codepen.io/anon/pen/xjrtE . Javascript составляющую я прокомментировал в коде. Буду рад любым предложениям по правке\оптимизации кода, потому что я сам туго представляю как работают слайдеры, да и большого опыта в javascript'e у меня тоже нет.

[Switch74]

Не пойму зачем вы берете количество слайдов

var counter = slides.length;

да, он показывает правильно 5

 

краткое описание вашего for в кнопке назад

for (counter; counter <= 0; counter--) 

предположим

counter =  5

тогда теоретически у нас есть возможность переместиться на слайд под номером 4

но в вашем цикле стоит условие counter <= 0 - т.е. если 5 <= 0, то мы counter--

естесно условие выполняться не будет НИКОГДА!

Edited September 26, 2014 by Switch74

[hitori]

Не пойму зачем вы берете количество слайдов

var counter = slides.length;

да, он показывает правильно 5

 

краткое описание вашего for в кнопке назад

for (counter; counter <= 0; counter--) 

предположим

counter =  5

тогда теоретически у нас есть возможность переместиться на слайд под номером 4

но в вашем цикле стоит условие counter <= 0 - т.е. если 5 <= 0, то мы counter--

естесно условие выполняться не будет НИКОГДА!

я беру кол-во слайдов, потому что кол-во слайдов может быть разным. А скрипт работает так:

-прогоняется по всем инпутам, которые отвечают за переключение сайдов

-смотрит какой по счету инпут чекнут

-снимает с чекнутого инпута св-во checked и присваивает его следующему\предыдущему (зависит от функции)

 

спасибо большое за помощь, сейчас все работает