Discussion :

[Amnesiak]

Bonsoir,

J'ai réussi à isoler dans l'exemple ci-dessous un phénomène que je n'arrive pas à expliquer. Il s'agit d'une animation d'un élément pour laquelle la fonction callback semble se lancer 2x. Pour bien comprendre ce que je cherche à réaliser, j'ai réduit le problème en un exemple simpliste de chat/souris :

Au chargement de la page, 1 petit div (qui représente une souris) se déplace aléatoirement de gauche à droite grâce à une animation infinie. Idéalement il doit y avoir plusieurs souris, mais pour simplifier je n'en affiche qu'une.
Lors du clic sur un bouton, 1 autre div (le chat) commence à se déplacer. Une fois l'animation du chat terminée, je décide d'immobiliser les souris.

Voici le code :

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
31
32
33
34
35
36
37
38
<script type="text/javascript">
$(document).ready(function(){
 
    // animation infinie d'une souris
    function animateMouse(id)
    {
        var x = Math.floor(Math.random()*200);
        $("#"+id).animate({"margin-left":x},400,"linear",function(){
            animateMouse(id);
        });
    }
 
    // démarre l'animation de la souris
    animateMouse("mouse1");
 
    $("#button").click(function(){
 
        console.log("click!");
 
        // démarre l'animation du chat
        $("#cat").animate({"margin-left":200},200,"linear",function(){
 
            console.log("cat stopped");
 
            // immobilise toutes les souris
            $(".mouse").each(function(i,e){
                console.log("stopping mouse "+$(e).attr("id"));
                $(e).stop();
            });
        });
    });
});
</script>
 
<input type="button" id="button" value="START" />
<div id="cat"></div>
<div id="mouse1" class="mouse"></div>
<div id="mouse2" class="mouse"></div>

Malgré que ce soit simple, il semble que le callback de l'animation du chat soit exécuté 2x, comme le montre le résultat suivant. Par contre j'ai l'impression que ce n'est pas systématique... on dirait que parfois ça se déroule correctement.

Code :

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

1
2
3
4
5
click!
cat stopped
stopping mouse mouse1
cat stopped
stopping mouse mouse1

Comble de l'étrangeté, si je commente la ligne $(e).stop(), l'animation se déroule toujours correctement. J'avoue ne pas y comprendre grand chose.

Quelqu'un aurait une explication ? Peut-être que je m'y prend mal pour la boucle infinie ? Merci pour vos idées, toute piste est bonne à prendre.

PS : J'ai testé sous Firefox 7.0.1 avec jQuery1.6.4, 1.6.3 et 1.6.2

[Amnesiak]

Bonjour, je ne sais pas si ma description du problème est assez compréhensible. Pour ceux qui souhaiteraient tester par l'exemple, je l'ai mis à disposition ici : http://www.geoffray.be/public/demo/anim_jquery.html

Merci d'avance

[MrPringle]

Sous Chrome chez moi le log n'a lieu qu'une seule fois. Tu utilises quel navigateur ?

[SpaceFrog]

Je ne vois rien d'anormal sous ffx ...

[Amnesiak]

Le cas ne se produit pas à chaque fois... il faut recharger la page puis cliquer sur le bouton. Parfois le log affiche 2x "cat stopped" alors que ça ne devrait apparaître qu'une seule fois.

Parfois le callback est appelé 2x :

Code :

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

1
2
3
4
5
click!
cat stopped
stopping mouse mouse1
cat stopped
stopping mouse mouse1

Et parfois une seule fois, ce qui est normal:

Code :

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

1
2
3
click!
cat stopped
stopping mouse mouse1

Après quelques essais, j'ai pu reproduire le problème sous Firefox 7.0.1 (XP et Seven), Chrome 14.0.835 (XP), Safari 5.0.5 (XP), ainsi qu'Opera 11.11 (XP)

[danielhagnoul]

Bonsoir

Je suggère de tester : $(e).stop(true, true);

[Amnesiak]

Bonjour Daniel, merci pour ta suggestion mais elle n'y change rien malheureusement. J'ai même essayé toutes les combinaisons de paramètres dans la méthode stop()

Honnêtement, je ne vois pas où je m'y prends mal... mais avant d'imaginer l'hypothèse d'un bug dans la lib, je préfère me dire qu'il doit y avoir une subtilité qui m'échappe... mais ou ?

PS : Est-ce que le problème indiqué se manifeste aussi chez d'autres personnes, ou suis-je le seul ?

