Om du vill göra det lättare så finns det en massa jquery plugins som sköter en snygg slide åt dig, kolla cycle plugin för exempel, den är kanon och har många snygga och bra funktioner.
Jag hade skrivit om koden: Ha en array av objekt med url och bild-url:en
wrappat ditt a-element med en div(som är ett block element eller ett annat inline-element om det krävs för dig). och kollat längden på objektet istället för att hårdkoda 5, bli enklare för dig om du bestämmer dig att lägga in fler bilder i din snurra.
Och plocka bort eval!
//Private
(function() {
var objects = [
{ url: "http://test.com", image: "http://placekitten.com/200/140"},
{ url: "http://test1.com", image: "http://placekitten.com/g/200/140"},
{ url: "http://test2.com", image: "http://placehold.it/200x140"}
],
images = [],
step = 0,
slide = document.getElementById("slide"),
slideA = slide.getElementsByTagName("a")[0], //Find first a-tag inside #slide-tag
slideImg = slide.getElementsByTagName("img")[0], //Find the first img-tag inside the #slide-tag
slideIt = function () {
slideA.href = objects[step].url;
slideImg.src = objects[step].image;
step = step < objects.length-1 ? ++step : 0;
setTimeout(slideIt,4000)
};
//Pre-load images
for (var i = 0, l=objects.length;i<l;i++) {
var image = new Image();
image.src = objects[i].image;
images.push(image);
}
slideIt();
})();
