Udvidet validering

Du kan prøv at rette funktionen til dette her, så den tjekker både filen og checkboxen, der også rette i <form>-tagen onsubmit:


<script type="text/javascript">
function validerFil(theform){
    reg = /\.jp[e]?g$|[\w{0}]/gi;
    if(!theform.strFile.value.match(reg)){
        alert('Vi modtager kun billeder af typen JPG eller JPEG!!');theform.strFile.focus();theform.strFile.select();return false;
    }
    if (!theform.accepter.checked) {
        alert('Du skal læse og accepter reglerne!!');
        return false;   
    }
    return true;
}
</script>
<form method="POST" enctype="multipart/form-data" action="UploadScript1.asp" onsubmit="return validerFil(this);">
    Dit navn<br>
    <input type="text" name="navn" size="30"><br>
    Din e-mail<br>
    <input type="text" name="email" size="30"><br>
    <input type="checkbox" name="accepter" value="ja">Jeg har læst
    <a href="#readme">nedenstående tekst</a>, og accepterer det der står i den.
    <br>
    <br>
    <input type="FILE" size="20" name="strFile" class="upload" style="height: 18px; border: 1px solid #000000"><br>
    <input type="SUBMIT" value="Send billede" name="func" class="upload" style="height: 18px; border: 1px solid #000000">
    <input type="hidden" value="Action" name="aktivitet">
    <input type="hidden" value="65" name="aID">
</form>

Hej Eagleeye

Der skal også valideres for at navn og email er udfyldt med noget, gerne med en gyldig e-mail, kan du fikse det.

function validateInput(f) {
  if (f.navn.value.length > 0) {
  } else {
    alert ("Feltet Navn skal udfyldes.");
    f.user.focus();
    return false;
  }
  if (f.theEmail.value != "") {
    if (!validateEmail(f.theEmail.value)) {
      alert("Skriv venligst en gyldig e-mail adresse");
      f.theEmail.focus();
      return false;
    }
  }
  return true;
}

// validate typed in email
function validateEmail(i) {
  var IsRegEx = false;
  if (window.RegExp) {
    var tempStr = "a";
    var tempReg = new RegExp(tempStr);
    if (tempReg.test(tempStr)) {
      IsRegEx = true;
    }
  }
  if (IsRegEx != true) {
    return (i.indexOf(".") > 2) && (i.indexOf("@") > 0);
  } else {
    var r1 = new RegExp("(@.*@)|(\\.\\.)|(@\\.)|(^\\.)");
    var r2 = new RegExp("^.+\\@(\\[?)[a-zA-Z0-9\\-\\.]+\\.([a-zA-Z]{2,3}|[0-9]{1,3})(\\]?)$");
    return (!r1.test(i) && r2.test(i));
  }
}

og
return validate(this)

f.user.focus();  = f.navn.focus();

lidt for meget copy paste :)

Du kan lave scriptet sådan her:


<script type="text/javascript">
function validerFil(theform){
    reg = /\.jp[e]?g$|[\w{0}]/gi;
    if(!theform.strFile.value.match(reg)){
        alert('Vi modtager kun billeder af typen JPG eller JPEG!!');theform.strFile.focus();theform.strFile.select();return false;
    }
    if (!theform.accepter.checked) {
        alert('Du skal læse og accepter reglerne!!');
        return false;   
    }
    reg = /^[\w]{1}[\w\.\-_]*@[\w]{1}[\w\-_\.]*\.[\w]{2,6}$/i;
    if(!theform.email.value.match(reg)){
        alert('Email skal udfyldes og være gyldig!!');theform.email.focus();theform.email.select();return false;
    }

    if (!theform.navn.value.length > 0) {
      alert("Du glemte at indtaste dit navn !");
      theform.navn.focus();theform.navn.select();return false;
    }
    return true;
}
</script>

Tak til begge, jeg har fortsat med EagleEyes forslag