form validation if-else

Status
Niet open voor verdere reacties.

Michiel1969

Gebruiker
Lid geworden
27 nov 2010
Berichten
5
Ik heb onderstaand stuk met 2 verschillen if-else reeksen in een scriptje staan. Het probleem is dat het niet werkt. Er hangt nog een submit button aan en als daarop geklikt wordt verschijnen keurig de alerts over de niet ingevulde velden, maar ondanks dat er niets is ingevuld volgt ook de alert over "koekebakker". Ik wil dat eerst alle velden worden ingevuld voordat de 2e alert volgt. Hoe doe ik dat?

var uitkomst3 = d-uitkomst2;

if(document.getElementById("a").value == "")
alert("Je hebt het veld a niet ingevuld.")

if(document.getElementById("b").value == "")
alert("Je hebt veld b niet ingevuld.")

if(document.getElementById("c").value == "")
alert("Je hebt veld c niet ingevuld.")

else;

if (uitkomst3 < 0)
alert("je bent een koekebakker");
else
alert("toperrrrr");
 
als je dit eens probeert, een check variabele die alleen maar true kan zijn als alle velden zijn ingevuld.

Ik ben niet sterk met javascript, maar ik geloof dat het op deze manier zou moeten kunnen!

var uitkomst3 = d-uitkomst2;
var check = false;

if(document.getElementById("a").value == "") {
alert("Je hebt het veld a niet ingevuld.")
check = false;
} else { check =true; }

if(document.getElementById("b").value == "") {
alert("Je hebt veld b niet ingevuld.")
check = false;
} else { check =true; }

if(document.getElementById("c").value == "") {
alert("Je hebt veld c niet ingevuld.")
check = false;
} else { check =true; }

if(true) {
if (uitkomst3 < 0) {
alert("je bent een koekebakker");
} else {
alert("toperrrrr");
}
}
 
Het kan iets korter (en netter, maar dat is stijl):[JS]var nietIngevuld = false; // neem aan dat alles is ingevuld

// zoals phobia al aangaf, in _elke_ if-check zet je de nietIngevuld aan:

if(document.getElementById("c").value == "") {
alert("Je hebt veld c niet ingevuld.")
nietIngevuld= true;
}// geen else nodig!


// ------


if (uitkomst3 < 0 && nietIngevuld == true)
{
alert("je bent een koekebakker");
}else{
alert("toperrrrr");
}[/JS]
 
post eens wat je gemaakte hebt, het nieuwe wat niet werkt!
 
Status
Niet open voor verdere reacties.
Terug
Bovenaan Onderaan