[AJAX] ma fonction ne retourne rien

Version imprimable

bonjour
voici ma fonction :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
function getmot(){
	var unmot;
	var requeteHttp=getRequeteHttp();
	if (requeteHttp==null){
    	alert("Impossible d'utilier Ajax sur ce navigateur")	
	}else{
    	requeteHttp.open('GET','mot.php',true);
		requeteHttp.onreadystatechange=function(){
			if (requeteHttp.readyState==4){
				if (requeteHttp.status==200){
					unmot = requeteHttp.responseText;
					alert(unmot);
				}else{
					alert("probleme pendant l'execution");
				}
			}
		}
	requeteHttp.send(null);
	}
	return unmot;
}

le alert(unmot); affiche bien une valeur mais quand je lance ma fonction dans un script elle renvoi undefined je ne comprend pas

merci de votre aide

Tout simplement parce que ton return est exécuté avant la réponse du serveur, il faut l'intégrer dans le callback de ton appel Ajax :
Code:

1 2 3 4 5 6 if (requeteHttp.status==200){ unmot = requeteHttp.responseText; return unmot; }else{ alert("probleme pendant l'execution"); }

toujours le problème avec la modif

j'appelle ma fonction avec un alert(getmot()); le premier alert me renvoi le mot le deuxième alert marque undefined

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
function getmot(){
	var unmot;
	var requeteHttp=getRequeteHttp();
	if (requeteHttp==null){
    	alert("Impossible d'utilier Ajax sur ce navigateur")	
	}else{
    	requeteHttp.open('GET','mot.php',true);
		requeteHttp.onreadystatechange=function(){
			if (requeteHttp.readyState==4){
				if (requeteHttp.status==200){
					unmot = requeteHttp.responseText;
					alert(unmot);
					return unmot;
				}else{
					alert("probleme pendant l'execution");
				}
			}
		}
	requeteHttp.send(null);
	}
 
}

Bonjour,
le problème est dû à la portée de ta variable unmot : elle n'est pas la même que celle déclarée en locale dans la fonction getmot().
Pour le pas avoir de problème, il suffit ... de ne pas l'utiliser :mrgreen:
Code:

1 2 3 4 5 6 7 8 9 10 requeteHttp.onreadystatechange=function(){ if (requeteHttp.readyState==4){ if (requeteHttp.status==200){ alert(requeteHttp.responseText); return requeteHttp.responseText; }else{ alert("probleme pendant l'execution"); } } }
EDIT : vue ta fonction initiale, il te suffit de déclarer unmot comme variable globale ...

A+

non toujours pareil..

quoi que je maite dans le returne mais une chaine (return "hello";) sa ne renvoi rien que je le place en desous du alert(requeteHttp.responseText);

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
function getmot(){
	var requeteHttp=getRequeteHttp();
	if (requeteHttp==null){
    	alert("Impossible d'utilier Ajax sur ce navigateur")	
	}else{
    	requeteHttp.open('GET','mot.php',true);
		requeteHttp.onreadystatechange=function(){
			if (requeteHttp.readyState==4){
				if (requeteHttp.status==200){
					alert(requeteHttp.responseText);
					return requeteHttp.responseText;
				}else{
					alert("probleme pendant l'execution");
				}
			}
		}
	requeteHttp.send(null);
	}
 
}

Citation:

Envoyé par E.Bzz

EDIT : vue ta fonction initiale, il te suffit de déclarer unmot comme variable globale ...

Citation:

Envoyé par trigone

non toujours pareil..

Arf ! :aie:
Je remtets dans l'ordre :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 var unmot; function getmot(){ var requeteHttp=getRequeteHttp(); if (requeteHttp==null){ alert("Impossible d'utilier Ajax sur ce navigateur") }else{ requeteHttp.open('GET','mot.php',true); requeteHttp.onreadystatechange=function(){ if (requeteHttp.readyState==4){ if (requeteHttp.status==200){ unmot = requeteHttp.responseText; alert(unmot); return unmot; }else{ alert("probleme pendant l'execution"); } } } requeteHttp.send(null); } }
Attention par contre, s'il doit y avoir plusieurs appels simultanés ...

A+

22/12/2008, 17h11
trigone

toujour pareil undefined

Le problème, c'est que return unmot renvoie bien la variable, mais à la fonction getMot(), pas à celle qui appelle getMot(), il faut procéder différemment...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
function getmot(){
	var requeteHttp=getRequeteHttp();
	if (requeteHttp==null){
    	alert("Impossible d'utilier Ajax sur ce navigateur")	
	}else{
    	requeteHttp.open('GET','mot.php',true);
		requeteHttp.onreadystatechange=function(){
			if (requeteHttp.readyState==4){
				if (requeteHttp.status==200){
					afficheMot(requeteHttp.responseText);
				}else{
					alert("probleme pendant l'execution");
				}
			}
		}
	requeteHttp.send(null);
	}
 
}
function afficheMot(mot){
    alert(mot);
}

22/12/2008, 17h22
RomainVALERI

Tiens ça me rappelle quelque chose.. ^^ (probablement cette discussion)

--> passe par une fonction de callback pour exécuter l'alert après réponse ajax : car la fin de la fonction getMot intervient avant le retour de cette réponse, donc un return final ne convient pas (ni d'ailleurs un return placé dans la définition de onreadystatechange, car ce n'est plus à getMot que ce return fait alors référence... enfin bon tout est dans le lien :mrgreen: merci encore a marcha et bovino en passant d'ailleurs)
22/12/2008, 17h37
trigone

en gros cette fonction ne devrai pas afficher elle doit juste maitre un mot dans une variable globale le problème c'est que le traitement continue sans attendre que la variable sois initialiser ...

comment faire pour que le script attend que le mot soi recuperai pour que sa continue?
22/12/2008, 17h38
E.Bzz

Citation:

Envoyé par trigone

comment faire pour que le script attend que le mot soi recuperai pour que sa continue?

Il faut faire ton appel Ajax en mode synchrone et non asynchrone comme actuellement ...

A+
22/12/2008, 17h45
Bovino

Citation:

Envoyé par E.Bzz

Il faut faire ton appel Ajax en mode synchrone et non asynchrone comme actuellement ...

A+

Code:

requeteHttp.open('GET','mot.php',false);
22/12/2008, 17h50
E.Bzz

Je n'avais pas détaillé car la modif ne se résume pas à ça (plus de onreadystatechange) ...

A+

Code:

1
2
3
4
5
6
requeteHttp.open('GET','mot.php',false);
requeteHttp.send(null);
if(requeteHttp.readyState == 4) {
unmot = requeteHttp.responseText;
alert(unmot);
}

A+

22/12/2008, 18h06
trigone

merci beaucoup

encore une foi developpez.com me prouve que j'ai encore plein de chose a apprendre...