Discussion :

[Emcy]

bonjour,

Pour faire tourner en boucle une appli Ajax asynchrone, j'utilise setTimeout que je lance dès que je reçois la réponse de ma requete

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
	if(xhr.readyState == 4 && ajax.Req.status == 200) {
		...
                ...
 
		xhr.onreadystatechange = null
		xhr.abort()
	        xhr = null
 
		setTimeout(function(){makeReq()},delayReq*1000)
	}

=> j'utilise cette méthode pour avoir l'état d'un appareil

=> je voudrais aussi que l'utilisateur puisse lancer manuellement des requètes pour donner des ordres à l'appareil. Donc j'ai créé un bouton qui lance cette fonction :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
function sendAction(){
	if (xhr){
                xhr.onreadystatechange = null
		xhr.abort()
	        xhr = null
        }
 
	makeReq(postvalue)
}

makeReq(postvalue) créer une requète xhr => si postvalue est vide, je crée une requete xhr avec la methode GET sinon je crée une requète xhr avec la methode POST + la valeur de postvalue.

=> le problème avec cette methode, c'est que ça crée des doublon d'appel de setTimeout ce qui réduit le temps de rafraichissement des infos (ce que je ne veux pas) => il y a des setTimeout qui tournent en même temps.
Comment faire pour résoudre mon problème ?

est-il possible de savoir si un setTimeout sur la fonction makeReq est actif ?

[E.Bzz]

Bonjour,
tu peux identifier tes setTimeout() par un handler (comme les fenêtres, par ex.).

Code :

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

1
2
if (!timer_observ)
timer_observ = setTimeout(function(){makeReq()},delayReq*1000);

Tu pourras ainsi identifier tes setTimeout() d'observation et ceux de commandes ...
Tu peux également passer un paramètre à ta fonction makeReq() : pour l'observation tu utilises setTimeout() et pour les commandes (à ne lancer qu'une fois, a priori), tu fais ton appel sans setTimeout() => plus de conflit.

Sinon, pour l'observation, un setInterval() serait plus approprié (que setTimeout() récursif)

A+

[Emcy]

j'ai fais ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
	if(xhr.readyState == 4 && ajax.Req.status == 200) {
		...
                ...
 
		xhr.onreadystatechange = null
		xhr.abort()
	        xhr = null
 
		if (ObjSetTimeout==0){
				ObjSetTimeout = setTimeout(function(){makeReq()},delayReq*1000)
			}
	}

au debut de ma fonction makeReq, je mets :

Code :

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

ObjSetTimeout  = 0

ObjSetTimeout est défini en global

je ne différencie pas les requêtes manuelles et automatiques => tout ce que je veux c'est que s'il y a setTimeout qui va s'enclencher, qu'un autre ne s'active pas.
=> mais ça ne marche pas

[E.Bzz]

Code :

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

if (!ObjSetTimeout)

[Emcy]

ça ne change rien ....

[Emcy]

c'est bon j'ai trouvé

donc dans ma reception de requète ajax, je fais

Code :

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

1
2
3
4
5
6
7
8
9
10
11
	if(xhr.readyState == 4 && ajax.Req.status == 200) {
		...
                ...
 
		xhr.onreadystatechange = null
		xhr.abort()
	        xhr = null
 
		ObjSetTimeout = setTimeout(function(){makeReq()},delayReq*1000)
 
	}

et au debut de ma fonction makeReq(), je mets :

Code :

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

1
2
3
	if (ObjSetTimeout){
		clearTimeout(ObjSetTimeout)
	}

ObjSetTimeout doit etre declaré en global

[E.Bzz]

A+