[danielhagnoul]

Bonjour

Le problème existe bien, c'est une bogue de la version 1.6.4 qui semble avoir été corrigée dans la version 1.7 b1. Attention cette version Beta à d'autres bogues surtout avec UI.

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
<!doctype html>
<html lang="fr">
<head>
	<meta charset="utf-8">
	<meta name="Author" content="Daniel Hagnoul">
	<title>Forum jQuery</title>
	<style>
		#cat {
			width:60px;
			height:20px;
			background-color:black;
		}
		.mouse {
			width:20px;
			height:20px;
			background-color:#c8c8c8;
		}
	</style>
</head>
<body>	
	<input type="button" id="button" value="START" />
	<div id="cat"></div>
	<div id="mouse1" class="mouse"></div>
	<!-- <script charset="utf-8" src="http://code.jquery.com/jquery-1.6.4.min.js"></script> -->
	<script charset="utf-8" src="http://code.jquery.com/jquery-1.7b1.min.js"></script>
	<script>
		$(function(){
			function animateMouse(id)
			{
				var x = Math.floor(Math.random()*200);
				$("#"+id).animate({"margin-left":x},400,"linear",function(){
					animateMouse(id);
				});
			}
 
			animateMouse("mouse1");
 
			$("#button").click(function(){
				console.log("click!");
				$("#cat").animate({"margin-left":200},200,"linear",function(){
					console.log("cat stopped");
					$(".mouse").each(function(i,e){
						console.log("stopping mouse "+$(e).attr("id"));
						$(e).stop();
					});
				});
			});
		});
	</script>
</body>  
</html>

[Amnesiak]

Merci Daniel pour cette nouvelle ! Aurais-tu par hasard plus d'infos sur le bug en question car je ne le trouve pas dans le changelog de la 1.7 http://blog.jquery.com/2011/09/28/jq...ta-1-released/
Il serait en effet utile de savoir dans quelles conditions le cas se produit, car si n'importe quel callback est susceptible d'être démarré plusieurs fois, c'est assez problématique !

En attendant une release stable, j'ai choisi d'opter pour un "hack" maison...

Code :

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

1
2
3
4
5
6
7
8
        $("#cat").data("anim",true).animate({"margin-left":200},200,"linear",function(){
            if ($(this).data("anim")) {
                $(this).data("anim",false);
                console.log("cat stopped");
 
                // ...
            }
        });

[danielhagnoul]

Bonsoir

Avec 1.6.4, en utilisant une vraie fonction plutôt qu'une fonction anonyme comme fonction de rappel, je ne vois plus la bogue causée par la méthode stop (voir la liste des corrections de bogue de 1.7b1).

Blog jQuery, extrait du "changelog" de 1.7b1

Effects

#5684: Effects: exception in animation callback causes endless loop
#6150: .stop sometimes doesn’t clear .delay
#6641: Calling stop() within animation finished callback causes other animations to freeze
#8685: Animations should keep track of animation state in order to properly address stacked animations
#9280: Allow multiple effect queues for animate()
#9548: animate does not work with fill-opacity css property for svg elements

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
<!doctype html>
<html lang="fr">
<head>
	<meta charset="utf-8">
	<meta name="Author" content="Daniel Hagnoul">
	<title>Forum jQuery</title>
	<style>
		#cat {
			width:60px;
			height:20px;
			background-color:black;
		}
		.mouse {
			width:20px;
			height:20px;
			background-color:#c8c8c8;
		}
	</style>
</head>
<body>	
	<input type="button" id="button" value="START" />
	<div id="cat"></div>
	<div id="mouse1" class="mouse"></div>
	<script charset="utf-8" src="http://code.jquery.com/jquery-1.6.4.min.js"></script>
	<!-- <script charset="utf-8" src="http://code.jquery.com/jquery-1.7b1.min.js"></script> -->
	<script>
		$(function(){
			function animateMouse(id)
			{
				var x = Math.floor(Math.random()*200);
				$("#"+id).animate({"margin-left":x},400,"linear",function(){
					animateMouse(id);
				});
			}
 
			function animateFnRap(){
				console.log("cat stopped");
 
				$(".mouse").each(function(i,e){
					console.log("stopping mouse "+$(e).attr("id"));
					$(e).stop();
				});
			}
 
			animateMouse("mouse1");
 
			$("#button").click(function(){
				console.log("click!");
				$("#cat").animate({"margin-left":200},200,"linear", animateFnRap);
			});
		});
	</script>
</body>  
</html>