Salvete,
hab's nur ueberflogen, aber mal kurz einen Satz zur Semantik folgenden Codes:
if (document.Registrierung.Nachname.value!="")
{
if (document.Registrierung.Vorname.value!="")
{
if (document.Registrierung.Alter.value!="" && isNaN(Alter)==false)
{
if (document.Registrierung.Wohnort.value!="")
{
if (document.Registrierung.Postleitzahl.value!="" && isNaN(PLZ)==false)
{
if (document.Registrierung.Box.checked==true)
{
window.alert("Überprüfung erfolgreich abgeschlossen.");
}
else {
window.alert("Sie müssen die AGB's akzeptieren");
}
}
else if (document.Registrierung.Postleitzahl.value=="" || isNaN(PLZ)==true)
{
Fehlermeldung("Postleitzahl");
}
}
else {
Fehlermeldung("Wohnort");
}
}
else if (document.Registrierung.Alter.value=="" || isNaN(Alter)==true)
{
Fehlermeldung("Alter");
}
}
else {
Fehlermeldung("Vorname");
}
}
Das ist furchtbar ;) Sehr unuebersichtlich und fehleranfaellig.
Ersichtlicher wird es, wenn du die Uebepruefungen nicht so arg
verschachtelst. Also in etwa so:
if (document.Registrierung.Nachname.value=="")
{
Fehlermeldung("Nachname");
return false;
}
if (document.Registrierung.Vorname.value=="")
{
Fehlermeldung("Vorname");
return false;
}
[...]
Wie gesagt, es ging mir jetzt nur um die Semantik. Dass die Ueberpruefungen
nicht wirklich taugen ist eine andere Sache.
MfG,
Sympatisant
--
"Non dura iubeantur, non prohibeantur inpura."
"Non dura iubeantur, non prohibeantur inpura."