Knud Gert Ellentoft <knudgert@mail.tele.dk> wrote:
>>Jeg undrer mig lidt over at du bruger writeln i selve body og ikke
>>bare lader javascriptet ordne det hele for dig?
>
>Jeg er ikke selv nogen javascripthaj, det er en jeg har engang
>har fået af en anden og jeg har kun lavet små ændringer.
Jeg spørger fordi jeg skal forstå koden. Men lidt tests viser at det
virker fint hvis du lægger javascripten i body ind i funktioner og
blot indlejrer det dynamske istedet for at skrive statisk html med
writeln (det sidste gør det meget læseuvenligt og sværere at skrive i
en editor).
>Jeg er ikke sikker på, at jeg tør lave noget mere radikalt om i
>den.
Som javascript eksempel synes jeg det bør være lavet på den mest
læsevenlige måde - og gerne med lidt kommentarer til - eller en
gennemgang af hvad koden gør.
Jeg har gennemskuet princippet nu - så ikke for min skyld, så jeg
nævner det kun fordi det netop er et demonstrations-eksempel.
Derudover synes jeg du skulle overveje en anden syntaks i koden. Du
skriver f.eks:
function changePage()
{
menuNum = document.ccDoubleList.ccBitsyG.selectedIndex;
if (menuNum == null){alert("Please sel ..");return;}
else
{
i = document.ccDoubleList.ccDoubleZ.selectedIndex;
window.location.href = url[i];
}
Det synes jeg er meget uvenligt at læse. Prøv at sammenligne med:
// Funktionen changePage kaldes ved tryk på knappen og dirigerer til
// den valgte side
function changePage() {
// Hent indeks for det valgte element i den første dropdown boks
menuNum = document.ccDoubleList.ccBitsyG.selectedIndex;
// Alarm-boks og returner hvis index er null-værdi
if (menuNum == null) {
alert("Please sel ..");
return;
}
// Hent URL fra array via det valgte index og diriger browser dertil
else {
i = document.ccDoubleList.ccDoubleZ.selectedIndex;
window.location.href = url[i];
}
}
Men hvis jeg glemte det, tak for hjælpen. Jeg har fået løst mit
problem.
--
Lars Kim Lund
http://www.net-faq.dk/