Hvad fanden er galt

Du har hele 2 funktioner der hedder det samme??

<script type="text/JavaScript">
function valider(f) {
  if (f.adresse.value=="") {
    alert("Husk at udfylde adresse-feltet");
    f.adresse.focus();
    return false;
  }

    if (f.fornavn.value=="") {
    alert("Husk at udfylde Fornavn-feltet");
    f.fornavn.focus();
    return false;
  }

    if (f.efternavn.value=="") {
    alert("Husk at udfylde Efternavn-feltet");
    f.efternavn.focus();
    return false;
  }

    if (f.bynavn.value=="") {
    alert("Husk at udfylde Bynavn-feltet");
    f.bynavn.focus();
    return false;
  }

    if (f.postnr.value=="") {
    alert("Husk at udfylde Postnr-feltet");
    f.postnr.focus();
    return false;
  }

    if (f.telefon.value=="") {
    alert("Husk at udfylde Telefon-feltet");
    f.telefon.focus();
    return false;
  }

    if (f.email.value=="") {
    alert("Husk at udfylde Email-feltet");
    f.email.focus();
    return false;
  }
 
  //everything went out peachy :)
  return true;
}
</script>


<form method="POST" action="mailForm.asp" onsubmit="return valider(this)">

virker det bedre? :) (uden at have testet det)

nej den vil gerne valider adresse men hvis man udfylder den køre den bare vidre

Hvis du har gjort som o-zone skriver burde det virke.
Og hvis du kun skal kontrollere om en masse felter af samme type er udfyldt, kunne du forkorte det til noget i stil med:

<script type="text/javascript">
function valider(f) {
var a = f.getElementsByTagName('input');
  for(i=0;i<a.length;i++){
    if (a[i].type=="text" && a[i].value=="") {
      alert("Husk at udfylde " + a[i].name + "-feltet");
      a[i].focus();
    return false;
    }
  }
  //everything went out peachy :)
  return true;
}
</script>

<form method="POST" action="mailForm.asp" onsubmit="return valider(this)">
  <input type="text" name="adresse"><br>
  <input type="text" name="fornavn"><br>
  <input type="text" name="efternavn"><br>
  <input type="text" name="bynavn"><br>
  <input type="text" name="postnr"><br>
  <input type="text" name="telefon"><br>
  <input type="email" name="email"><br>
  <input type="submit" value="Gojkænd">
</form>

/psc

Super lækker kode psykochiken! Respekt goes your way! :)

>> o-zone: takker og bukker ;o)

fint tak

lægger du lige et svar psykochicken så vi kan få det afsluttet



ja og tak for hjælpen :)

ok - men giv halvdelen til o-zone, han kom jo først med en løsning :o)

/psc

ja. det skal jeg nok

..og tak for point :o)