Discussion :

[welcsk]

Bonjour ,
Je cherche à faire en sorte que mon script ajax se lance maximum X fois (ici 3) , chaque instance doit traiter une ligne du textarea , j'ai donc fait ceci :

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
25
26
27
28
29
30
<textarea></textarea>
<script type="text/javascript">
function start(){
	var lines = $('textarea').val().split('\n');
	var i = 0;
	var thread_count = 0;
	var div = document.getElementById('logs');
	while(i < lines.length) {
		if(thread_count < 3){
			thread_count ++;
			$.ajax({
				url: "monscript.php?line="+lines[i],
			type: 'GET',
			dataType: 'text',
			success : function(msg){
						div.innerHTML = div.innerHTML + msg + "<br>";	
						thread_count--;
					},
			error : function(){
						alert("Une erreur Ajax.");
					}
			});
			i++;
		}
	}
};
</script>
<input type="button" value="run" onclick="start()"/>
<div id="logs">
</div>

Mais , soucis , le navigateur à vraiment du mal à supporter la boucle while (vu que je suppose que c'est le problème) , quelqu'un aurait une solution pour garder cette solution d'instance simultanées en évitant de faire crash monsieur le navigateur ?
Merci.

[Watilin]

Il y a une faille dans ton algorithme. Tu as un test imbriqué dans une boucle, selon ce schéma :

Code :

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

1
2
3
4
5
while (A) {
  if (B) {
    …
  }
}

Quand B est faux, c’est équivalent à :

Code :

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

while (A) { }

Ton instruction i++ se trouve dans le corps du test B. Ton crash est dû tout bêtement à une boucle infinie.

Si ton objectif est de limiter le nombre de requêtes concurrentes à 3, il ne faut pas tester ce nombre dans le while. Le while ne fait que lancer les 3 premières requêtes, le reste doit être géré par les fonctions de callback.
Avec jQuery tu as l’évènement complete qui te donne la possibilité d’agir à la fin d’une requête, que celle-ci ait réussi ou échoué.

[Doksuri]

Avec jQuery tu as l’évènement complete qui te donne la possibilité d’agir à la fin d’une requête, que celle-ci ait réussi ou échoué.

petite precision :

jqXHR.always(function( data|jqXHR, textStatus, jqXHR|errorThrown ) { }); (added in jQuery 1.6)
An alternative construct to the complete callback option, the .always() method replaces the deprecated .complete() method.

https://api.jquery.com/jQuery.ajax/

[welcsk]

Si ton objectif est de limiter le nombre de requêtes concurrentes à 3, il ne faut pas tester ce nombre dans le while. Le while ne fait que lancer les 3 premières requêtes, le reste doit être géré par les fonctions de callback.
Avec jQuery tu as l’évènement complete qui te donne la possibilité d’agir à la fin d’une requête, que celle-ci ait réussi ou échoué.

C"est exactement ce que je souhaite faire , cependant , même avec cette indication je ne vois pas comment faire...

[Watilin]

J’allais te proposer une solution basée sur les méthodes queue et dequeue de jQuery, mais je n’ai pas l’impression qu’elles permettent de lancer un certain nombre de requêtes simultanées et de mettre les autres en attente.


Dans tous les cas, tu dois faire une file d’attente de requêtes. Je te donne l’idée générale et je te laisse gérer les détails, ok ?

D’abord, la file, c’est un simple tableau. JS n’a pas d’objet spécial comme dans certains langages pour représenter précisément une file, mais les tableaux jouent ce rôle si tu utilises conjointement leurs méthodes push et shift (ou unshift et pop).

Pour mettre une requête en attente, n’appelle pas $.ajax directement. Prends l’objet { … } qui contient les paramètres de la requête, c’est cet objet que tu mets dans la file d’attente. Il te suffira de passer cet objet à $.ajax au moment voulu.

Ta boucle while parcourt les lignes de ton textarea. Pour éviter de t’embrouiller encore avec le i, je te conseille d’opter pour une boucle for, voire même pour la méthode forEach. Certains diront que c’est moins performant, mais ça te permet d’éviter le piège du i qui a sa valeur finale partout.

Ta boucle va faire deux choses selon le nombre de requêtes actuellement en cours. Soit ce nombre n’est pas au maximum et elle lance une nouvelle requête immédiatement ; soit elle met la requête dans la file d’attente.

Dans la fonction always (et non pas complete, merci Doksuri pour la précision), tu vas regarder s’il reste des requêtes en attente, et le cas échéant en retirer une pour la lancer. C’est également dans cette fonction always que tu détecteras que toutes les requêtes sont terminées, et appelleras une fonction de finalisation si besoin (par exemple, remettre les résultats dans l’ordre, car tu n’as aucune garantie qu’ils vont arriver dans l’ordre).

Évidemment ce n’est plus success qui gère la file de requêtes, donc tu peux retirer l’instruction thread_count--;. Par contre tu auras peut-être besoin de gérer thread_count dans always.


Je vais te laisser chercher un peu, mais n’hésite pas à revenir poser des questions si tu bloques