Discussion :

[Myfred]

Bonjour à tous,
voilà déjà quelque heures que je me galère à faire sortir d'une fonction un objet Json.

J'utilise la librairie prototype pour mon script.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
function getJson()
{
 
	new Ajax.Request('js/ajax.php', {
	    parameters:'id_appel=8',
	    asynchronous: false,
	    onComplete: function(reqa) {
		    if (200 == reqa.status) 
		    	return reqa.responseJSON;
		}
 
	});
}

Si par exemple, avant le "return", je fais une alert(reqa.responseJSON.pseudo), ca m'affiche bien le pseudo.

par contre si je fais

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
var testJson = getJson();
alert(testJson.pseudo);

ben là, il me met 'undefined"

Je sais pas d'ou ca vient, surtout que ca marche quand on est dans la fonction...
Je travail en local, ca pose un pb?
au niveau du php voici ce qui est retourné (simplifié):

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
<?php
header('Content-type: application/json');
echo '{"pseudo" : "jean", "age" : "20" }';
?>

Si quelqu'un à une idée....
Merci par avance

[DoubleU]

C'est normal, tu ne peux pas faire de return sur une réponse Ajax.

Il faut que tu mettes ton traitement complet qui utilise ta réponse dans le handler onComplete

[SpaceFrog]

ajax ne peut retourner que du string ...
à javascript ensuite de l'exploiter

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
var MonObj=window[xhr.responseText]
alert (MonObj)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
var MonObj=eval(xhr.responseText)
alert (MonObj)

[Bovino]

ajax ne peut retourner que du string ...
à javascript ensuite de l'exploiter

Tout à fait...
Si Prototype te retourne du JSON sur une requête Ajax, j'imagine que c'est en utilisant sa méthode toJSON sur le responseText...

[Myfred]

marche toujours pô:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
function getJson()
{
 
	new Ajax.Request('js/ajax.php', {
	    parameters:'id_appel=8',
	    asynchronous: false,
	    requestHeaders: {Accept: 'application/json'},
	    onComplete: function(reqa) {
		    if (200 == reqa.status) 
		    	var test = reqa.responseText.evalJSON();
		    	return test;
 
		}
 
	});
 
}

prototype documentation:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 
/*Using JSON with Ajax
 
Using JSON with Ajax is very straightforward, simply invoke String#evalJSON on the transport’s responseText property:
*/
 
 
new Ajax.Request('/some_url', {
  method:'get',
  onSuccess: function(transport){
     var json = transport.responseText.evalJSON();
   }
});
 
//If your data comes from an untrusted source, be sure to sanitize it:
 
 
new Ajax.Request('/some_url', {
  method:'get',
  requestHeaders: {Accept: 'application/json'},
  onSuccess: function(transport){
    var json = transport.responseText.evalJSON(true);
  }
});

[DoubleU]

C'est normal, tu ne peux pas faire de return sur une réponse Ajax.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
	    onComplete: function(reqa) {
		    if (200 == reqa.status) 
		    	var test = reqa.responseText.evalJSON();
		    	return test;// pas possible ca...

[Myfred]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
	    onComplete: function(reqa) {
		    if (200 == reqa.status) 
		    	var test = reqa.responseText.evalJSON();
		    	return test;// pas possible ca...

c'est étonnant qu'on ne puisse pas faire ca, mais je te fais confiance.
Que faut il faire dans ce cas là?

[DoubleU]

Il faut que tu mettes ton traitement complet qui utilise ta réponse dans le handler onComplete

En gros, tu remplaces ton return test par tout le code qui devrait être exécuté avec la réponse que tu voulais retourner.

[SpaceFrog]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
onComplete: function(reqa) {
		    if (200 == reqa.status) 
		    	window['test'] = reqa.responseText.evalJSON();

?

[Myfred]

oui mais le problème c'est que je ne veux pas faire ca.
J'ai pas loin de 300 lignes derrière, et je veux garder cette fonction pour mettre a jour régulièrement le json.

Je vais enlever le "var" devant le nom de variable pour récuperer une global.
Je pense que c le plus simple à faire

[SpaceFrog]

heu...

Je vais enlever le "var" devant le nom de variable pour récuperer une global.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

window['test'] = reqa.responseText.evalJSON();

le fait d'enlever var devant ne rends pas la variable globale ...


mais le fait de declarer

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

window['test']

rend la varible test globale ....

[Myfred]

je connais pas ce window['test'], tu peux m'en dire plus stp?

[DoubleU]

Ca changera rien à cause de l'asynchronisme de la fonction.

Si tu fais:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
	new Ajax.Request('js/ajax.php', {
	    parameters:'id_appel=8',
	    asynchronous: false,
	    requestHeaders: {Accept: 'application/json'},
	    onComplete: function(reqa) {
		    if (200 == reqa.status) 
		    	test = reqa.responseText.evalJSON();
                        // ou window['test'] = reqa.responseText.evalJSON(), c'est pareil
 
		}
 
                alert(test);

Au moment ou tu fera le alert, tu n'auras pas encore ta réponse, donc alert sera encore undefined.

Le plus simple pour ton cas, c'est de mettre en paramètre de ta fonction de 300 lignes ton json et de la faire exécuter sur le onComplete.

[DoubleU]

le fait d'enlever var devant ne rends pas la variable globale ...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
function foo(){
 a = "foooo";
}
 
foo();
 
alert(a);

[SpaceFrog]

pas du tout syntaxiquement correct et surtout pas crossbrowser !

[DoubleU]

c'est pas incorrect, c'est juste déconseillé

Sinon, avec quel navigateur ca plante que je fasse gaffe la prochaine fois?

[SpaceFrog]

sur IE et FFX ça passe

a mon grand étonnement !

a tester sur safairi et Opéra ...

Mais personnellement je fuis ce genre de syntaxe

[DoubleU]

Sur opéra aussi ca passe, donc ca m'étonnait que tu dises que c'était pas crossbrowser.

Après, on est bien d'accord que c'est crade comme écriture ^^

[SpaceFrog]

Méa culpa ...

j'étais persuadé m'être déja fait avoir avec ce genre de déclaration sans var ....
avec un undefined ....