改进这个连续的 jQuery 动画

Question

我刚刚在 jQuery 中创建了一个简单、连续的 反弹效果 但我觉得代码不是所有这些都经过优化，并且正在寻求改进。

var $square = $("#square");
bounce();
function bounce() {
    $square.animate({
        top: "+=10"
    }, 300, function() {
        $square.animate({
            top: "-=10"
        }, 300, function() {
            bounce();
        })
    });
}

$square.hover(function() {        
    jQuery.fx.off = true;
}, function() {
    jQuery.fx.off = false;
});

我所做的基本上就是创建一个动画，将 +10 添加到元素的顶部坐标，并且作为回调函数，我从顶部坐标中减去 10。

这会创建一个（几乎平滑的）弹跳效果，但我感觉还可以改进。

此外，我想在 mouseenter 上停止动画，并让它在 mouseleave 上继续。

stop(true, true) 不起作用，dequeue() 也不起作用，所以我求助于使用 jQuery.fx.off 关闭所有动画效果= true （愚蠢，不是吗？）如果

有任何有关如何优化的反馈，我将不胜感激。

这是一个 jsFiddle。

编辑：我刚刚意识到 jQuery 在禁用和重新启用效果时开始抛出太多递归错误。

提前致谢，

Marko

Answer 1

请尝试以下代码演示：http://jsfiddle.net/LMXPX/

$(function() {
    var $square = $("#square"),flag = -1;
    var timer = bounce = null;
    (bounce = function () {
        timer = setInterval(function() {
            flag = ~flag + 1;
            $square.animate({ 
                top: "+="+(flag*10)
            }, 300)
        },300);
    })();                
    $square.hover(function() {        
        clearInterval(timer);
    }, function() {
        bounce();
    });
});

编辑：我猜你的代码中多个 CallBack 函数是递归过多的原因

Answer 2

没有太大的改进，但这是我的尝试：

var $square = $("#square");

(function loop () {
    $square
        .animate({ top: "+=10" }, 300)
        .animate({ top: "-=10" }, 300, loop );
})();

$square.hover(function() {
    $.fx.off = !$.fx.off;
});