jquery - 使动画脚本更加高效

Question

我写了一个小动画脚本，几片雪花飘落下来。一切都很好，但我想知道我是否可以让它更有效率，只是因为我不认为自己是一个很好的程序员，而且我确信我在这里或那里犯了一些错误。只是想学习。先感谢您!这是 jsfiddle 的链接

或脚本本身:

    $(document).ready(function ()
{
     var arr = ["1000", "2000", "3000", "4000"];
     var pause = 0;
    var myDiv = $('div');
    function scroll()
    {
       if (pause == 0)
       {
        myDiv.each(function (index)
        {
            myDiv.eq(index).delay(arr[index]).animate({"margin-top":"150px","opacity":"1"},
           {
            duration:3000,
            queue: true,
               complete: function ()
               {
                   $(this).css({"margin-top":"0","opacity":"0"});
                 pause = 0;
                 scroll();  
               }
           });

        });
    }  


    }
    scroll();
});

Answer 1

http://jsfiddle.net/2gxkX/9/

我在这里做了一些调整。总的来说，您的代码很好。

• myDiv -> myDivs 因为它是多个
• myDivs.eq(index) 替换为 $(this)
• 在链接中我添加了新行以使其更具可读性
• 我将 "margin-top" 替换为 top，因为它们是绝对定位的
• "0" 等替换为 0 因为 jquery 知道如何处理数字
• 删除数组以支持更简单的表达式
• 我自己调用了该函数

编辑:简单版本:http://jsfiddle.net/2gxkX/10/