Passage de variable refusée (incomprehensible)

Version imprimable

bonjour je créé un formulaire avec une question qui fonctionne un peu comme un captcha, j'ai un petit problème de variable. Mon premier alert() fonctionne bien il me donne un mais mon deuxième alert() est vide je n'arrive pas a sortir la variable de la fonction transport. Si quelqu'un aurait une idée merci car une fois ce problème de variable resolu mon script devrait fonctionner correctement. voici le code

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
	var err = "";
      if ($F('nom')=="") {new Effect.Appear(document.getElementById('inc_nom',1)); err= "1"}
	  else{new Effect.Fade(document.getElementById('inc_nom',1));}
 
	   if ($F('prenom')=="") {new Effect.Appear(document.getElementById('inc_prenom',1));err= "1"}
	   else{new Effect.Fade(document.getElementById('inc_prenom',1));}
 
 
	   if ($F('tel')=="") {new Effect.Appear(document.getElementById('inc_tel',1));err= "1"}
	   else{new Effect.Fade(document.getElementById('inc_tel',1));}
 
 
 
     new Ajax.Request('reponse.php', {   
         method:'post',  
         parameters: {  
             "question": $('question').value 
         },  
         onSuccess: function(transport) {  
 
            var response = transport.responseText || "error";
 
 
			 if (response!="ok") {
 
			 new Effect.Appear(document.getElementById('inc_question',1));
			 err= "1";			  
			 alert(err);
			  return err;
			 }
	  		else{new Effect.Fade(document.getElementById('inc_question',1));}
			          }  
     });  
 
	   alert(err);

21/10/2008, 10h30
E.Bzz

Bonjour,
ne la déclare qu'une seule fois, et en globale, et tu ne devrais plus avoir le problème ....

A+
21/10/2008, 10h45
tombabibel

non, dsl j'ai modifié mon code, le 2e "var err" était une faute de frappe.
Ma variable n'est déclarée qu'une fois au début du script.
Et ca ne fontionne toujours pas.
21/10/2008, 11h04
DoubleU

On ne peut pas faire de return sur le callback d'une fonction Ajax

Il faut que tu mettes ton traitement à la suite dans ton onSuccess

J'ai modifié mon script est j'ai toujours ce problème de variable qui ne veut pas sortir de ce onSuccess. Voici le nouveau code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
var err = "";
      if ($F('nom')=="") {new Effect.Appear(document.getElementById('inc_nom',1)); err= "1"}
	  else{new Effect.Fade(document.getElementById('inc_nom',1));}
 
	   if ($F('prenom')=="") {new Effect.Appear(document.getElementById('inc_prenom',1));err= "1"}
	   else{new Effect.Fade(document.getElementById('inc_prenom',1));}
 
 
	   if ($F('tel')=="") {new Effect.Appear(document.getElementById('inc_tel',1));err= "1"}
	   else{new Effect.Fade(document.getElementById('inc_tel',1));}
 
 
 
 
     new Ajax.Request('reponse.php', {   
         method:'post',  
         parameters: {  
             "question": $('question').value 
         },  
         onSuccess: function(transport) {		 
            var response = transport.responseText || "error";
				requ(response);	  		
		}  
     });  
 
 function requ(response){
  		if (response!="ok") {
 
			 new Effect.Appear(document.getElementById('inc_question',1));
			err= "1";			  
			 alert(err);
			 return err; // avec ou sans le return le resultat est le meme.			
		 }
	  	else{new Effect.Fade(document.getElementById('inc_question',1));
		}
 }
	   alert(err);

21/10/2008, 11h32
DoubleU

Citation:

J'ai modifié mon script est j'ai toujours ce problème de variable qui ne veut pas sortir de ce onSuccess.

Comme je l'ai écrit en gros en rouge, ca c'est pas possible à faire...
21/10/2008, 11h36
E.Bzz

