alternativer loesungsansatz (teil2): code review (teil 3)

[peterS.]

... aehm ja ... langsam wirds peinlich - hier ist noch 'nen copy-paste-massengrab:

  

>       (function () { // anonymus [initialize].  
>   
>         slideList = slideshowData[idxOfSlideshowElm];  
>         amountOfSlides = slideList.length;  
>   

         //falsch:  

>       //for (var regXDataSrc = (/\s/), src = elmSlide.src, len = slideList.length, i = 0; i<len; ++i) {/*  

         //richtig:  
           for (var regXDataSrc = (/\s/), src = elmSlide.src, i = 0; i<amountOfSlides; ++i) {/*  
  
           //falsch:  

>           console.log("i : " + i + " - regXDataSrc : " + regXDataSrc.compile(arr[i].toRegExpString(), "i"));  
>           console.log("regXDataSrc.test(\"" + src + "\") : " + regXDataSrc.compile(arr[i].toRegExpString(), "i").test(src));*/  

           /*richtig:  
             console.log("i : " + i + " - regXDataSrc : " + regXDataSrc.compile(slideList[i].toRegExpString(), "i"));  
             console.log("regXDataSrc.test(\"" + src + "\") : " + regXDataSrc.compile(slideList[i].toRegExpString(), "i").test(src));*/  
  
           //falsch:  

>         //if (regXDataSrc.compile(arr[i].toRegExpString(), "i").test(src)) {  

           //richtig:  
             if (regXDataSrc.compile(slideList[i].toRegExpString(), "i").test(src)) {  
  

>             idxSlide = i;  
>             break;  
>           }  
>         }  
>         idxSlide = ((idxSlide == -1) ? (0) : (idxSlide));  
>   
>         refreshCounter();  
>   
>         elmPrevious.onclick = onPrevious;  // [link:http://developer.yahoo.com/yui/docs/YAHOO.util.Event.html#method_on@title=Evt.on](elmPrevious, "click", onPrevious); // YUI-lib;  
>         elmNext.onclick = onNext;          // Evt.on(elmNext, "click", onNext);  
>   
>       //[link:http://developer.yahoo.com/yui/docs/YAHOO.util.Dom.html#method_setStyle@title=DOM.setStyle](elmControl, "display", "block"); // YUI-lib;  
>         elmControl.style.display = "block";  
>   
>         delete arguments.callee; // throwing away anonymus [initialize].  
>       })();  

bitte entschuldigt dieses *live*-debugging. ich denke damit sind jetzt alle schnitzer behoben.

so long - peterS. - [e-mail]