Hjælp til kode

Det ville være nemmere med et link.

Generelt skrives "or" dog || og <> skal være !=

AND = &&

Sådan tror jeg din linje skal være:

if ((document.forms[0].antalkolli.value == 0 || document.forms[0].akdato2.value == '' || document.forms[0].kulturnr2.value == '' || document.forms[0].kulturnavn2.value == '' || document.forms[0].prkolli3.value == 0 || document.forms[0].kvalitet3.value == 0) || (document.forms[0].antalkolli4.value <> 0 && (document.forms[0].kulturnr2.value == '' || document.forms[0].kulturnavn2.value == '' || document.forms[0].prkolli4.value == 0 || document.forms[0].kvalitet4.value == 0))){

if () tager kun een parameter -- men den måde at skrive det på er ikke specielt overskuelig

- jeg ville nok prøve at skrive det à la:

function valider(f){
 
  if (f.antalkolli.value == 0) {
    return false;
  }
  if (f.akdato2.value == '') {
    return false;
  }
...

  }
  return true;
}

og så i form-tagget:

<form ... onsubmit="return valider(this);">

HTH
jes

Takker for points ;~}

og lige, for at det skal være korrekt (<> / !=):

if ((document.forms[0].antalkolli.value == 0 || document.forms[0].akdato2.value == '' || document.forms[0].kulturnr2.value == '' || document.forms[0].kulturnavn2.value == '' || document.forms[0].prkolli3.value == 0 || document.forms[0].kvalitet3.value == 0) || (document.forms[0].antalkolli4.value != 0 && (document.forms[0].kulturnr2.value == '' || document.forms[0].kulturnavn2.value == '' || document.forms[0].prkolli4.value == 0 || document.forms[0].kvalitet4.value == 0))){