[JS] Pakt alleen de eerste van de array

woensdag 9 april 2008 19:46

Acties:

0 Henk 'm!

Administrator

Topicstarter

Hallo Tweakers,
Ik ben bezig met een soort in/uitklap menu voor mn site dat mensen willekeurig menu's kunnen sluiten zegmaar en dat bewaar ik dan in cookies

Nu is het probleem dat hij alleen maar de eerste van de array inklapt terwijl de andere cookies ook gewoon bestaan

Code:

JavaScript:

function scan_menu(menu_array)
{
    for( i=0; i<menu_array.length; i++ )
    {
        var menu_naam = menu_array[i];
        
        var id = document.getElementById(menu_naam);
        var menu = document.getElementById(menu_naam + "menu");
        var img = document.getElementById(menu_naam + "img");
        
        if( getcookie("hide_" + menu_naam) == 1 )
        {
            id.style.display = "none";
            menu.style.margin = "0px 0px 5px 0px";
            img.style.background = "url(http://www.saven.nl/i/png/plus.png)";
        }
        else
        {
            id.style.display = "";
            menu.style.margin = "";
            img.style.background = "url(http://www.saven.nl/i/png/min.png)";
        }
    }
}

En dat roep ik dan zo aan:

HTML:

1	<body onload="scan_menu(Array('poll', 'topics', 'crewtopics', 'blogs', 'blogreacties', 'linkpartners', 'downloads'));">

Het probleem is dus dat hij alleen maar de poll inklapt

Kan iemand mij vertellen wat ik fout doe, en hoe ik dit kan fixen?

Alvast bedankt

woensdag 9 april 2008 21:12

Acties:

0 Henk 'm!

Pkunk

Wat doet getcookie() ? Hoe ziet je html eruit? Wat heb je zelf al geprobeert?
Volgens mij kan je dit prima debuggen hoor..

Hallo met Tim

woensdag 9 april 2008 21:42

Acties:

0 Henk 'm!

crisp

Devver

Pixelated

Javascript

Doe eens een alert(menu_naam) binnen je loop, zie je dan wel alle menu-items langskomen?
Zoniet dan vermoed ik dat functie getcookie() ook iets met variabele 'i' doet zonder deze te declareren net zoals je 'm in scan_menu() niet declareert en het dus een global variabele is.

Intentionally left blank

donderdag 10 april 2008 10:43

Acties:

0 Henk 'm!

r0bert

Misschien een beetje afgezaagd, maar ik zou even oppassen met termen die gereserved zijn.

Ok, ik zég dus niet dat hier reserved words zijn (geen idee). Alleen dat je erop moet letten, aangezien ik hier het idee heb dat dat niet gebeurd.

[ Voor 45% gewijzigd door r0bert op 10-04-2008 11:26 ]

donderdag 10 april 2008 11:00

Acties:

0 Henk 'm!

RM-rf

1 2 3 4 5 7 6 8 9

code:

1	var id = document.getElementById(menu_naam);

noem die variabele bv 'mijnElement'... 'id' is een gereserveerde variebele en nar ik meen explorer kan er behoorlijk op 'hangen' en hele vreemde dingen van gaan doen.

Intelligente mensen zoeken in tijden van crisis naar oplossingen, Idioten zoeken dan schuldigen

donderdag 10 april 2008 11:01

Acties:

0 Henk 'm!

crisp

Devver

Pixelated

Javascript

'id' is helemaal geen reserved word hoor

Intentionally left blank

vrijdag 11 april 2008 13:35

Acties:

0 Henk 'm!

Verwijderd

Als ik jouw code hier uitprobeer, dan krijg ik allereerst een fout omdat de functie getcookie() niet bestaat. Als ik dit deel uitcommentarieer, dan worden alle stappen in de for-lus hier gewoon uitgevoerd. Is er niet iets fout met die getcookie functie, waardoor de lus wordt afgebroken?

vrijdag 11 april 2008 14:06

Acties:

0 Henk 'm!

Sendy

RM-rf schreef op donderdag 10 april 2008 @ 11:00:
code:
1
var id = document.getElementById(menu_naam);
noem die variabele bv 'mijnElement'... 'id' is een gereserveerde variebele en nar ik meen explorer kan er behoorlijk op 'hangen' en hele vreemde dingen van gaan doen.

"mijnElement"

. "id" is niet handig omdat het resultaat van getElementById geen "id" is, en "mijn" is niet handig omdat het niets zegt. Neem iets als menu_element.

[ Voor 5% gewijzigd door Sendy op 11-04-2008 14:07 ]

Onderwerpen