JS elende...

Door Eris - op 26-06-2006 15:04

1.473 views

function validate2(field,mode,field2) {
var vars = document.getElementById(field).value;
//alert(vars);
if(field2 == null || field2 == ''){
var com = 'test.php?mode='+mode+'&var='+vars;
}else{
var com = 'test.php?mode=repeat&var='+vars+'&var2='+ document.getElementById(field2).value;
}
xmlhttp.open('GET',com ,true);
xmlhttp.onreadystatechange=function() {
if (xmlhttp.readyState==4) {
if(xmlhttp.responseText == 1){
//maak achtergond groen
document.getElementById(field+'_img').innerHTML= '<img src="pass.gif" alt="help" width="14" height="14"/>';
r = 1;
}else{
//helaas we hebben een error
document.getElementById(field+'_img').innerHTML= '<img src="fail.gif" alt="help" width="14" height="14" onmouseover="domTT_activate(this, event, \'content\', xmlhttp.responseText, \'trail\', true, \'delay\', 0);"/>';
//geen error block weer
r = 2;
}

}
}
xmlhttp.send(null);
return r;
}

Error: r is not defined
Source File: http://www.onlinegastenboek.nl/new/ajax.js
Line: 80

Ik kom er niet achter wat ik fout doe..

De JS code past wel het "formulier aan zoals ik wens alleen hij geeft geen variable / waarde terug voor verder validatie van het formulier... Wat doe ik fout?

Legolas

26-06-2006 15:10

die r komt uit het niets, bestaat helemaal niet in validate2() (return)

Eris -

26-06-2006 15:47

r = 1;

en

r = 2;

Het rare is dat het goed gaat bij een regel omhoog...

Eris -

26-06-2006 16:00

Ben even verder klooien...

function validate2(field,mode,field2) {
 var vars = document.getElementById(field).value;
//alert(vars);
if(field2 == null || field2 == ''){
var com = 'test.php?mode='+mode+'&var='+vars;
}else{  
var com = 'test.php?mode=repeat&var='+vars+'&var2='+ document.getElementById(field2).value;
} 
xmlhttp.open('GET',com ,true);
 xmlhttp.onreadystatechange=function() {
  if (xmlhttp.readyState==4) {
   if(xmlhttp.responseText == 1){
     //maak achtergond groen
	 document.getElementById(field+'_img').innerHTML= '<img src="pass.gif" alt="help" width="14" height="14"/>';
    var r = '1';
    return r;
	 }else{
    //helaas we hebben een error
	 document.getElementById(field+'_img').innerHTML= '<img src="fail.gif" alt="help" width="14" height="14" onmouseover="domTT_activate(this, event, \'content\', xmlhttp.responseText, \'trail\', true, \'delay\', 0);"/>';
	 //geen error block weer
    var r = '2';
      return r;
	 }

  }
 }
 xmlhttp.send(null);
}
//submit
function validate_all(){
  //valideer alle velden
  var allow = true;
  var val1 = validate2('user','user');
  alert(val1);
}

Als ik van
var r = '1';
return r;

var r = '1';
alert(r);
return r;
maak krijg ik een alert window met 1 en daarna nog een met undifiened...

Voorbeeld
http://www.onlinegastenboek.nl/new/

Jelmer -

26-06-2006 16:07

xmlhttp.onreadystatechange=function() {
  if (xmlhttp.readyState==4) {
   if(xmlhttp.responseText == 1){
     //maak achtergond groen
     document.getElementById(field+'_img').innerHTML= '<img src="pass.gif" alt="help" width="14" height="14"/>';
    var r = '1';
    return r;
     }else{
    //helaas we hebben een error
     document.getElementById(field+'_img').innerHTML= '<img src="fail.gif" alt="help" width="14" height="14" onmouseover="domTT_activate(this, event, \'content\', xmlhttp.responseText, \'trail\', true, \'delay\', 0);"/>';
     //geen error block weer
    var r = '2';
      return r;
     }

  }

Zie, je maakt een functie aan. Dat is de minder leuke kant van asynchroon, deze functie loopt niet meer gelijk of in de vorige validate2-functie.

Hoe je dit moet oplossen weet ik zo even niet, [google]sajax[/google] (een of ander php-ajax framework) heeft er een oplossing voor weet ik, al weet ik zo niet hoe die werk.

Eris -

26-06-2006 16:09

Oke jammer daar gaat mijn gouden idee... Dan maar gewoon weer met een PHP refresh...

Jelmer -

26-06-2006 16:30

Eris schreef op 26.06.2006 16:09
Oke jammer daar gaat mijn gouden idee...

Sorry daarvoor.

Een andere oplossing is wel om met 1 functie alle velden te blokkeren tegen veranderingen, en de request uit te voren. Daarna, waneer er dus antwoord is gegeven, komt de 2e functie in beeld, die dan de velden weer unlockt (of wat het dan ook maar moet doen) Een aparte functie voor het verzoek en het antwoord dus.

Eris -

26-06-2006 16:32

Hij checkt nu sowiso alle velden afzonderlijk maar wil nog voor het verzenden kijken of er een of meerdere velden vergeten / niet voldoende zijn..

Jelmer -

26-06-2006 16:37

Dus voor ieder velt stuurt hij een vraag op? Gaat dat niet ontzettend veel server-energie kosten? Kan je niet beter alle velden in 1 keer controleren?

Eris -

26-06-2006 16:59

Ik vraag me af of het zo veel energie gaat kosten.. Het is niets meer dan 1 hit en dat was het geen lastige mysql berekening...

Als je het in een keer wilt gaat checken moet je toch een manier hebben om alle gegevens te gelijk te gaan versturen...

Legolas

26-06-2006 17:53

en de load voor de client?
ik heb een variant van live search gemaakt, en dat wordt soms iets te veel voor firefox :P

(live search = zoeken tijdens het typen van zoekwoorden)

Reageren

Inloggen om te reageren