Ta variable n'est visiblement toujours pas globale ... :roll:
Tu n'aurais pas besoin de faire de return ....
21/10/2008, 11h38
tombabibel

oui je suis d'accord, mais simplement me dire : "ce n'est pas possible" ne m'aide pas vraiment même si c'était marqué en rouge. J'aurais bien voulu un conseil pour faire autrement, un indice, une autre voie à prendre...
Ça m'aiderait beaucoup merci.
21/10/2008, 11h40
E.Bzz

Citation:

Envoyé par tombabibel

oui je suis d'accord, mais simplement me dire : "ce n'est pas possible" ne m'aide pas vraiment même si c'était marqué en rouge. J'aurais bien voulu un conseil pour faire autrement, un indice, une autre voie à prendre...
Ça m'aiderait beaucoup merci.

C'est ce qu'on fait depuis le début !
Tu suivrais les conseils, ça aiderait ...

Citation:

Envoyé par E.Bzz

Ta variable n'est visiblement toujours pas globale ... :roll:
Tu n'aurais pas besoin de faire de return ....

A+
21/10/2008, 11h41
tombabibel

Citation:

Envoyé par E.Bzz

Ta variable n'est visiblement toujours pas globale ... :roll:
Tu n'aurais pas besoin de faire de return ....

je dirias pour ma défense que ma variable est globale partotu sauf dans mon onSuccess. Car sans ce problème, le script marche parfaitement.
21/10/2008, 11h43
DoubleU

Citation:

Ta variable n'est visiblement toujours pas globale ...

Ca ne changerait rien, puisqu'au moment ou il arriverait à l'instruction suivant le Ajax.Request pour utiliser ta variable globale, la requete ne sera pas encore exécutée, donc ta variable pas peuplée.

Citation:

oui je suis d'accord, mais simplement me dire : "ce n'est pas possible" ne m'aide pas vraiment même si c'était marqué en rouge. J'aurais bien voulu un conseil pour faire autrement, un indice, une autre voie à prendre...
Ça m'aiderait beaucoup merci.

Comme je l'ai écrit en dessous de ce qui était marqué en rouge, il faut que tu mettes tous les traitements que tu comptais faire avec ta variable en dehors du onSuccess DANS le onSuccess. De facon à ce qu'ils ne soient exécutés que dès que ta requete aura bien retourné son resultat.

Citation:

Envoyé par E.Bzz

C'est ce qu'on fait depuis le début !
Tu suivrais les conseils, ça aiderait ...

A+

Ok dsl, peut être que le script entier pourrait vous aider a comprendre. Je fais un formulaire avec vérification. Et j'ai creer une sorte de captcha avec une question. Le script va vérifier la réponse sur la page reponse.php et si toutes les conditions sont réunies le mail est envoyé.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
function validation() 
{ 	 						
		var err = "";
      if ($F('nom')=="") {new Effect.Appear(document.getElementById('inc_nom',1)); err= "1"}
	  else{new Effect.Fade(document.getElementById('inc_nom',1));}
 
	   if ($F('prenom')=="") {new Effect.Appear(document.getElementById('inc_prenom',1));err= "1"}
	   else{new Effect.Fade(document.getElementById('inc_prenom',1));}
 
 
	   if ($F('tel')=="") {new Effect.Appear(document.getElementById('inc_tel',1));err= "1"}
	   else{new Effect.Fade(document.getElementById('inc_tel',1));}
 
 
 
 
     new Ajax.Request('reponse.php', {   
         method:'post',  
         parameters: {  
             "question": $('question').value 
         },  
         onSuccess: function(transport) {		 
            var response = transport.responseText || "error";
				requ(response);	  		
		}  
     });  
 
 function requ(response){
  		if (response!="ok") {
 
			 new Effect.Appear(document.getElementById('inc_question',1));
			err= "1";			  
			 alert(err);
 
		 }
	  	else{new Effect.Fade(document.getElementById('inc_question',1));
		}
 }
	   alert(err);
 
		if (err!="") {return false;}
 
 
		else { 
 
				 new Effect.Fade(document.getElementById('formu',1)); // effet de disparition de la div formu
				// $('formulaire').style.display = 'none'; // rend invisible le formulaire
				// document.getElementById('formulaire').remove(); // supprime le formulaire
 
					new Ajax.Request('envoi.php',{
						method: 'post',
						asynchronous: true,
						contentType:  'application/x-www-form-urlencoded',
						encoding:     'UTF-8',
						parameters: { tel : $F('tel'),message : $F('message'),nom : $F('nom'),prenom : $F('prenom'),type : $F('type')},
						onSuccess: function(){MailOK();}, 
						onFailure: function(){alert('ca na pas marché');}		});
					function MailOK(){
					setTimeout("new Effect.Appear(document.getElementById('textOK',1));",1000)  // effet d'apparition de la div textok apres une temps de 1 seconde		
					}
       } 
 
}

