Takze, niekolko rad:
Skus si tie nazvy obrazkov vlozit do
pola:
Kód:
var obrazky = ['X41.jpg', 'X42.jpg', 'X43.jpg', 'X44.jpg'];
S tym sa ti bude omnoho lahsie pracovat. Premenna modulus bude pri deleni styrmi nadobudat hodnoty 0 - 3, preto bude stacit proste
Kód:
obr.setAttribute('src', obrazky[modulus]);
Mas to tak bez switchu, omnoho jednoduchsie.
Dalej vyhod to appendChild. Ten <img> element tam uz je, nikde ho neodstranujes, takze ho ani netreba zasa pripajat.
Dalej tam mas jednu logicku chybu - podmienka modulus<5 je vzdy pravdiva, vetva else sa nikdy nevykona.
Ohladom tych anonymnych fcii - proste je to o tom, ze kus kodu vlozis do fcie, ktoru nijak nepomenujes ani ju nebudes moct potom zavolat. Hodi sa to hlavne vtedy, ked ti staci fciu volat raz, velmi dobre to vidno na priklade so setIntervalom:
Kód:
setInterval(function() {
//kod
}, 1000);
vs
Kód:
function nejakyzbytocnynazov() {
//kod
}
setInterval(nejakyzbytocnynazov, 1000);
Si ale na dobrej ceste, drzim palce. Zide sa ti Firefoxova Error Console (Ctrl+Shift+J), ktora ti vyhodi chyby v skripte. V inych prehliadacoch najdes cosi podobne.
Nauc sa pouzivat anonymne fcie:
