Miksi tämä scripti ei toimi?

Rumahkoa koodia tuo oli -- ja antiikkinen tapa tehdä kuvaslideshow -- mutta tuossa skriptissäsi oli tyypillinen virhe (switchImage funktiossa) kun välitetään parametrina setTimeout() metodille sellainen funktio jolle välitetään myös parametri.

Parasta on välittää argumenttina anonyymifunktio joka sitten kutsuu haluttua funktiota näin:

timerID = setTimeout (function() {switchImage(place); }, interval);


switchImage funktiomäärittelystä puuttui myös päättävä }. Tässä koko koodi (vähän siistin sitä samalla).




Javascript slide show testi


var interval = 1500;
var random_display = 0;
var imageDir = "";
var imageNum = 0;

imageArray = new Array();
imageArray[imageNum++] = new imageItem(imageDir + "Etusivu.jpg");
imageArray[imageNum++] = new imageItem(imageDir + "Etusivu.gif");
imageArray[imageNum++] = new imageItem(imageDir + "Etusivu.png");
imageArray[imageNum++] = new imageItem(imageDir + "Etusivu.jpg");
imageArray[imageNum++] = new imageItem(imageDir + "Etusivu.png");
var totalImages = imageArray.length;

function imageItem(image_location) {
this.image_item = new Image();
this.image_item.src = image_location;
}

function get_ImageItemLocation(imageObj) {
return imageObj.image_item.src;
}

function randNum(x, y) {
var range = y - x + 1;
return Math.floor(Math.random() * range) + x;
}

function getNextImage() {
if (random_display) {
imageNum = randNum (0, totalImages - 1);
}
else {
imageNum = (imageNum + 1) % totalImages;
}
var new_image = get_ImageItemLocation(imageArray[imageNum]);
return new_image;
}

function getPrevImage() {
imageNum = (imageNum - 1) % totalImages;
var new_image = get_ImageItemLocation(imageArray[imageNum]);
return new_image;
}

function prevImage(place) {
var new_image = getPrevImage();
document[place].src = new_image;
}

function switchImage(place) {
var new_image = getNextImage();
document[place].src = new_image;
timerID = setTimeout (function() {switchImage(place); }, interval);

}





play slide show
pause
previous
next




Ron "hedgehog" Jeremy

Hiukan toi skripti tarvitsi vielä viilaamista, sinulla puuttui m.m kokonaan nextImage() funktio, muutin myös switchImage() funktiota jotta voit käyttää getNextImage() funktiota sellaisenaan next linkissä...myös pieniä muutoksia siellä täällä (kaikki muutokset eivät olleet välttämättömiä):



Javascript slide show testi


var interval = 1500;
var random_display = false;
var imageDir = "";
var imageNum = 0;

var imageArray = new Array();
imageArray[imageArray.length] = new imageItem(imageDir + "Etusivu.jpg");
imageArray[imageArray.length] = new imageItem(imageDir + "Etusivu.gif");
imageArray[imageArray.length] = new imageItem(imageDir + "Etusivu.png");
imageArray[imageArray.length] = new imageItem(imageDir + "Etusivu.jpg");
imageArray[imageArray.length] = new imageItem(imageDir + "Etusivu.png");

var totalImages = imageArray.length;

function imageItem(image_location) {
this.image_item = new Image();
this.image_item.src = image_location;
}

function get_ImageItemLocation(imageObj) {
return imageObj.image_item.src;
}

function randNum(x, y) {
var range = y - x + 1;
return Math.floor(Math.random() * range) + x;
}

function getNextImage() {
imageNum = (imageNum + 1) % totalImages;
var new_image = get_ImageItemLocation(imageArray[imageNum]);
return new_image;
}

function getPrevImage() {
if (imageNum == 0) {
imageNum = totalImages - 1;
}
else {
imageNum = (imageNum - 1) % totalImages;
}
var new_image = get_ImageItemLocation(imageArray[imageNum]);
return new_image;
}

function prevImage(place) {
var new_image = getPrevImage();
document[place].src = new_image;
}

function nextImage(place) {
var new_image = getNextImage();
document[place].src = new_image;
}


function switchImage(place) {
if (random_display) {
var currentNum = imageNum;
while (currentNum == imageNum)
imageNum = randNum (0, totalImages - 1);
}
else {
imageNum = (imageNum + 1) % totalImages;
}
document[place].src = get_ImageItemLocation(imageArray[imageNum]);
timerID = setTimeout (function() {switchImage(place); }, interval);

}







play slide show
pause
previous
next




Ron "hedgehog" Jeremy