IsObject !?

Du har ikke brug for eval() til dynamiske objekt referencer, du kan bruge array notation:

for (var index=1; index<=8; index++){
  if (document[\"menuImg\" + index]){ // Hvis f.eks. menuImg5 findes
    // Så gør noget med den
  }
}

/Thor

jumper:> normalt er jeg ligeglad, men det skal virke i NS 4+. Nedenfor er hele min kode i den funktion (activeIndex=0; i toppen af js. fil så den er fin nok)

for (var index=1;index<=5;index++) {
    if (index==activeIndex) onoff=\"hpil.gif\" else onoff=\"pil0.gif\";

    if (n)
        lay=\"document.menuLayer\"+index+\".document.images.menuImg\"+index+\".src=\'picts/\"+onoff+\"\';\"
    else
        lay=\"document.menuImg\"+index+\".src=\'picts/\"+onoff+\"\';\"
               
    eval(lay);
}

der kan du bl.a. se hvorfor jeg bruger eval().

PS. Jeg er INGEN mester i JS så det kan sikkert godt være at det kunne laves bedre ;-)

tdaugaard, array notation virker også fint i NS4+, det har været en del af grund-sproget siden Netscape 2.0 (p.g.a. associative arrays) ;)

Du har ihvertfald ikke brug for eval. En måde at lave det på kunne være:

for (var index=1, onoff, dEl; index<=5; index++){
  onoff = (index==activeIndex) ? \"hpil.gif\" : \"pil0.gif\";
  dEl = document.layers ? document[\"menuLayer\" + index].document.images : document;
  dEl[\"menuImg\" + index].src = \"picts/\" + onoff;
}

/Thor

lignede jeg måske en der fattede hvad du havde skrevet *S* jeg prøver det lige (okay jeg fatter det godt ..)

Det virker .. dog ikke helt efter hensigten :-(

hvis jeg laver <=5 om til <=8 så den skulle løbe til 8, men breake hvis f.eks. nr. 5 ikke fandtes så får jeg en \"undefined er ikke et objekt\"

Ahh okie, så ændrer vi bare lidt (jeg ændrede kun på den sidste kode du kom med :)

for (var index=1, onoff, dEl; index<=5; index++){
  onoff = (index==activeIndex) ? \"hpil.gif\" : \"pil0.gif\";
  dEl = document.layers ? document[\"menuLayer\" + index].document.images : document;
  if (dEl[\"menuImg\" + index]){ // Hvis f.eks. menuImg5 findes
    dEl[\"menuImg\" + index].src = \"picts/\" + onoff;
  } else {
    break; // Fjern break hvis den bare skal prøve alle
  }
}

/Thor

Der er bare nogle der kan det der, hva\' !? *S*
Jeg bukker og takker mange gange :-)

ehm... Netscape 4.7 var åben bart ikke enig i din påstand om at det også funkede i den ;-(

document[\"menuLayer\" + index] has no properties.

får jeg ..

det skal lige siges at menuLayer er et <div> layer, hvis det har nogen betydning.

Doh! Jeg glemmer altid at teste i NS4 (er ved at være træt af den :). Nå pyt:

for (var index=1, onoff, dEl, d=document; index<=5; index++){
  onoff = (index==activeIndex) ? \"hpil.gif\" : \"pil0.gif\";
  dEl = d.layers && d[\"menuLayer\" + index] ? d[\"menuLayer\" + index].document.images : d;
  if (dEl[\"menuImg\" + index]){ // Hvis f.eks. menuImg5 findes
    dEl[\"menuImg\" + index].src = \"picts/\" + onoff;
  } else {
    break; // Fjern break hvis den bare skal prøve alle
  }
}

Var det bedre ? ;)

/Thor

*S* (ved at være ? - blev jeg allerede inden den kom! *G*)

nu gik NS ned, så jeg kan ikke lige teste,men jeg regner med det er ok .. du lyder som om du ved hvad du snakker om :-)