$('#'+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;