Functies samenvoegen

[Wiv]

Hallo,

Weet iemand hoe ik onderstaande functies kan samenvoegen?

Functie 1:

function checkForm( ) 
{ 
valid = true; 

if ( document.invoer.adviseur.selectedIndex == 0 ) 
{ 
document.getElementById("adviseurError").style.display = "block"; 

valid = false; 

} 

return valid; 
}

Functie2:

function checkForm2( ) 
{ 
valid = true; 

if ( document.invoer.terecht.checked == false ) 
{ 
document.getElementById("terechtError").style.display = "block"; 

valid = false; 

} 

return valid; 
}

 

[NielsDesmet]

Wat bedoel je precies met "Samenvoegen"?

 

[Wiv]

Nou ik wil van beide functies 1 functie maken omdat ik via onsubmit maar maximaal 1 functie kan aanroepen.

Aangezien de functies hetzelfde doen, alleen voor een ander veld in het formulier denk ik dat het mogelijk moet zijn om ze samen te voegen, ik weet alleen niet hoe...

 

[Glest]

Simpelweg het if block van functie 2 onder het if blok van de ander plakken. De variable valid is een boolean die op false wordt gezet wanneer er een fout is. Hij wordt geinitialiseerd op "true" (geen fouten) en wordt op "false" gezet wanneer er fouten zijn. Vervolgens returned de functie de waarde van de variable valid. Op die manier kun je dus makkelijk meerdere checks invoegen in één functie.

function checkForm( ) 
{ 
  valid = true; 

  if ( document.invoer.adviseur.selectedIndex == 0 ) 
  { 
    document.getElementById("adviseurError").style.display = "block"; 
    valid = false; 
  } 

  if ( document.invoer.terecht.checked == false ) 
  { 
    document.getElementById("terechtError").style.display = "block"; 
    valid = false; 
  } 

  return valid; 
}

Daarnaast, als de functies te gecompliceerd zijn om samen te voegen kun je altijd iets als dit doen:

function realCheckForm()
{
  return checkForm() && checkForm2();
}

 

[Wiv]

Heel erg duidelijk op deze manier, ook de uitleg is duidelijk. Nu snap ik er weer wat meer van!

Bedankt!

 

[Tha Devil]

Ik meende dat je meerdere functie gewoon aan kan roepen.
Kwestie van een ; achter de vorige functie zetten.

onsubmit="checkForm();checkForm2()"

Uiteraard is het gebruik maken van een enkele functie natuurlijk vele malen beter maar het is volgens mij dus wel mogelijk.

 

[Glest]

dat kan inderdaad, maar dan wordt het formulier gewoon verstuurd ook al geeft een van de twee functies aan dat het formulier niet verstuurd moet worden.

Wat wel zou kunnen is onsubmit="return check1()&&check2();"