Премахване на активен клас от разделите на Mootools

освен производителността, кодът ви е наред. под производителност имам предвид - не трябва да се налага да отива в dom при всяко щракване, за да получи списъка с елементи или текущия активен, за да направи това, което трябва - всичко това може да се кешира. вашият код ще работи срещу всеки .active в колекция, която първоначално ще има .length 0, така че при първото щракване нищо няма да се случи. проблемът с производителността е да се изберат отново всички елементи и тези с .active, след което да се изпълни .each върху тях за единствения член, който може да го има.

можете да направите нещо подобно вместо това:

http://jsfiddle.net/dimitar/m66Ju/2/

(function() {
    var ul = document.getElement("ul.examples"), lis = ul.getElements("li"), active;
    lis.addClass('transition');

    lis.each(function(item){
        item.addEvent('click',function(event){
            if (active) {
                // there is one, remove it.   
                active.removeClass('active');
            }
            if (active == this) {
                // is the active one already open?
                active = false;
                return;
            }
            // make a new active.
            active = this.addClass('active');
        });
    });
})();

това ще бъде най-ефективно и ще работи и с първоначален активен li.

няма ръководство за това как да структурирате кода си за производителност и повторна употреба, но има няколко беседи, които могат да помогнат, като Програмиране към шаблони на Арън Нютон (с mootools) и т.н. Основното правило е, не правете преждевременна оптимизация, а програмирайте така че най-бавният поддържан браузър (обикновено IE6 или 7) работи възможно най-бързо. събитията с щракване дават известна свобода на действие, но тези с преминаване на мишката могат да бъдат скъпи.

И така, вашият проблем може да се разложи на две фази:

1. Избиране на вашите целеви li елементи.
2. Премахнете класа active.

Не забравяйте, че MooTools е обектно-ориентиран. И така, върху кои обекти ще работите? DOM елементи. Това означава, че трябва да разгледате Element документацията на MooTools.

За да изберете, вие търсите Element.getElements метод (или $$) .

След това, за премахване на клас… Е, просто Element.removeClass ;)

И така, вашият код трябва да изглежда нещо като:

$$('li.active').removeClass('active');