Formularüberprüfung

[anno]

Was haltet ihr davon?

function checkForm()
   {
   var f=document.kontaktformular;

if(!f.abteilung[0].checked && !f.abteilung[1].checked && !f.abteilung[2].checked)
         {
         alert("Bitte wählen Sie eine Fachabteilung aus.");
         return false;
         }

if(f.vorname.value=="")
         {
         alert("Vornamen eingeben");
         f.vorname.focus();
         return false;
         }

if(f.famname.value=="")
         {
         alert("Familiennamen eingeben");
         f.famname.focus();
         return false;
         }

if(f.email.value.length<=8)
         {
         alert("E-Mail Adresse eingeben");
         f.email.focus();
         return false;
         }

mailat=f.email.value.indexOf("@");
         maildot=f.email.value.indexOf(".");

if(f.email.value != "")
         {
               if((mailat==-1) || (maildot==-1))
               {
               alert("Bitte geben Sie eine korrekte E-Mail-Adresse ein.");
               f.email.focus();
               f.email.select();
               return false;
               }
         }

if(f.betreff.value=="")
         {
         alert("Bitte geben Sie einen Betreff an.");
         f.betreff.focus();
         return false;
         }

if(f.nachricht.value=="")
         {
         alert("Bitte nennen Sie uns Ihr Anliegen.");
         f.nachricht.focus();
         return false;
         }

window.open("danke.html","intFenster","height=150,width=250,left=430,top=350");
  }

[Cheatah]

Hi,

Was haltet ihr davon?

sehr schlechter Coding Style. Oder worauf willst Du hinaus?

Cheatah

[rfb]

Moin

Was haltet ihr davon?

nix, spätestens nach dem 2ten Nervrequester würde ich JavaScript deaktivieren und das Formular ungeprüft versenden.
Wenn überhaupt eine userseitige Prüfung dann überprüf doch alle Pflichteingabefelder auf einmal!

Gruß
rfb