[Javascript] fout: object vereist in "uitklapmenu"

[Theske]

Beste mensen,

Ik ben bezig geweest met een "uitklap menu" voor een website. Het gaat allemaal erg goed, alleen toen ik in IE het gele driehoekje zag kropen de rillingen over mn lichaam ;-)

De error gaf aan:

code:

regel: 239
teken: 12
fout: object vereist

De betreffende functie doet het wel gewoon en in FF geeft hij helemaal geen errors erover.

Graag hoor ik van jullie of jullie weten hoe je deze fout kan verhelpen... voor zover ik weet geeft het geen problemen met zich mee, maar het staat niet zo netjes die error.

Alvast bedankt!

JavaScript:

<script type="text/javascript">
function expandOrCollapse(menu)
{
var ul = menu.nextSibling;
while(ul.tagName != "UL")
{
ul = ul.nextSibling;
if ( ul == null )
return true;
}
if ( ul.style.display == "none"
|| ul.style.display == "" )
ul.style.display = "block";
else
ul.style.display = "none";
return false;
}


startList = function() {
if (document.all&&document.getElementById) {
navRoot = document.getElementById("nav");
for (i=0;  i < navRoot.childNodes.length; i++) {   // [b]regel 239, de tweede i in de regel is teken 12 [/b]
node = navRoot.childNodes[i];
if (node.nodeName=="LI") {
node.onmouseover=function() {
this.className+=" over";
  }
  node.onmouseout=function() {
  this.className=this.className.replace(" over", "");
   }
   }
  }
 }
}
window.onload=startList;

</script>

[CodeCaster]

-

[ Voor 100% gewijzigd door CodeCaster op 23-10-2006 01:10 . Reden: laat maar ]

[André]

Regel 239 is welke regel in bovenstaand script? Verder maak je gebruik van document.all, dat is al verschrikkelijk verouderd.

[johanmulder]

code:

for (i=0;  i < navRoot.childNodes.length; i++)

navRoot.childNodes geeft waarschijnlijk de waarde 'undefined' en dan gaat dus het opvragen van de length ook helemaal fout. Vang dat eerst dus af.

[ Voor 4% gewijzigd door johanmulder op 23-10-2006 08:53 ]

[Theske]

André schreef op maandag 23 oktober 2006 @ 08:33:
Regel 239 is welke regel in bovenstaand script? Verder maak je gebruik van document.all, dat is al verschrikkelijk verouderd.

Als je in de code kijkt zie je wat regel 239 is, het zal wel iets te vroeg zijn geweest voor je

johanmulder schreef op maandag 23 oktober 2006 @ 08:53:

code:

for (i=0;  i < navRoot.childNodes.length; i++)

navRoot.childNodes geeft waarschijnlijk de waarde 'undefined' en dan gaat dus het opvragen van de length ook helemaal fout. Vang dat eerst dus af.

Hoe zou je dit dan kunnen afvangen?
Ik ben niet zo'n held met JS, ik heb deze code via een "cursus/artikel" weten te bemachtigen. Ik hoop dat je me hierbij wat meer de goede kant op kan sturen.
Het script doet het dus wel correct, alleen geeft hij deze foutmelding.

[crisp]

defensief programmeren:

JavaScript:

if (document.getElementById)
{
  var navRoot = document.getElementById('nav');
  if (navRoot && navRoot.hasChildNodes())
  {
    for (var i=0; i < navRoot.childNodes.length; i++)
    {
      //-- etcetera
    }
  }
}

[user109731]

De betreffende functie doet het wel gewoon en in FF geeft hij helemaal geen errors erover.

Doe gebruikers van andere browsers een lol en verwijder dat 'document.all&&'... Je code word daardoor in firefox niet uitgevoerd, vandaar datie geen errors geeft

[Theske]

Mensen, bedankt allemaal voor jullie reactie!

Ik heb het stukje script van crisp gebruik en dat werkt en IE geeft geen errors meer terug!
Ik heb er weer iets van op gestoken, dus de dag is weer leerzaam "begonnen".

Bedankt!

[Verwijderd]

Ik zit met hetzelfde probleem, maar met deze oplossing lukt het nog altijd niet.
Ik werk ook met een uitklapmenu en een aantal zaken mogen niet weergegeven worden bij sommige profielen. Het lukt allemaal, maar bij de laatste if geeft IE ook een foutboodschap, namelijk ook dat er een object vereist is.
Als ik dezen laatste if weg laat werkt het volledig.

Dit is de code:

HTML:

Lap code :S

Ik ben hieraan niet zelf begonnen, maar moet verderwerken op het bestaande. Ik heb al een aantal dingen toegevoegd en een aantal dingen grotendeels kunnen overnemen en voor dit werkte dus alles. Dit laatste heb ik dan ook overgenomen en dus alleen op de 'trcRSalesForecast' geeft hij de foutmelding.

[ Voor 97% gewijzigd door BtM909 op 23-05-2007 16:01 ]

[Verwijderd]

heel raar, ik het niets veranderd, maar toch werkt het nu wel volledig, opgelost dus

[BtM909]

Modbreak:

Post volgende keer dan ook relevante code ipv 500 regels aan code