Discussion :

[Anonyme1784]

Hello, petit problème avec mon code qui convulse quand je clique 2 fois :p

Code javascript :

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

1
2
3
4
5
6
7
8
9
10
11
12
var timer;
function rafraichirmsg(desti){
    clearTimeout(timer);
        $.ajax({
    type: 'POST',
    url: 'chatmsg.php',
    data: {desti: desti},
    success: function(b) {
    timer = setTimeout(rafraichirmsg(desti), 5000);
    $('#messages').html(b);}
});
}

Alors, c'est très simple, le rafraichirmsg(desti) est déclenché au onClick, je clique sur un desti(nataire) (1 fois) ça fonctionne, mais en cliquant sur un autre desti (une 2ème fois) ça blink entre les messages de desti1 et desti2. Donc la fonction ne clear pas le settimeout parce que c'est comme s'ils s'additionnaient...

Help please

[www.diazis.com]

il faut enlever le var devant le timer de la fonction rafraichirmsg

[Anonyme1784]

Merci, mais j'ai déjà essayé sans "var" ça donne la même chose :/

[www.diazis.com]

c'est sans doute une question de porté de la variable timer.
Sans le var dans ta fonction ca me semble juste en javascript.
Maintenant en jquery je ne sais pas.

Bon courage,

[Anonyme1784]

Ah ok merci :/ je cherche toujours, en fait ceci ne fonctionne pas plus :

Code javascript :

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

1
2
3
function cleartimer(){
	clearTimeout(timer);
}

[danielhagnoul]

Voir : http://www.developpez.net/forums/d15...n/#post8272397

[Watilin]

La déclaration de timer est juste. Déclarée au niveau global, une variable sera toujours globale, avec ou sans var. Mais la bonne pratique est de toujours utiliser var. Voir portée des variables.

En réalité, le problème est ici :

Code :

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

timer = setTimeout(rafraichirmsg(desti), 5000);

setTimeout attend comme premier paramètre une référence à une fonction. Là tu lui passes le résultat de l'appel à rafraichirmsg(desti), donc en fait il se passe deux choses :

• rafraichirmsg est appelée immédiatement, et pas 5 secondes après ;
• setTimeout reçoit undefined et donc ne fait rien.

Pour réussir à passer rafraichirmsg à setTimeout tout en gardant le paramètre desti, il faut utiliser une fonction anonyme :

Code :

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

timer = setTimeout(function(){ rafraichirmsg(desti); }, 5000);

[Anonyme1784]

Merci infiniment ^^

[Anonyme1784]

Petit soucis :

[Error] ReferenceError: Can't find variable: d
(anonymous function)

Avec le code ci-dessous, alors que j'appelle la fonction seulement avec rafraichirmsg('general') !

Code javascript :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
function rafraichirmsg(destina){
	desti = destina;
	clearTimeout(timerrefresh);
        $.ajax({
	type: 'POST',
	url: 'chatmsg.php',
	data: {d: destina},
	success: function(b) {
		if(destina!='conx'){
			timerrefresh = setTimeout(function(){ rafraichirmsg(destina); }, 3000);
		}
		$('#messages').html(b);
	}
});
}

Ya un autre problème, c'est qu'avant "destina" c'était "d" mais j'ai changé, cependant j'ai toujours une erreur sur "d" :O

[SpaceFrog]

essaye

Code :

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

data: {"d": destina},

[Watilin]

Ça me paraît bizarre cette histoire de d. J'ai plutôt l'impression qu'il y a une histoire de cache ou de version. To175, tu codes avec quel éditeur ? Et tu débogues avec quel navigateur ?

[Anonyme1784]

Désolé c'est un "d" qui s'était glissé dans une autre fonction