Validering af datofelter.

følgende kunne sikkert bruges, selvom det nok kunne gøres meget smartere

jeg vil prøve at lære dig hvad der skal til, så sig endelig til hvis du har spørgsmål

først skal vi til vores form, dette gør vi for at spare kode senere hen.Det gør vi ved at skrive:

df = document.forms[0]

Hvis det er den første formular på siden... 1 hvis det er formular 2 etc etc
Når javascript selv tæller elementer starter den med at tælle fra 0

Det næste er at vi skal have lavet en funktion vi kan kalde for at validere datoen....

function valiDate(){
d = parseInt(df.dag.options[df.dag.selectedIndex].text)
/*
text hvis det er udenfor option taggen, value hvis det er i value attributten
Vi bruger parseInt til at lave værdien fra selectboxen om til et tal (hel tal, men det er ligegyldigt)
*/
m = parseInt(df.maaned.options[df.maaned.selectedIndex].text)



maxMonths = (m==1|m==3|m==5|m==7|m==8|m==10|m==12)?31:(m==2)?27:30
/*Den ovenstaaende linie checker for om det er en af de første 31 dages maaneder, i saa fald 31, ellers checkes om maaneden er nummer 2, saa er det 27 dage (der tages ikke hoejde for skudaar)
*/
switch (maxMonths){
case 31:
return true  //her behoever vi ikke tjekke mere
break;

case 30:
if(d>30){
alert('denne maaned har kun 30 dage')
return false
}else{
return true
}
  break;

case 27:
if(d>27){
alert("februar har kun 27 dage")
return false
}else{
return true
}
break;
}


}

så kan du i din <form> tag skrive <form onsubmit="return valiDate()">

Håber du lærte lidt af det ;-)

Hvis det nu er en formular, der skal videre til opret.php, hvordan fletter jeg så det ind? Lige nu står det bare under action i <form>-taggen, men så vliderer den jo ikke.

En skønhedsfejl:

maxMonths = (m==1|m==3|m==5|m==7|m==8|m==10|m==12)?31:(m==2)?27:30

skal være:

maxMonths = (m==1||m==3||m==5||m==7||m==8||m==10||m==12)?31:(m==2)?28:30

Da enkelt-| er en bit-operator og februar har 28 dage udenfor skudår !-)

Der er stadigvæk noget galt, for jeg kan sagtens oprette en dato der hedder 31-02-2003 og det burde jo ikke kunne lade sig gøre.

du burde ikke kunne submitte

og din form tag skal hedde

<form action="opret.php" onsubmit="return valiDate()">

roenving, tak fordi du retter mine skønhedsfejl, var lidt under tidspres... så skrev bare løs ud fra det blå ;-)

Har selv denne liggende, som bruger et Date-objekt til validering, for så slipper jeg selv for at tage stilling til skudår, lange og korte måneder m.m.m.

<script language='javascript' type='text/javascript'>
function checkDato( dmy ) {
        var dag = parseInt(dmy[0],10);
        var maaned = parseInt(dmy[1],10)-1;
        var dato = new Date(parseInt(dmy[2]), maaned, dag );
        alert(dag+" "+maaned+" "+dato.getDate()+" "+dato.getMonth());
        if (dato.getDate() == dag && dato.getMonth() == maaned) {
            return true;
        }
    return false;
};

function valider (formObjekt) {
    var dmy = [formObjekt.day.options[formObjekt.day.selectedIndex].value, formObjekt.month.options[formObjekt.month.selectedIndex].value, formObjekt.year.options[formObjekt.year.selectedIndex].value];
    if (!checkDato(dmy)) {
        alert("den indtastede dato duer ikke.\nBenyt formatet DD-MM-YYYY");
        return false;
    }
    return true;
};
</script>

<form method=post action="form.htm" onsubmit="return valider(this);">
<select name="day">
<option value="23">23</option>
</select>
<select name="month">
<option value="06">06</option>
</select>
<select name="year">
<option value="2003">2003</option>
</select>
<input type="submit">
</form>

roenving. Det ser ud til at virke, men jeg får to alert-bokse op. Den ene står der bare NaN NaN NaN NaN

Det må være fra denne linje, og det forstår jeg ikke, for så burde det ikke virke !-)

        alert(dag+" "+maaned+" "+dato.getDate()+" "+dato.getMonth());

-- den skal forøvrigt slettes, da den er brugt til debugging ...

jeg får i øvrigt også fejl hvis jeg vælger en dato der er god nok. Er det denne linies skyld?

-- nej det er ikke linjens skyld, men prøv lige at vise een af dine selects !-)

-- og lav også denne linje om:

        var dato = new Date(parseInt(dmy[2],10), maaned, dag );

Så er den hjemme, det viste sig at mine options slet ingen values havde. Det hjalp, da jeg fik dem på. Point til roenving hvis du smider et svar.

Velbekomme '-)

-- og tak for points ;~}