Показать сообщение отдельно
  #5 (permalink)  
Старый 30.09.2011, 19:14
sinistral
Посмотреть профиль Найти все сообщения от melky
 
Регистрация: 28.03.2011
Сообщений: 5,418

$('#'+id).eq(0)
            .css('padding',p+'px')
            .css('width',(w*3+p*6)+'px')
            .css('height',(h+p*2)+'px')
            .css('position','relative')
            .css('overflow','hidden')
            .find('.spslider3sld')
                .css('float','left')
                .css('margin',p+'px')
                .css('overflow','hidden')
                .css('height',h+'px')
                .css('width',w+'px')
            ;
        ;

ууу... вызовы css можно превратить в один, передав объект
$.css({"top":"5px", "left":"0"})

..
зачем писать
$('#'+id)*!*.eq(0)*/!*

если элемент с любым айдишником всегда уникален ? т.е. один

..............
if(spd==undefined || spd==0 || spd=='') spd=1200;
        if(id==undefined || id==0 || id=='') id='spslider3';
        if(w==undefined || w==0 || w=='') w=200;
        if(h==undefined || h==0 || h=='') h=400;
        if(p==undefined || p==0 || p=='') p=10;

заместо такой проверки преобразовывайте в boolean
var undef, def="defined";
alert([!!undef, !!def]);

или сразу так - часто это называют "настройкой по-умолчанию" (т.е. если аргумент не передан, его значение становится указанным)
var undef, def="defined";
alert([  *!*undef || "default" , def || "default"*/!* ]);


это вообще страсть. не могу понять принципа работы - он понятен только писавшему, к сожалению. Но, этот кусок ужасно выглядит.
var s=$('#'+id).eq(0).find('.spslider3imgi').eq(i).html();
        $('#'+id).eq(0).find('.spslider3sld').eq(0).html(s);
        if(i < this.len[id]-1) i++; else i=0;
        var s=$('#'+id).eq(0).find('.spslider3imgi').eq(i).html();
        $('#'+id).eq(0).find('.spslider3sld').eq(1).html(s);
        if(i < this.len[id]-1) i++; else i=0;
        var s=$('#'+id).eq(0).find('.spslider3imgi').eq(i).html();
        $('#'+id).eq(0).find('.spslider3sld').eq(2).html(s);
        if(i < this.len[id]-1) i++; else i=0;
Ответить с цитированием