Boucle sur une fonction

Bonjour à tous,
d'habitude je trouve mes réponses en cherchant, mais la... rien
j'ai ce bout de script qui fonctionne (c'est sous greasemonkey, mais c'est à prioris assez proche de la fonction xmlhttprequest de base):

Code:

---

1 2 3 4 5 6 7 8

GM_xmlhttpRequest({ method: "GET", url: urlnd[0], onload: function recup(response) { alert(nd[0]); alert(response.responseText); } });

---

1ere question, peut on boucler cette fonction? (les tables urlnd et nd sont définies et fonctionne parfaitement) lorsque je fais une boucle for, ca ne m'affiche que la dernière, cause que j'ai compris puisque la fonction ce fait sur le onload

2eme question, a quoi correspond l'argument response? puisque en mettant n'importe quoi (d'identique évidemment) sur les 2 lignes ca fonctionne toujours?

3eme question, pourquoi lorsque je sors la fonction de cette facon:

Code:

---

1 2 3 4 5 6 7 8 9 10

function recup(response) { alert(nd[0]); alert(response.responseText); }   GM_xmlhttpRequest({ method: "GET", url: urlnd[0], onload: recup(response), });

---

ca ne fonctionne pas, je n'ai meme pas le alert(nd[0] alors que je l'ai en supprimant l'argument:

Code:

---

1 2 3 4 5 6 7 8 9 10

function recup() { alert(nd[0]); alert(response.responseText); }   GM_xmlhttpRequest({ method: "GET", url: urlnd[0], onload: recup(), });

---

il me semble que j'avais déja eu ce problème sur un script complètement différent, sur un onclick me semble t il, sans argument, aucun soucis, dès que je passais un argument ca bloquait.

Merci d'avance pour vos réponses, remarques, et suggestions eventuelles

Citation:

---

Windows XP / Firefox 11.0

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11

for(var i = 0; i < 10; i++){ GM_xmlhttpRequest({ method: "GET", url: urlnd[i], onload: function recup(response) { alert(nd[i]); alert(response.responseText); } }); }

---

Tu fais une boucle sur i qui ira de 0 a 10

Pour chaque valeur de i, il fera un appel a GM_xmlhttpRequest.

Les 10 appels se font en une fraction de seconde, maintenant ton i vaut 10.

Un des 10 appels de GM_xmlhttpRequest recoit sa première réponse (une requête ajax donc, une page correspondant à l'url fourni).

Tu as donc la fonction de callback/succes/onload qui se déclenche comme tu l'as définie :

Code:

---

1 2 3 4

function recup(response) { alert(nd[i]); alert(response.responseText); }

---

response devrait valoir la bonne page appelée comme voulu par contre ton i vaut 10, rappelle-toi.

Donc un alert(nd[i]); équivaudra à alert(nd[10]); le dernière élément pour toutes les réponses que tu recevras du serveur.

---------------------

Maintenant que tu as (je l'espère) compris le problème, je vais te proposer une des solutions le plus courante : l'encapsulation

Code:

---

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

for(var i = 0; i < 10; i++){ (function(j){ // on déclare une fonction anonyme et on l'exécute directement GM_xmlhttpRequest({ method: "GET", url: urlnd[j], // ici peu importe si on utilise "i" ou "j", cette fonction (GM_xmlhttpRequest) sera entièrement executée avant de passer au "i" suivant. onload: function recup(response) { alert(nd[j]); // le "j" étant déclaré individuellement à l'intérieur de chacune des 10 fonctions anonymes, sa valeur ne change pas avec la boucle for, chaque "j" garde sa valeur même longtemp après la fin de la boucle lorsque le onload recevra la réponse du serveur. alert(response.responseText); } }); })(i); // on passe la valeur de i à chaque déclaration/exécution de notre nouvelle fonction }

---

edit: ou mieux n'encapsuler que la partie qui sera exécutée à retardement :

Code:

---

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

function bind_recup(j){ return function(response){ alert(nd[j]); alert(response.responseText); }; }   for(var i = 0; i < 10; i++){ GM_xmlhttpRequest({ method: "GET", url: urlnd[i], onload: bind_recup(i) }); }

---

Merci pour ta réponse, effectivement j'avais cerné la cause de l'affichage de la dernière valeur uniquement, je n'ai pas étudiée ta solution mais elle me parait très bien expliquée, je verrais ca demain.

Merci beaucoup

Alors premiers essais, très concluants, merci :)
Je ne comprends pas tout sachant que ceci fonctionne également:

Code:

---

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

function bind_recup(k){ return function(response){ alert(nd[k]); alert(response.responseText); }; }   for(var k = 0; k < 3; k++){ GM_xmlhttpRequest({ method: "GET", url: urlnd[k], onload: bind_recup(k) }); }

---

en fait c'est le fait de mettre une fonction dans une fonction?

Re merci en tout cas

Code:

---

onload: bind_recup(k)

---

Non !
Tu ne peux pas faire ça.
Voir : http://www.developpez.net/forums/d11...-cachees-eval/

Citation:

---

Envoyé par toinounet21

Merci beaucoup

Alors premiers essais, très concluants, merci :)
Je ne comprends pas tout sachant que ceci fonctionne également:

Code:

---

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

function bind_recup(k){ return function(response){ alert(nd[k]); alert(response.responseText); }; }   for(var k = 0; k < 3; k++){ GM_xmlhttpRequest({ method: "GET", url: urlnd[k], onload: bind_recup(k) }); }

---

en fait c'est le fait de mettre une fonction dans une fonction?

Re merci en tout cas

---

Un fait tu exécute une fonction (bind_recup) directement PENDANT la boucle qui elle renvoit une autre fonction qui sera assignée (liée) au "onload".

Plus tard quand le "onload" (donc ton autre fonction) se déclenchera et que ton i (ou autre indice de ta boucle) n'aura plus la bonne valeur, ta fonction créée dans une autre aura toujours la bonne valeur (variable local à la capsule qui elle s'était instancié directement pendant la boucle).


edit: en fait, dans mon exemple j'avais donné un nom différent à "i" et à "j" pour bien marquer le fait qu'on créé une nouvelle variable indépendante de celle qui nous donne sa valeur. (en fait une nouvelle variable (et fonction) est déclarée pour chaque itération de ta boucle "for", ça permet de conserver le contexte au moment du passage dans la boucle.)

@Bovino: le code est tout à fait correct, si la fonction exécutée renvoit une autre fonction.

Citation:

---

Envoyé par Willpower

@Bovino: le code est tout à fait correct, si la fonction exécutée renvoit une autre fonction.

---

Tout à fait, ma remarque était essentiellement liée au code du premier message qui laisse supposer que toinounet21 ne différencie pas correctement la notion de fonction de rappel et celle d'appel de fonction ;)

Citation:

---

Envoyé par Bovino

Tout à fait, ma remarque était essentiellement liée au code du premier message qui laisse supposer que toinounet21 ne différencie pas correctement la notion de fonction de rappel et celle d'appel de fonction ;)

---

Ah juste, j'avais même pas remarqué cela dans son post initial :ccool:

donc @toinounet21

dans ton code initial tu faisais

Code:

---

onload: recup()

---

ce qui n'avais aucun sens puisque tu passais au "onload" ... RIEN (car ta fonction "recup" n'a pas de return) alors que tu aurais du passer la fonction elle même (sans l'exécutée) :

Code:

---

onload: recup

---

Note que dans mon exemple A MOI, tu dois bien laisser les parenthèses pour que la première fonction s'exécute et en renvoit une autre.

J'espère qu'on t'aura aidé à mieux comprendre le principe, si t'as d'autres questions, n'hésite pas. ;)

Code:

---

1 2 3

(function(j){ ... })(i);

---

Très belle closure Willpower, j'allais justement proposer une solution de ce genre, tu m'as coupé l'herbe sous le pied :P

Sinon il s'agit d'un problème connu, excepté qu'ici on n'a pas de <div> auxquelles attacher notre indice de boucle. En fait c'est un problème qui survient dès lors qu'on a une fonction de rappel dans une boucle for.