Partage de variable entre fonction

Bonjour à tous,

Voilà, j'ai un script js me permettant de récupérer via ajax (j'utilise jQuery pour plus de facilité) une valeur que je doit ensuite mettre dans un attribut de l'objet duquel découle cette même fonction.
L'ennui, c'est que JS n'arrive pas à faire une chose pareil. Serais-ce une histoire de callback ? (pas bien compris le principe).

Mon script :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

var monObj = function(){ var maVar = null; // Attribut privé à changer   sendAjaxRequest = function(){ $.ajax({ url : ((new Error()).fileName.replace(/(^.*)(\/)(.*$)/gi, "$1$2"))+"monfichierserver.php", type : "POST", data : "bla=blabla", cache : false, success : function(valR){ if(valR!==0 && valR!==null){ maVar = valR; } } });   return (maVar!==null); // CE TEST VAUT FALSE !!! } };

---

En gros, l'attribut maVar doit se faire remplacer par le résultat de ma requette, mais impossible...
Je précise que le problème ne viens pas d'ajax qui me retourne la bonne valeur comme prévu. Je n'arrive juste pas bien a comprendre la portée des variables en objet dans js...

D'avance merci pour votre aide...

essaye de attacher ta variable à la fonction ..

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

var monObj = function(){ this.maVar = null; // Attribut privé à changer   sendAjaxRequest = function(){ $.ajax({ url : ((new Error()).fileName.replace(/(^.*)(\/)(.*$)/gi, "$1$2"))+"monfichierserver.php", type : "POST", data : "bla=blabla", cache : false, success : function(valR){ if(valR!==0 && valR!==null){ monObj.maVar = valR; } } });   return (nomObj.maVar!==null); // CE TEST VAUT FALSE !!! } };

---

Ben surtout, attends d'avoir récupéré la valeur avant d'essayer de l'affecter...
:fleche: Un classique AJAX : utiliser les données au bon moment !

Citation:

---

JS n'arrive pas à faire une chose pareil

---

:roll:
Ce serait pas plutôt : "je n'arrive pas à faire une chose pareille avec JS" ?
:aie:

ha oui le return dans le success ^^

Merci à vous deux.

Citation:

---

Envoyé par Bovino

:roll:
Ce serait pas plutôt : "je n'arrive pas à faire une chose pareille avec JS" ?
:aie:

---

En effet :p

@SpaceFrog : Si je fais ce que tu préconise, maVar n'est alors plus un attribut privé et se voi donc accessible et modifiable a l'extérieur de l'objet, ce que je ne souhaite absolument pas...
De plus, il me semble que faire un return dans une fonction anonyme est déprécié...

@Bovino : Merci pour le lien. Il est vrai que je fais (hélas) encore ce genre d'erreur de base... Que préconises-tu dans ce contexte alors ? Car mettre l'instruction ajax dans une autre fonction serait juste inutile à mon sens puisque j'en ai besoin uniquement pour cette fonction.


Plus généralement, travaillant avec Komodo Edit, ce dernier m'a indiqué le problème de return dans le success ajax. Est-ce un réel problème ? Quels changements implique ?

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

var monObj = function(){ var maVar = null; // Attribut privé à changer   sendAjaxRequest = function(){ $.ajax({ url : ((new Error()).fileName.replace(/(^.*)(\/)(.*$)/gi, "$1$2"))+"monfichierserver.php", type : "POST", data : "bla=blabla", cache : false, success : function(valR){ return (valR!==0 && valR!==null) ? valR : null; } });   // Je met quoi ici ? maVar = valR; ou maVar = avant le $.ajax ?   return (maVar!==null); // CE TEST VAUT FALSE !!! } };

---

D'avance merci...

Citation:

---

Envoyé par FotoXe33

Bonjour à tous,

Voilà, j'ai un script js me permettant de récupérer via ajax (j'utilise jQuery pour plus de facilité) une valeur que je doit ensuite mettre dans un attribut de l'objet duquel découle cette même fonction.
L'ennui, c'est que JS n'arrive pas à faire une chose pareil. Serais-ce une histoire de callback ? (pas bien compris le principe).

Mon script :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

var monObj = function(){ var maVar = null; // Attribut privé à changer   sendAjaxRequest = function(){ $.ajax({ url : ((new Error()).fileName.replace(/(^.*)(\/)(.*$)/gi, "$1$2"))+"monfichierserver.php", type : "POST", data : "bla=blabla", cache : false, success : function(valR){ if(valR!==0 && valR!==null){ maVar = valR; } } });   return (maVar!==null); // CE TEST VAUT FALSE !!! } };

---

En gros, l'attribut maVar doit se faire remplacer par le résultat de ma requette, mais impossible...
Je précise que le problème ne viens pas d'ajax qui me retourne la bonne valeur comme prévu. Je n'arrive juste pas bien a comprendre la portée des variables en objet dans js...

D'avance merci pour votre aide...

---

essaye simplement d'ajouter à ton code initial, le paramètre async : false à ta requête ajax. (je pense que ça devrait résoudre ton problème, même si les requêtes synchrones ne sont pas ce qu'il y a de mieux puisqu'elles bloquent ton code jusqu'à l'obtention d'une réponse.)

Citation:

---

Envoyé par Willpower

essaye simplement d'ajouter à ton code initial, le paramètre async : false à ta requête ajax. (je pense que ça devrait résoudre ton problème, même si les requêtes synchrones ne sont pas ce qu'il y a de mieux puisqu'elles bloquent ton code jusqu'à l'obtention d'une réponse.)

---

Une méthode qui fait de l'ajax est asynchrone, donc elle ne doit pas retourner de valeur, mais plutôt appeler une methode de callback qui recevra cette valeur en paramètre.

Citation:

---

Envoyé par Willpower

essaye simplement d'ajouter à ton code initial, le paramètre async : false à ta requête ajax. (je pense que ça devrait résoudre ton problème, même si les requêtes synchrones ne sont pas ce qu'il y a de mieux puisqu'elles bloquent ton code jusqu'à l'obtention d'une réponse.)

