You Want Infuriating? I'll Show You Infuriating!

Ben Horne · November 22nd, 2004, 02:25 PM

As you can see, I've been having problems with the mwcatering web page that I have been working on for my Intro to Javascript class. This is how I have the banner cycle code laid out:

Code:

<SCRIPT LANGUAGE="JAVASCRIPT">
<!-- Hide from old browsers
    var banners = new Array("banner1.jpg","banner2.jpg","banner3.jpg","banner4.jpg")
    var bnrCntr = 0
    function bancycle()
            {
            bnrCntr = bnrCntr + 1
            if (bnrCntr == 4)

               bnrCntr = 0
            }
            document.Banner.src = banners[bnrCntr]
            setTimeout("bancycle()",3000)
          }

//-->                 
</SCRIPT>

I did it this way at the suggestion of my instructor but I'm getting the error message saying that bancycle is not defined. I'm using Firefox to test the page, BTW.

Ben Horne
-------------------------
I don't want to sound like I haven't made any mistakes. I'm confident I have.

Most likely using FireFox and concocting my next Macromedia Flash project
Snibworks Forums Moderator

Welcome to the New Age

Imar · November 22nd, 2004, 04:44 PM

Please be more specific with your subjects. This makes it much easier to determine whether a message is worth reading / answering or not for your fellow P2P members.

A topic like "Problem with setTimeout method" or "IE only code refuses to work in Firefox" really makes things much easier for everyone.

Besides that, it wasn't an original subject anyway ;)

Imar
---------------------------------------
Imar Spaanjaars
Everyone is unique, except for me.
While typing this post, I was listening to: I Could Have Lied by Red Hot Chili Peppers (Track 6 from the album: Blood Sugar ************ Magik) What's This?

Greg Griffiths · November 22nd, 2004, 04:49 PM

I think you've missed a bracket, and a few semicolons try :

Code:

    function bancycle()
    {       bnrCntr = bnrCntr + 1;
            if (bnrCntr == 4)
            {     
               bnrCntr = 0;
            }
            document.Banner.src = banners[bnrCntr];
            setTimeout("bancycle()",3000);
    }

I assume that you are setting the initial value of bnrCntr elsewhere ?