Javascript werkt niet....

Status
Niet open voor verdere reacties.
P

PhilKill

Nieuwe gebruiker
Lid geworden
28 feb 2010
Berichten
3
Hallo, Ik ben bezig met een webdesign cursus en moet nu voor het eerst met Javascript werken. Als voorbeeld moet ik volgende code gebruiken welke bij mij gewoon niet wil werken:

<body>
<div id=tijdscript></div>

<script>
var rDe = 0
var hr = -1;
var nD = -1
function stijdscript()
{var today = new Date();
if ( hr != today.getHours())

{if ( nD != today.getDate())
{nFY = today.getFullYear();
nM = today.getMonth();
nD = today.getDate();
todayA = new Date(nFY,nM,nD,4,0,0)
endofyear = new Date(nFY-1, 11, 31,6,0,0);
Tday = Math.ceil((todayA.getTime()-endofyear.getTime())/(1000*60*60*24));
W1day = Tday;
if( endofyear.getDay() <6) W1day += endofyear.getDay() + 1;
Week = Math.ceil((W1day)/7);
Mes2 = ".";
for (ir = 1; ir <= rDe; ir++)
{eval("Rm = rV" + ir + "[0] - 1; Rd = rV" + ir +"[1]"); if ( Rm == nM && Rd == nD) Mes2 =
"" + eval("rV" + ir + "[2]");}
Mese = (". Dag " + Tday + " - kalender week " + week + Mes2)}
hr = today.getHours();
gr = "morgen";
if (hr >= 12) gr = "middag";
if (hr >= 18) gr = "navond";
Mesb = ("Goede" + gr +" op ")}

document.getElementById('tijdscript').innerHTML = (Mesb + today.toLocaleString() + Mese);
setTimeout('stijdscript()',1000;}
Stijdscript();
</script>
</body>


Kan iemand mij a.u.b. zeggen wat er mis mee is?
 
Het gaat direct in het begin al fout, je sluit alleen de tweede variable af met een ; en dat moet bij elke regel behalve als er een { of } volgt. Hieronder de regels die moeten worden aangepast:

var rDe = 0;
var hr = -1;
var nD = -1;

todayA = new Date(nFY,nM,nD,4,0,0);
Mese = (". Dag " + Tday + " - kalender week " + week + Mes2);}
hr = today.getHours();
gr = "morgen";
Mesb = ("Goede" + gr +" op ");}

document.getElementById('tijdscript').innerHTML = (Mesb + today.toLocaleString() + Mese);
setTimeout('stijdscript()',1000);}

Dit is even snel gecorrigeerd, want je code is niet echt overzichtelijk, als ik je een tip mag geven... Plaats comments bij je code en maak gebruik van de enter toets op je toetsenbord, niet alles achter elkaar door typen.

Dus bijvoorbeeld

[JS]
//comment
for each(waarde w in waardes)
{
if(eigenschap == "waarde")
{
//Comment die bij deze actie hoort
doeEenActie();
doeNogEenActie();
}
}
[/JS]
 
Sorry als het onoverzichtelijk is maar ik heb het precies zo getypt als het in het voorbeeld stond. Ik kan niet geloven hoe veel fouten er in de tekst zitten. Ik dacht dat LOI een goede naam heeft voor thuisopleidingen... Ik heb de fouten aangepast maar het werkt nog steeds niet. Ik ga straks proberen de code wat op te knappen.
 
Ik heb hem even netjes gemaakt, probeer hem zo nog maar eens. Ik kan hem niet controleren aangezien ik niet weet wat hij moet doen.

[JS]
var rDe = 0;
var hr = -1;
var nD = -1;

function stijdscript()
{

var today = new Date();
if ( hr != today.getHours())
{
if ( nD != today.getDate())
{
nFY = today.getFullYear();
nM = today.getMonth();
nD = today.getDate();
todayA = new Date(nFY,nM,nD,4,0,0);
endofyear = new Date(nFY-1, 11, 31,6,0,0);
Tday = Math.ceil((todayA.getTime()- endofyear.getTime())/(1000*60*60*24));
W1day = Tday;
if( endofyear.getDay() < 6)
{
W1day += endofyear.getDay() + 1;
}

Week = Math.ceil((W1day)/7);
Mes2 = ".";
for (ir = 1; ir <= rDe; ir++)
{
eval("Rm = rV" + ir + "[0] - 1; Rd = rV" + ir +"[1]");
if ( Rm == nM && Rd == nD)
{
Mes2 = "" + eval("rV" + ir + "[2]");
}
}
Mese = (". Dag " + Tday + " - kalender week " + week + Mes2)
}
hr = today.getHours();
gr = "morgen";
if (hr >= 12)
{
gr = "middag";
}
if (hr >= 18)
{
gr = "navond";
}
Mesb = ("Goede" + gr +" op ")
}

document.getElementById('tijdscript').innerHTML = (Mesb + today.toLocaleString() + Mese);
setTimeout('stijdscript()',1000);

}

Stijdscript();
[/JS]
 
Status
Niet open voor verdere reacties.
                    Steun ons                    

Nieuwste berichten

Terug
Bovenaan Onderaan