hvorfor virker det ikke?

function validate(){
  var isError = false
  for (i=0;i<document.tilmeld.sp.length;i++){
 
  if (document.tilmeld.sp[i].checked == true){
    isError = true
  }
  if(!document.tilmeld.navn.value.length>0){
        isError = true
    }
  if(isError==false){
    alert(\'Du mangler at udfylde et eller flere af felterne\')
    }
  return isError
}

hmm mangler et mere }

function validate(){
  var isError = false
  for (i=0;i<document.tilmeld.sp.length;i++){
 
      if (document.tilmeld.sp[i].checked == true) {

        isError = true
      }
  }
  if(!document.tilmeld.navn.value.length>0){
        isError = true
    }
  if(isError==false){
    alert(\'Du mangler at udfylde et eller flere af felterne\')
    }
  return isError
}

den kommer med \'document.tilmeld.sp\' er ikke et objekt

Hej igen!

Logikken vender forkert. Nu kalder jeg den isOK, og sætter den til true hvis der er klikket på en radioknap. Derefter bliver den sat til false hvis textfeltet er tomt

Og så har jeg tilføjet de korrekte body og head tags

<html>
<head>
<script language=\"JavaScript\">
<!--
function validate(){
  var isOK = false
  for (i=0;i<document.tilmeld.sp.length;i++){
 
   if (document.tilmeld.sp[i].checked == true) isOK = true
   }
  if(document.tilmeld.navn.value.length == 0){
        isOK = false
    }
  if(isOK==false){
    alert(\'Du mangler at udfylde et eller flere af felterne\')
    }
  return isOK
}
// -->
</script>
</head>
<body>
<form name=tilmeld onSubmit=\"return validate(tilmeld);\" >
<input type=radio name=sp value=\"Ja\">
<input type=radio name=sp value=\"Nej\">
<INPUT TYPE=\"text\" NAME=\"navn\">
<input type=\"Submit\">
</form>
</body>
</html>

Det er jo lige til at græde over, hvad have jeg glemt/gjort forkert?

if(!document.tilmeld.navn.value.length>0){
        isError = true
    }
satte isError = true hvis feltet var tomt, altså hvis der var fejl

for (i=0;i<document.tilmeld.sp.length;i++){
      if (document.tilmeld.sp[i].checked == true) {
         isError = true
      }
  }
satte isError = true hvis der var valgt noget, altså hvis der IKKE var fejl.

Det er mere logisk at have en variabel der er true hvis alt er ok, fordi du så umiddelbart kan returnere den til onSubmit, og bremse formen med den

Det kan være MEGET svært at validere en stor form med mange kombinationer og forbehold, denne her var ikke slem, men ... :o)

så er jeg lidt mere med, js er endnu ikke min stærke side, det kommer forhådentligt ;o), jeg takker endnu en gang for hjælpen