Как правильно организовать работу с условием if?

Question

[Артем Шеленговский]

У меня есть меню которое я хочу листать вверх вниз с помощью jquery.
Смысл в том, что как только количество скрытых пунктов меню станет равным нулю (down_clix === 0) нужно сделать кнопку вниз не активной, нажимая кнопку вверх значение (down_clix) должно увеличиваться (что оно и делает), но кнопка вниз активной для щелчка по ней не становится. Я думаю ошибка в постановке условия. Помогите пожалуйста разобраться.

Прилагаю ссылку для удобства:
jsfiddle.net/jfjpo189

Answer 1

[Андрей Ежгуров]

В обработчике $("#down_button").click(... убери 2 строчки с .bind и .unbind - и всё будет работать.

Comments

[Евгений Петров]

И добавить проверку в прокрутке наверх аналогично той, что вниз.

[Артем Шеленговский]

Спасибо! Все работает только я не понял логику я же вроде конкретно говорю ему - если ноль то сними клик с кнопки - и код не работает, а если удалить bind и unbind как вы сказали - код работает, а логика в этом такая что ли - если 0 то не делать ничего вообще при клике и не считать клики и не двигать список. Так получается?

[Андрей Ежгуров]

bind у тебя записан неправильно. Так что реально он ничего не делает. Да и не должно там bind быть: ты уже установил все нужные обработчики, зачем ставить что-то ещё?

У тебя unbind в обработчике события 'click' для #down_button снимает обработчик события 'click' для this. Но this в этом месте - это и есть #down_button. Т.е. обработчик сам себя отключает. И обратно он нигде не включается. Даже если бы bind был записан правильно - обработчик уже отключен и bind выполнен не будет.

Снимать обработку события внутри обработки этого события - это не лучшая практика.

Ни надо никаких лишних установок/снятий обработчиков. При наступлении события в обработчике проверяешь - должен ли он реагировать на него. Должен - выполняешь действия, не должен - выходишь из обработчика. Собственно, именно так это у тебя и работает - после исправления.

[Артем Шеленговский]

@eandr_67: Вот вам огромное спасибо! Все бы так подробно объясняли, как вы.