mystisk fejl dekleration af varibel

Tror jeg har fundet fejlen.... måske...

Jeg tror problemet er at jeg bruger setInterval() og hvis jeg sætter "var n = 0" ind i funktionen bliver n sat til 0 hvergang.

rotate() skal lave en form for banner.

linien

(n==(imgs.length-1))?n=0:n++;

er noget vrøvl. Den måde at skrive en if-sætning på kan ikke stå alene, men bruges til at vælge mellem to værdier som skal sendes til en funktion eller tildeles en variabel.

skriv sådan:
n++;
if (n>=imgs.length) n=0;

du bruger var UDENFOR function-defifnitions-blokken, for at gøre n til en top-level variabel. Så kan du INDENI funktionen skrive n eller window.n, og så er det den samme n i hvert timeout. Hvis du bruger var INDE I funktionen, laver du en ny variabel for hvert interval, som altid er nul :^(

n bliver brugt til at angive hvilket billede der skal vises. hvordan kan jeg lave koden så det hele står i en funktion?

For at køre det har jeg:
<img id='11' src='images/top13a.png' alt='' border='0'>
<script>setInterval("rotate(11,'top13a.png;top13b.png;top13c.png');",500);</script>

Men setInterval() skal vel ind i funktionen på en måde?

næ, setInterval er en gentage-det-samme-i-en-uendelighed funktion.
Du burde faktisk lave to funktioner, én til at lave en array af Image-objekter, og én til at styre rotationen. Image-objektet kan nemlig 'preloade' billedet, så browseren er klar til at skifte ud hurtigt.
For at virke på den måde skal Image objekterne genereres når siden loades (fks i et script i head) og ikke, som i din funktion ovenfor, på det tidspunkt hvor billederne skal bruges.


nb: Hvis der er flere 'bannere' på samme side får du problemer med variabelnavne...

jeg har netop flere flere bannere på samme side... men jeg kan ikke lige gennemskue hvordan jeg løser dette..

fks
<img id='11' src='images/top13a.png' alt='' border='0'>
<script>
    var imgs = fixPreloadArray ('top13a.png;top13b.png;top13c.png');
    setInterval("rotate('11',imgs);",500);
</script>

nå, nu kom mit svar efter din kommentar, jeg havde ikke læst den da jeg skrev det

det med variabel-navne var egentlig grundet til at holde det i en funktion, der så bare kunne køres på hver banner.

Du må finde en måde de forskellige variable kan blive holdt ude fra hinanden.
Det nemmest er for hver banner at lave et objekt som indeholder en tæller og en array med billeder

problemet er at funktionen bliver kaldt på ny for hver interval, og så oprettes nye 'interne' variabler

du mener ikke det kan løses på nogen måde ved kun at have én funktion eller 2?

<script type="text/javascript">
var n = new Array();
function rotate(linkid,strfilename){
if(!n[linkid])n[linkid] = 0;
var path='images/';
var arrlinks = strfilename.split(';');
document.getElementById(linkid).src=path+arrlinks[n[linkid]];
n[linkid] = (++n[linkid])%arrlinks.length;
}
</script>

-- der indgår en preloader, som ikke giver nogen mening, når alle billeder loades hver gang, derfor har jeg udeladt den ...

kan jeg ikke lave et array hvor hver element består af et id og en tæller?
så kan jeg vel lave et opslag for at se hvorlangt en banner er nået.

aa så ikke dit svar :)

<script>
function fix_preload(strfilename)
{
var path='images/';
var arrlinks = strfilename.split(';');
var imgs = new Array(), c;
   
for ( i=0; i<arrlinks.length; i++) {
c = new Image();
c.src = path+arrlinks[i];
imgs.push(c);
}
return imgs;
}

function rotate (obj) {
    obj.n ++;
    if (obj.n >= obj.imgs.length) obj.n = 0;
    document.getElementById(obj.imgID).src = obj.imgs[obj.n].src;
}
</script>
----------------------
<img id='11' src='images/top13a.png' alt='' border='0'>
<script>
    var obj_11 = {
        'imgs' : fix_preload ('top13a.png;top13b.png;top13c.png'),
        'imgID' : '11',
    'n' : 0
    }
    setInterval("rotate(obj_11);",500);
</script>

hehe, der er mange måder at flå en kat, som det hedder på engelsk...
q:^)

roenving <- jeg kan ikke lige få det script til at virke

-- du har også en ugyldig reference, da et id _ikke_ må starte med et tal, det havde jeg ikke kigget på, men så:

<img id='i11' src='images/top13a.png' alt='' border='0'>
<script>setInterval("rotate('i11','top13a.png;top13b.png;top13c.png');",500);</script>

Jeg fik det til at virke også uden det med id'et.

Jeg kan se at min mozilla står og "Transfering data from .." hver gang billedet skirfter. Hvorfor går den det ligepludselig?
Det gjorde den ikke før.

-- men principielt er det det også bedre at bruge knudx' ide, som også preloader billederne, men han har heller ikke være opmærksom på den html-fejl, det er at bruge et id med et tal forrest eller den javascript-fejl, det tilsvarende vil være at bruge identifier, som starter med et tal. Desuden kan en lille forbedring laves:

<script type="text/javascript">
function fix_preload(strfilename){
var path='images/';
var arrlinks = strfilename.split(';');
var imgs = new Array(), c;
   
for ( i=0; i<arrlinks.length; i++) {
c = new Image();
c.src = path+arrlinks[i];
imgs.push(c);
}
return imgs;
}

function rotate (obj) {
    obj.n = (++obj.n)%obj.imgs.length;
    document.getElementById(obj.imgID).src = obj.imgs[obj.n].src;
}
</script>
----------------------
<img id='i11' src='images/top13a.png' alt='' border='0'>
<script type="text/javascript">
    var obj_11 = {
        'imgs' : fix_preload ('top13a.png;top13b.png;top13c.png'),
        'imgID' : 'i11',
    'n' : 0
    }
    setInterval("rotate(obj_11);",500);
</script>

-- desuden er principielt nødvendigt at angive en type for et script-tag ...

-- og den sidste script-blok kan du forkorte til een eller to linjer:

<script type="text/javascript">
    var obj_11 = {'imgs' : fix_preload ('top13a.png;top13b.png;top13c.png'),'imgID' : 'i11','n' : 0};
    setInterval("rotate(obj_11);",500);
</script>

når jeg sætter flere bannere går der alligevel noget galt.
Det er som om mozilla ikke kan håndtere flere setInterval().

IE virker fint.

Jeg har prøvet dit script knudx og nu kommer mozilla ikke med "Trasfering data from.." længere, men mozilla har stadig problemer med setInterval()..

Kan i se hvorfor?

Jeg har ikke haft problemer med flere setIntervals, men vi kan da snyde lidt:

<script type="text/javascript">
    var obj_11 = {'imgs' : fix_preload ('top13a.png;top13b.png;top13c.png'),'imgID' : 'i11','n' : 0,'ival':0};
    obj_11.ival = setInterval("rotate(obj_11);",500);
</script>

ok.. vil I dele pointne?

Ej, der er vist ikke noget at del ,-)

hmm.. min mozilla står stadig og siger "Transfering data..." meget hurtigt. Kan jeg løse dette på nogen måde?

Ved I hvorfor den gør det?

Med den konstruktion vil den preloade alle bannere fra start, så den burde blive færdig ret hurtigt, og så ikke igen have brug for det ...

Den gør det alligevel heletiden...

Hmm jeg har lavet 3 bannere med henholdsvis et A,B,C på... de kommer ikke i rækkefølge... Hvad kan dette skyldes?