Click here to Skip to main content
Rate this: bad
good
Please Sign up or sign in to vote.
See more: Javascript HTML
I am using the following codes to swap images one after another but only the image mentioned in <img> tag is displayed. The images are not swaping as i want. please help me.
 
<html>
<head>
<script language="javascript">
(function(){
 var rotator=document.getElementById('rotator');
 var imageDir='';
 var delayInSeconds=2;
 var images=['2.jpg','3.jpg','4.jpg','5.jpg','4.jpg'];
 
 var num=0;
 var changeImage=function(){
    var len=images.length;
    rotator.src=imageDir+images[num++];
    if(num==len){
        num=0;
        }
    };
    setInterval(changeImage, delayInSeconds * 1000);
    })();
</script>
</head>
<body>
    <img src="4.jpg" alt="rotating image" width="700" height="600" id="rotator">
</body>
</html>
Posted 4-Oct-12 6:15am

1 solution

Rate this: bad
good
Please Sign up or sign in to vote.

Solution 1

You are trying to get a reference to the image element in your main closure, before the document has loaded.
 
If you move the var rotator=... line inside the changeImage function, it should work.
  Permalink  

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

  Print Answers RSS
0 Sergey Alexandrovich Kryukov 565
1 Maciej Los 359
2 Kornfeld Eliyahu Peter 309
3 DamithSL 196
4 OriginalGriff 188
0 OriginalGriff 6,303
1 DamithSL 4,764
2 Maciej Los 4,306
3 Kornfeld Eliyahu Peter 3,914
4 Sergey Alexandrovich Kryukov 3,538


Advertise | Privacy | Mobile
Web03 | 2.8.141220.1 | Last Updated 4 Oct 2012
Copyright © CodeProject, 1999-2014
All Rights Reserved. Terms of Service
Layout: fixed | fluid

CodeProject, 503-250 Ferrand Drive Toronto Ontario, M3C 3G8 Canada +1 416-849-8900 x 100