---

Ah oui pas bête, à tester...

Citation:

---

Envoyé par dukej

Une méthode qui fait de l'ajax est asynchrone, donc elle ne doit pas retourner de valeur, mais plutôt appeler une methode de callback qui recevra cette valeur en paramètre.

---

Oui c'est ce que je souhaitais faire à la base mais j'avoue ne pas avoir tout compris du callback :/ J'vais me taper un p'tit tutos et essayer de voir ça...

ce qu'on appelle "callback" n'est autre que la fonction qu'on exécute lorsqu'on reçoit la réponse du serveur.

dans ton cas :

Code:

---

1 2 3 4 5

success : function(valR){ if(valR!==0 && valR!==null){ maVar = valR; } }

---

EST la fonction de callback. c'est à partir de là que tu dois déclencher les mises à jour de ta page.

Dans mon cas impossible...

Quant à async : false, marche pas :/

il suffit de faire le return dans le success ...

Citation:

---

Envoyé par SpaceFrog

il suffit de faire le return dans le success ...

---

Code:

---

success: function(){...}

---

Non, ce n'est pas possible, si tu mets le return dans le callback success, c'est la fonction anonyme qui va renvoyer sa réponse, donc pas la fonction de départ (d'ailleurs, celle-ci à logiquement terminé son exécution).

@FotoXe33 : si pour toi attendre la réponse du serveur n'est pas une option possible, ni en synchrone ni en asynchrone, il te reste plus qu'à te tirer une balle dans le sexe ! :aie:
Plus sérieusement, il doit y avoir une solution, mais pour t'aider, il faudrait nous en dire plus sur ce que tu cherches à faire...

Citation:

---

Envoyé par FotoXe33

Dans mon cas impossible...

---

quoi donc ? la solution du callback ? tu souhaites donc que ta fonction soit blocante en attendant la réponse ? donc synchrone (async:false) ?

Citation:

---

Envoyé par FotoXe33

Quant à async : false, marche pas :/

---

Attention de ne pas mélanger les différentes solutions qu'on t'as proposé, pour l'ajout du paramètre async:false je t'ai bien précisé de reprendre ton code initial. (donc avec modification de variable et non retour dans la fonction succes) :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

var monObj = function(){ var maVar = null; // Attribut privé à changer   sendAjaxRequest = function(){ $.ajax({ url : ((new Error()).fileName.replace(/(^.*)(\/)(.*$)/gi, "$1$2"))+"monfichierserver.php", type : "POST", data : "bla=blabla", cache : false, async : false, success : function(valR){ if(valR!==0 && valR!==null){ maVar = valR; } } });   return (maVar!==null); // CE TEST VAUT FALSE !!! } };

---

Citation:

---

Envoyé par Bovino

@FotoXe33 : si pour toi attendre la réponse du serveur n'est pas une option possible, ni en synchrone ni en asynchrone, il te reste plus qu'à te tirer une balle dans le sexe ! :aie:
Plus sérieusement, il doit y avoir une solution, mais pour t'aider, il faudrait nous en dire plus sur ce que tu cherches à faire...

---

Oui je vais éviter l'option de la balle :p
Si c'est possible, je veux attendre la réponse, et

Code:

---

async:false;

---

ne marche pas :/

Citation:

---

Envoyé par Willpower

Attention de ne pas mélanger les différentes solutions qu'on t'as proposé, pour l'ajout du paramètre async:false je t'ai bien précisé de reprendre ton code initial.

---

Oui je n'allait pas mélanger tout. J'avais déjà le code que tu as posté mais sans succès :s

Citation:

---

Oui je vais éviter l'option de la balle :p

---

:mouarf:
Je sais pas pourquoi, mais je ne suis pas étonné ! :mrgreen:

Sinon, décris-nous un peu mieux ce que tu souhaites faire ;)

Bon le principe :

Une fonction qui génére un identifiants unique à partir de certains éléments.

FONCTION GENERER{
    SI laClef N'EST PAS GENERER
        laClef = ENVOIE_REQUETTE_AJAX()
        retour = GENERER();
    SINON
        test = ENVOIE_REQUETTE_AJAX(laClef) // Pour tester si la clef est ok
        SI test===VRAI
            retour = laCLef;
        SINON
            retour = FAUX;
        FIN SI
    FIN SI
}

Honni sois-tu, Willpower, d'avoir suggéré de l'Ajax synchrone ! Il y a toujours moyen de faire autrement.

@FotoXe33, je pense que tu as un problème avec les objets en JavaScript. Pour être précis, je pense que tu confonds la déclaration de la classe (ou plutôt du constructeur), et l'instanciation de l'objet.

Par exemple, ceci est un constructeur :

Code:

---

1 2 3 4

var Bidule = function( poids, couleur ) { this.poids = poids; this.couleur = couleur; }

---

Et ceci est une instanciation :

Code:

---

var monBidule = new Bidule('500kg', 'rose');

---

Actuellement, ta variable monObj reçoit la fonction anonyme que tu as écrite, autrement dit monObj est une fonction ! De plus, si tu exécutes cette fonction, la variable maVar ne sera visible de personne, et la variable sendAjaxRequest sera globale. Ce n'est sans doute pas ce que tu veux obtenir.

Si j'ai bien compris, tu veux lancer une requête Ajax au moment où tu construis ton objet. Ce que je vais dire va t'aider à comprendre les callbacks, je pense.

Une requête Ajax c'est un peu comme un expresso : tu appuies sur le bouton et, en attendant que le café soir servi, tu fais autre chose. Maintenant, suppose que tu as une cafetière hyper moderne capable de se déplacer et de t'apporter ton café. En plus de ça, elle est relativement intelligente et comprend les ordres que tu lui donnes.
Si tu appuies sur le bouton et que tu lui dis « apporte-moi mon café », elle va t'apporter ton café tout de suite alors qu'il n'est pas forcément prêt. C'est ce que fait ton script actuellement.
Par contre, si tu lui dis « apporte-moi mon café quand il sera prêt », elle va attendre que le café soit prêt avant de te l'apporter. C'est ce que font les fonctions de callback.

Code:

---

1 2 3 4 5 6 7 8

$.preparerCafe({ type: 'expresso', force: 'bien serré', sucre: 'pas trop', success: function( cafePret ) { apporterCafe(cafePret); } });

---

Comme je n'ai pas exactement compris ce que tu cherches à faire, je vais te laisser chercher une solution à la lumière de ce que je viens de t'apprendre.

Une dernière chose :

Code:

---

(new Error()).fileName

---

Pourquoi pas :

Code:

---

location.href

---

?

Citation:

---

Envoyé par FotoXe33

Bon le principe :

Une fonction qui génére un identifiants unique à partir de certains éléments.

FONCTION GENERER{
SI laClef N'EST PAS GENERER
laClef = ENVOIE_REQUETTE_AJAX()
retour = GENERER();
SINON
test = ENVOIE_REQUETTE_AJAX(laClef) // Pour tester si la clef est ok
SI test===VRAI
retour = laCLef;
SINON
retour = FAUX;
FIN SI
FIN SI
}

---

je ne suis pas sûr d'avoir tout compris, mais sous forme de callback, je te proposerai une solution comme ceci :

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 key; function suite_execution(){ ... } function suite_execution_negative(){ ... } function tester(){ $.ajax({ ...:__, ...:__, succes:function(obj){ if(obj===true) suite_execution(); else suite_execution_negative(); } }); } function generer(){ if(!key){ $.ajax({ ...:__, ...:__, succes:function(obj){ key = obj; tester(); } }); } else{ tester(); } } generer();

---

Citation:

---

Envoyé par Watilin

Honni sois-tu, Willpower, d'avoir suggéré de l'Ajax synchrone ! Il y a toujours moyen de faire autrement.

---

je sais, j'ai eu moi-même un peu honte. :oops:

mais bon, je lui offrais une solution simple où il n'avait qu'à rajouter un paramètre booléen.

Citation:

---

Envoyé par Watilin

@FotoXe33, je pense que tu as un problème avec les objets en JavaScript. Pour être précis, je pense que tu confonds la déclaration de la classe (ou plutôt du constructeur), et l'instanciation de l'objet.

Par exemple, ceci est un constructeur :

Code:

---

1 2 3 4

var Bidule = function( poids, couleur ) { this.poids = poids; this.couleur = couleur; }

---

Et ceci est une instanciation :

Code:

---

var monBidule = new Bidule('500kg', 'rose');

---

---

J'ai eut pendant 2 ans des cours objets, tkt je fais très bien la différence ;) C'est avec js que je suis un peut pommé...

Citation:

---

Envoyé par Watilin

Actuellement, ta variable monObj reçoit la fonction anonyme que tu as écrite, autrement dit monObj est une fonction ! De plus, si tu exécutes cette fonction, la variable maVar ne sera visible de personne, et la variable sendAjaxRequest sera globale. Ce n'est sans doute pas ce que tu veux obtenir.

---

Mon objet EST instancié ! Tout de même ! Et si, je VEUX que maVar ne soit accessible qu'à l'intérieur de l'objet. Idem pour sendAjaxRequest...

Citation:

---

Envoyé par Watilin

Si j'ai bien compris, tu veux lancer une requête Ajax au moment où tu construis ton objet.

---

Absolument pas :/ Je souhaite lancé ma requette quand j'appelle ma fonction sendAjaxRequest, lors de la génération de la clef...

Citation:

---

Envoyé par Watilin

Ce que je vais dire va t'aider à comprendre les callbacks, je pense.

Une requête Ajax c'est un peu comme un expresso : tu appuies sur le bouton et, en attendant que le café soir servi, tu fais autre chose. Maintenant, suppose que tu as une cafetière hyper moderne capable de se déplacer et de t'apporter ton café. En plus de ça, elle est relativement intelligente et comprend les ordres que tu lui donnes.
Si tu appuies sur le bouton et que tu lui dis « apporte-moi mon café », elle va t'apporter ton café tout de suite alors qu'il n'est pas forcément prêt. C'est ce que fait ton script actuellement.
Par contre, si tu lui dis « apporte-moi mon café quand il sera prêt », elle va attendre que le café soit prêt avant de te l'apporter. C'est ce que font les fonctions de callback.

Code:

---

1 2 3 4 5 6 7 8

$.preparerCafe({ type: 'expresso', force: 'bien serré', sucre: 'pas trop', success: function( cafePret ) { apporterCafe(cafePret); } });

---

Comme je n'ai pas exactement compris ce que tu cherches à faire, je vais te laisser chercher une solution à la lumière de ce que je viens de t'apprendre.

---

Merci pour cette explication par contre, car le callback je voyais plus ou moins ce que c'était mais sans plus. L'idée d’appeler une fonction dedans le callback est loin d'être con, j'y avait pas du tout pensé (mais bon une fonction qui sert juste à mettre une valeur dans un attribut d'objet ça fait chier, 'fin bref...)

Citation:

---

Envoyé par Watilin

Une dernière chose :

Code:

---

(new Error()).fileName

---

Pourquoi pas :

Code:

---

location.href

---

?

---

Euh bonne question xD C'est exactement la même valeur ? Et si cette page est appelé via ajax ça marche quand même avec location.href ?

Citation:

---

Envoyé par Willpower

je ne suis pas sûr d'avoir tout compris, mais sous forme de callback, je te proposerai une solution comme ceci :

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 key; function suite_execution(){ ... } function suite_execution_negative(){ ... } function tester(){ $.ajax({ ...:__, ...:__, succes:function(obj){ if(obj===true) suite_execution(); else suite_execution_negative(); } }); } function generer(){ if(!key){ $.ajax({ ...:__, ...:__, succes:function(obj){ key = obj; tester(); } }); } else{ tester(); } } generer();

---

---

Hélas, cela ne convient pas du tout à ma situation :/ Je sais que sans mon code exacte c'est un peut dure de tout comprendre mais je ne peut pas trop en dire plus pour le moment (expérimental :p)


Après plusieurs test (dont la solution de Watilin) et n'ayant pas de résultats plus concluant je vais tenter de mieux me faire comprendre.

J'ai une classe maClass ayant pour contenu :

• Un attribut maClef
• Un attribut generer (booléen) pour tester si la clef a déjà été générée
• Un attribut time car j'ai mit une expiration de la clef
• Une methode testTime(nbSecAuMax) pour tester si la clef a été généré il y a moins nbSecAuMax
• Une méthode sendAjaxRequest pour envoyer les requettes
• Une méthode générer_clef() qui génére, test, puis retourne la clef
• Une méthode definir_clef() qui prends un paramètre ou non. Si un paramètre est présent (la clef), on définit la valeur de l'attribut maClef par le paramètre reçut. Dans tous les cas (paramètre ou non), on a return (maClef!==null);

Je précise que SEUL générer_clef() doit être PUBLIC.

On instancie (var monObj = new maClass();) puis on génére comme cela : var laClefGénérée = monObj.générer_clef();Il faut donc que générer_clef() ait l'algo suivant :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

var valeurRetournée = false;   if(maClef === null && "test du temps"){ time = "le temps actuelle"; sendAjaxRequest(); if(definir_clef()===true){ valeurRetournée = générer_clef(); // A savoir cette fonction } } else{ sendAjaxRequest(); valeurRetournée = (definir_clef()===true) ? vid : false; }   return valeurRetournée;

---

Pour se la jouer à la AOE2, "de l'aide, merci" :p

lu un peu en diagonal mais une deux réflexions me viennent

1ére réflexion :
pourquoi ne pas traiter coté serveur la validité de la clef?
si on envoi null alors après test on en génère une et on la retourne, dans le cas contraire j'informe que c'est OK.

2éme réflexion :
si tu es en asynchrone tu vas avoir intérêt à donner une adresse de retour à ta fonction, car retourner une valeur c'est bien dans le vide c'est moyen. Cela peut être l'appel à une méthode de ton objet qui va elle atteindre ta variable.

Citation:

---

Envoyé par NoSmoking

1ére réflexion :
pourquoi ne pas traiter coté serveur la validité de la clef?
si on envoi null alors après test on en génère une et on la retourne, dans le cas contraire j'informe que c'est OK.

---

C'est le cas ! Lors de la première requête, le programme génère la clef et la retourne si tout c'est bien passé. Ensuite une seconde requête ac la clef est envoyé côté serveur qui test si c'est la clef qu'il vient de générer ou pas.

Citation:

---

Envoyé par NoSmoking

2éme réflexion :
si tu es en asynchrone tu vas avoir intérêt à donner une adresse de retour à ta fonction, car retourner une valeur c'est bien dans le vide c'est moyen. Cela peut être l'appel à une méthode de ton objet qui va elle atteindre ta variable.

---

Peux-tu m'expliquer ce que tu entends par adresse de retour ?

Citation:

---

Peux-tu m'expliquer ce que tu entends par adresse de retour ?

---

en rapport avec ce qu'a écrit Watilin, apporterCafe(cafePret);.

ou tu fais tes traitements dans la fonction anonyme, ou il te faut bien retourner le résultat quelque part, car en tout état de cause ta fonction meurt après exécution (ça c'est une évidence :D), donc lorsque cette fonction, en asynchrone, se termine qui sait où va atterrir un éventuel return( laclef).
La mise en place de l'appel à une fonction régle ce problème.

Bah sans savoir que ça s'appelle comme ça je l'avais fait suite au conseil de Watilin, mais sans succès toujours :/

Je crois que je n'ai pas d'autre solution, voilà le code complet, dites-moi si quelqu'un trouve quelque chose...

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

var maClass = function(){ var vid = null; var time = null; var generate = true;   timeOutTest = function(nbSecMax){ return (time!==null && ((Math.round(+new Date()/1000))-time<nbSecMax)); }   setVid = function(){ if(arguments[0]){ generate = !generate; vid = (generate===false) ? arguments[0] : null; }   return (vid!==null); }   sendAjaxRequest = function(){ $.ajax({ url : ((new Error()).fileName.replace(/(^.*)(\/)(.*$)/gi, "$1$2"))+"traitement.php", type : "POST", data : "time="+time+"&vid="+vid+"&generate="+generate, cache : false, success : function(valR){ if(valR!==0 && valR!==null){ setVid(valR); } } });   return true; }   this.generate_vid = function(){ var valReturned = false;   if(vid===null && (time===null || false===timeOutTest(30))){ time = Math.round(+new Date()/1000); sendAjaxRequest(); if(setVid()===true){ valReturned = this.generate_vid(); } } else{ sendAjaxRequest(); valReturned = (setVid()===true) ? vid : false; }   return valReturned; } };

---

c'est ta façon de concevoir les appels qui est erronée :

ce que tu veux faire :

Code:

---

1 2 3 4

appeler une fonction, que la fonction fasse une requête au serveur, attendre la réponse, retourner la réponse reçue

---

en général, "attendre" en javascript comme dans beaucoup de langages, c'est mal car ça bloque l'exécution. même si c'est permis et même conçu pour ("arguments synchrone") je te le déconseille vivement.


la bonne façon de faire est :

Code:

---

1 2 3

appeler une fonction, que la fonction fasse une requête au serveur, s'arrêter là pour le moment.

---

Code:

---

1 2 3

utiliser une autre fonction(callback/success) qui se déclenche lorsque la réponse arrive appeler l'une ou l'autre méthode en fonction de la réponse, du status de la page etc.. utiliser la réponse et eventuellement mettre la page à jour.

---

oublie donc les valeurs de retour avec ce système, tu peux par contre créer des timers (setTimeout) pour detecter des délais trop long par exemple.

en gros tu as qqe chose du genre :

Code:

---

1 2 3

// mauvaise utilisation avec attente d'un retour d'un appel ajax var maVideo = monObjet.generate_vid(); document.body.appendChild(maVideo); // exemple d'utilisation du retour de ta fonction generate_vid();

---

tu dois le changer en :

Code:

---

1 2	// bonne utilisation d'un appel ajax sans attente monObjet.generate_vid();

---

et par exemple utiliser ta méthode setVid pour mettre la vidéo dans la page :

Code:

---

1 2 3 4 5 6 7 8 9

//setVid est déclenché à partir du success de l'appel ajax setVid = function(obj){ if(obj && generate){ vid = obj; // on utilise donc la valeur de retour ici document.body.appendChild( vid ); } return vid; }

---

Citation:

---

Envoyé par Willpower

c'est ta façon de concevoir les appels qui est erronée :

ce que tu veux faire :

Code:

---

1 2 3 4

appeler une fonction, que la fonction fasse une requête au serveur, attendre la réponse, retourner la réponse reçue

---

en général, "attendre" en javascript comme dans beaucoup de langages, c'est mal car ça bloque l'exécution. même si c'est permis et même conçu pour ("arguments synchrone") je te le déconseille vivement.


la bonne façon de faire est :

Code:

---

1 2 3

appeler une fonction, que la fonction fasse une requête au serveur, s'arrêter là pour le moment.

---

Code:

---

1 2 3

utiliser une autre fonction(callback/success) qui se déclenche lorsque la réponse arrive appeler l'une ou l'autre méthode en fonction de la réponse, du status de la page etc.. utiliser la réponse et eventuellement mettre la page à jour.

---

oublie donc les valeurs de retour avec ce système, tu peux par contre créer des timers (setTimeout) pour detecter des délais trop long par exemple.

---

Merci pour ce petit rappelle :)

Citation:

---

Envoyé par Willpower

en gros tu as qqe chose du genre :

Code:

---

1 2 3

// mauvaise utilisation avec attente d'un retour d'un appel ajax var maVideo = monObjet.generate_vid(); document.body.appendChild(maVideo); // exemple d'utilisation du retour de ta fonction generate_vid();

---

tu dois le changer en :

Code:

---

1 2	// bonne utilisation d'un appel ajax sans attente monObjet.generate_vid();

---

et par exemple utiliser ta méthode setVid pour mettre la vidéo dans la page :

Code:

---

1 2 3 4 5 6 7 8 9

//setVid est déclenché à partir du success de l'appel ajax setVid = function(obj){ if(obj && generate){ vid = obj; // on utilise donc la valeur de retour ici document.body.appendChild( vid ); } return vid; }

---

---

Quand tu dis document.body.appendChild( vid );, comment je fais pour récupérer ma valeur "vid" dans mon script après ? Car je dois l'utiliser pour une autre requête d'un autre script. En faite je fais dans mon autre ajax :

Code:

---

1 2 3 4 5 6

$.ajax({ ... data : '...&vid='+monObj.generer_vid(), ... });

---

Donc tu vois, avec ma méthode c'était plus pratique puisque cette requête ajax NE DOIT PAS partir si le vid n'a pas été générer... L'asynchrone me correspondais mieux mais hélas il ne marche pas :/

Citation:

---

Envoyé par FotoXe33

Quand tu dis document.body.appendChild( vid );, comment je fais pour récupérer ma valeur "vid" dans mon script après ? Car je dois l'utiliser pour une autre requête d'un autre script. En faite je fais dans mon autre ajax :

Code:

---

1 2 3 4 5 6

$.ajax({ ... data : '...&vid='+monObj.generer_vid(), ... });

---

Donc tu vois, avec ma méthode c'était plus pratique puisque cette requête ajax NE DOIT PAS partir si le vid n'a pas été générer... L'asynchrone me correspondais mieux mais hélas il ne marche pas :/

---

C'était un exemple d'utilisation, insérer ton résultat dans la page.
Mais si tu veux faire une seconde requête sur ce résultat, tu n'as qu'à lancer cette seconde requête dans la fonction (success) recevant le premier résultat.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

// premier appel ajax $.ajax({ url : ((new Error()).fileName.replace(/(^.*)(\/)(.*$)/gi, "$1$2"))+"traitement.php", type : "POST", data : "time="+time+"&vid="+vid+"&generate="+generate, cache : false, success : function(valR){ // second appel ajax à partir du résultat du premier. $.ajax({ ... data : '...&vid='+monObj.generer_vid(), ... }); } });

---

ps:
synchrone = quand tu attends ta réponse avant de poursuivre.
asynchrone = quand tu poursuis l'exécution de ton code et qu'une autre fonction se déclenche lors de la réception de la réponse.

avec jquery, c'est "asynchrone" par défaut et si tu veux "synhcrone", tu dois mettre "async:false".

À mon avis, c'est l'algorithme lui-même qui est à revoir :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

FONCTION GENERER{ SI laClef N'EST PAS GENERER laClef = ENVOIE_REQUETTE_AJAX() (1) retour = GENERER(); (2) SINON test = ENVOIE_REQUETTE_AJAX(laClef) (1) SI test===VRAI (2) retour = laCLef; SINON retour = FAUX; FIN SI FIN SI } (3)

---

Sur les points marqués (1), tu ne peux pas utiliser la valeur de retour de ces fonctions car elles retournent immédiatement après l'envoi de leurs requêtes Ajax, et n'ont à ce moment-là pas encore de valeur à renvoyer.

Sur les points marqués (2), tout le code en orange sera exécuté immédiatement après l'envoi des requêtes Ajax, sans attendre l'arrivée du résultat. Ce code est à placer dans les fonctions de rappel.

Le point (3) représente la fin de la fonction generer. Comme tu n'as pas encore le résultat des requêtes, tu n'as pas de valeur à renvoyer ici non plus. Autrement dit, la fonction generer ne peut rien renvoyer.

Ce qu'il te faut, c'est une nouvelle propriété sur ton objet (publique ou non, à toi de voir), qui recevra le résultat quand il sera prêt. Appellons-la resultatPret. Tu auras également une fonction, mettons suiteTraitement, qui sera appelée seulement quand tu seras sûr que le résultat est prêt.

Voici le nouvel algorithme :

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

FONCTION GENERER{ SI laClef N'EST PAS GENERER ENVOIE_REQUETTE_AJAX({ ... success: function( result ) { laClef = result; GENERER(); } }) SINON ENVOIE_REQUETTE_AJAX({ data: laClef, ... success: function( result ) { test = result; SI test===VRAI resultatPret = laCLef; SINON resultatPret = FAUX; FIN SI suiteTraitement(); } }) FIN SI }

---

Le reste est entre tes mains ;)

Citation:

---

Envoyé par Willpower

C'était un exemple d'utilisation, insérer ton résultat dans la page.
Mais si tu veux faire une seconde requête sur ce résultat, tu n'as qu'à lancer cette seconde requête dans la fonction (success) recevant le premier résultat.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

// premier appel ajax $.ajax({ url : ((new Error()).fileName.replace(/(^.*)(\/)(.*$)/gi, "$1$2"))+"traitement.php", type : "POST", data : "time="+time+"&vid="+vid+"&generate="+generate, cache : false, success : function(valR){ // second appel ajax à partir du résultat du premier. $.ajax({ ... data : '...&vid='+monObj.generer_vid(), ... }); } });

---

---

Non car les requêtes sont externes à l'objet, on ne sait pas à ce moment ce qu'elle sera et comment elle sera construite :/

Citation:

---

Envoyé par Willpower

ps:
synchrone = quand tu attends ta réponse avant de poursuivre.
asynchrone = quand tu poursuis l'exécution de ton code et qu'une autre fonction se déclenche lors de la réception de la réponse.

avec jquery, c'est "asynchrone" par défaut et si tu veux "synhcrone", tu dois mettre "async:false".

---

Oui autant pour moi, mes mains ont dérrapés :p

Citation:

---

Envoyé par Watilin;6389519

Ce qu'il te faut, c'est une nouvelle propriété sur ton objet (publique ou non, à toi de voir), qui recevra le résultat quand il sera prêt. Appellons-la [codeinline

resultatPret[/codeinline]. Tu auras également une fonction, mettons suiteTraitement, qui sera appelée seulement quand tu seras sûr que le résultat est prêt.

Voici le nouvel algorithme :

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

FONCTION GENERER{ SI laClef N'EST PAS GENERER ENVOIE_REQUETTE_AJAX({ ... success: function( result ) { laClef = result; GENERER(); } }) SINON ENVOIE_REQUETTE_AJAX({ data: laClef, ... success: function( result ) { test = result; SI test===VRAI resultatPret = laCLef; SINON resultatPret = FAUX; FIN SI suiteTraitement(); } }) FIN SI }

---

Le reste est entre tes mains ;)

---

C'est justement ce que j’essayai d'éviter... Ré-écrire deux fois le même code à 99%... Mais comment faire ?

Car après vos nombreuses remarques à tous, je comprends clairement le problème qui se pose : le script n'attends pas ajax et donc les valeur ne sont pas renseignées au bon moment et donc les return sont foirés.

Bref je suis toujours autant dans le flou car je trouve ça dommage qu'en programmation "objet" (désolé j'en suis à mes début en js) on soit obligé d'aller à l'inverse même des principes fondateurs en matière de ré-utilisabilité...

Citation:

---

Envoyé par FotoXe33

Bref je suis toujours autant dans le flou car je trouve ça dommage qu'en programmation "objet" (désolé j'en suis à mes début en js) on soit obligé d'aller à l'inverse même des principes fondateurs en matière de ré-utilisabilité...

---

non non, le code peut bien être réutilisé, faut juste trouver le code qui te convient mais on a du mal à te proposer le bon car on ne comprend pas la suite logique de ce que tu veux faire avec les réponses ajax.

si on avait tous les éléments on te proposerai directement une réponse complète, mais je crois comprendre que tu désires garder une partie de ton code "secret" si je ne me trompe ?

Citation:

---

Envoyé par FotoXe33

Non car les requêtes sont externes à l'objet, on ne sait pas à ce moment ce qu'elle sera et comment elle sera construite :/

---

à quel moment sait-on quelles requêtes ou quelles méthodes appliquer à l'objet ? désolé mais pour te proposer la solution complète on a besoin de savoir ce que tu désires faire de tes objets reçus.

en fait, il faut voir ton code comme du code évènementiel et non séquentiel, mais je conçois que c'est déroutant quand on n'a pas l'habitude.

C'est même pas le "code" :p Le php marche parfait, les réponses se font comme je le souhaite.

En faite voilà le "secret" :p

Je fait une classe made in moi pour protéger les requêtes ajax. Il me faut donc un test php car c'est plus compliqué de faire quelque chose de non hackable avec js :p

L'idée :

La personne fait une requête ajax en mettant comme un de ses paramètres une clef unique généré en php et expirant toutes les 30 secondes. Du côté serveur il y a une simple fonction qui permet de vérifier que la clef soit correcte et qu'elle n'est pas expirée. Donc dans l'idée, une personne qui veut protéger sa requête côté client fait :

Code:

---

1 2 3 4 5 6 7 8

var monObj = new maClasse();   $.ajax{ ... data : "toto=tata&vid="+monObj.generer_vid(), // Il faut qu'à ce moment un vid soit générer, tester par php une fois, puis retrouné ... });

---

Mais ça je l'ai déjà expliqué donc je ne vois pas comment expliquer plus :/

Citation:

---

Envoyé par FotoXe33

C'est même pas le "code" :p Le php marche parfait, les réponses se font comme je le souhaite.

En faite voilà le "secret" :p

Je fait une classe made in moi pour protéger les requêtes ajax. Il me faut donc un test php car c'est plus compliqué de faire quelque chose de non hackable avec js :p

---

Impossible même de faire du nom hackable coté client, il faut toujours revérifier coté serveur.

Citation:

---

Envoyé par FotoXe33

L'idée :

La personne fait une requête ajax en mettant comme un de ses paramètres une clef unique généré en php et expirant toutes les 30 secondes. Du côté serveur il y a une simple fonction qui permet de vérifier que la clef soit correcte et qu'elle n'est pas expirée. Donc dans l'idée, une personne qui veut protéger sa requête côté client fait :

Code:

---

1 2 3 4 5 6 7 8

var monObj = new maClasse();   $.ajax{ ... data : "toto=tata&vid="+monObj.generer_vid(), // Il faut qu'à ce moment un vid soit générer, tester par php une fois, puis retrouné ... });

---

Mais ça je l'ai déjà expliqué donc je ne vois pas comment expliquer plus :/

---

on avance même si je ne vois toujours pas l'aboutissant, je peux tenter de te proposer :

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

var key = null; var time = 0; function generateKey(boolVerify){ $.ajax({ success:function(response){ key = response; time = (new Date()).getTime(); if(boolVerify) verifyKey(); } }); } function action(key){ ... } function error(key){ ... } function verifyKey(){ if((new Date()).getTime() - 30000 > time) generateKey(true); // true pour forcer la vérification après else{ $.ajax({ data:"key="+key success:function(response){ if(response===true) action(key); else error(key); } }); } }   // demande une nouvelle "key" qui sera stocké dans la variable key sans callback une fois la réponse reçue. generateKey();     // exécute "verifyKey" après 5 secondes, si la clé datait de plus de 30 secondes, alors en génére une nouvelle et demande une vérification une fois la clée obtenue. sinon si le temps de 30 secondes n'était pas écoulé, alors fait une requete de vérification au serveur et s'il renvoit "true" déclenche la fonction "action" sinon la fonction "error". setTimeout(verifyKey,5000);

---

je suis toujours dans la lecture en diagonale :oops:

mais on peut ré-exécuter une fonction à partir d'elle même en utilisation la propriété callee de arguments.

exemple :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

function fctTest( param){ // get la fonction appelante var refFonction = arguments.callee; // get argument passe var valParam = arguments[0]; // traitement suivant le cas if( valParam){ alert( 'Le parametre est : ' +valParam); } else{ // on la rapelle avec un parametre refFonction('Bonjour!'); } } // appel sans parametre fctTest();

---