21/10/2008, 11h48
tombabibel

Citation:

Envoyé par DoubleU

Ca ne changerait rien, puisqu'au moment ou il arriverait à l'instruction suivant le Ajax.Request pour utiliser ta variable globale, la requete ne sera pas encore exécutée, donc ta variable pas peuplée.

Comme je l'ai écrit en dessous de ce qui était marqué en rouge, il faut que tu mettes tous les traitements que tu comptais faire avec ta variable en dehors du onSuccess DANS le onSuccess. De facon à ce qu'ils ne soient exécutés que dès que ta requete aura bien retourné son resultat.

je vais essayé de faire comme tu as dis. Je te tiens au courant.

Je vous ai écouté et mon script marche. Merci a vous !

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
function validation() 
{ 	 						
 
     new Ajax.Request('reponse.php', {   
         method:'post',  
         parameters: {  
             "question": $('question').value 
         },  
         onSuccess: function(transport) {		 
            var response = transport.responseText || "error";
 
			var err = "";
 
			if ($F('nom')=="") {new Effect.Appear(document.getElementById('inc_nom',1)); err= "1"}
			else{new Effect.Fade(document.getElementById('inc_nom',1));}
 
			if ($F('prenom')=="") {new Effect.Appear(document.getElementById('inc_prenom',1));err= "1"}
			else{new Effect.Fade(document.getElementById('inc_prenom',1));}
 
 
			if ($F('tel')=="") {new Effect.Appear(document.getElementById('inc_tel',1));err= "1"}
			else{new Effect.Fade(document.getElementById('inc_tel',1));}	  		
 
 
		   if (response!="ok") {					 
		   new Effect.Appear(document.getElementById('inc_question',1));
			err = "1";			  
 
		   }
			else{new Effect.Fade(document.getElementById('inc_question',1));
 
		   }
 
 
 
			if (err!="") {return false;}
 
 
			else { 
 
						 new Effect.Fade(document.getElementById('formu',1)); // effet de disparition de la div formu
						// $('formulaire').style.display = 'none'; // rend invisible le formulaire
						// document.getElementById('formulaire').remove(); // supprime le formulaire
 
							new Ajax.Request('envoi.php',{
								method: 'post',
								asynchronous: true,
								contentType:  'application/x-www-form-urlencoded',
								encoding:     'UTF-8',
								parameters: { tel : $F('tel'),societe : $F('societe'),nom : $F('nom'),prenom : $F('prenom'),type : $F('type')},
								onSuccess: function(){MailOK();}, 
								onFailure: function(){alert('ca na pas marché');}
							});
 
							function MailOK(){
 
									setTimeout("new Effect.Appear(document.getElementById('textOK',1));",1000)  // effet d'apparition de la div textok apres une temps de 1 seconde		
							}
 
				}		
 
		}
 
    });  
 
 
}

21/10/2008, 13h33
SpaceFrog

Citation:

Je vous ai écouté et mon script marche.

C'est vrai que sur developpez on a pour habitude de dire n'importe quoi :roll: