为什么这两种setTimeout执行顺序不一样

Question

const obj = {

    name: " jsCoder",

    skill: ["es6", "react", "angular"],

    say: function ()

    {

        var that = this;

        for (var i = 0, len = this.skill.length; i < len; i++)

        {

            setTimeout((function()

            {

                console.log(i);

                console.log(that.skill[i]);

            })(), 0)

            console.log(i);

        }

    }

};

obj.say();

这段代码在Node里执行会报错，但可以在浏览器中执行，执行结果：

第二种写法：

const obj = {

    name: " jsCoder",

    skill: ["es6", "react", "angular"],

    say: function () {

        var that = this;

        for (var i = 0, len = this.skill.length; i < len; i++) {

            (function () {

                var j = i;

                setTimeout((function () {

                    console.log(j);

                    console.log(that.skill[j]);

                }), 0)

            })()

            console.log(i);

        }

    }

};

obj.say();

为什么，立即执行函数是异步任务吗？

Answer 1

你的setTimeout的写法是错的（如果我没猜错你的原意的话），setTimeout的第一个参数是一个回调函数，第二个参数是延迟执行的毫秒数。你的第一个参数虽然好像是一个函数，但你把这个函数用括号括起来，又在后面加了个括号立即去调用它，这样setTimeout的第一个参数遍被你设置成了这个函数的返回值而不是让setTimeout来帮你执行这个函数，从你写的这个函数来看，这个返回值应该是undefined，而undefined不是一个函数，它不是可调用的，我猜node里报的错应该是类似TypeError [ERR_INVALID_CALLBACK]: Callback must be a functionat&nbsp;setTimeout&nbsp;(timers.js:425:11)这样的错误吧？正确的写法应该是：setTimeout(function() {&nbsp; // Do something here}, someTime);换种说法就是，你需要传给setTimeout的第一个参数是一个函数指针/引用，而不是当场调用这个函数——这样你将把函数的返回值而不是这个函数本身作为第一个参数传入。另外，setTimeout确实是异步的，并且现在推荐用setImmediate代替setTimeout(func, 0)。