Nodejs callback 函数的 scope 变数范围问题

Nodejs callback 函数的 scope 变数范围问题

在 User_UidSearch 函式当中我想要取用上一层 for 回圈的i ，但没有办法，这个 i 的scope 让我有点疑惑，不晓得各位前辈有没有不使用全域变数的解决办法呢？

Task_TidSearchExecution = function(tid, callback) {
	var sql = "SELECT * FROM execution WHERE task = '" + tid + "'";
	dbclient.query(sql, function (err, results) {
		if (err || results.length <= 0)
			callback(false);
		else {
			console.log(results.length);
			for (var i = 0 ; i < results.length ; i++) {
				User_UidSearch(results[i].employee, function (user) {
					console.log(i);
					// results[i]['email'] = user.email;
				});
			}
			
			callback(results);
		}
	});
}

console.log(result[i]); 输出都是 undefined。

如果我的 results.length 为 2，console.log(i); 都是 2


6 回复

Node.js Callback 函数的 Scope 变数范围问题

在编写 Node.js 应用程序时，我们经常会遇到回调函数中的变量作用域问题。特别是在循环中使用回调函数时，可能会出现一些意料之外的结果。比如在 User_UidSearch 函数中，我们想获取上一层循环中的变量 i，但发现它总是输出循环结束后 i 的值。

示例代码分析

Task_TidSearchExecution = function(tid, callback) {
    var sql = "SELECT * FROM execution WHERE task = '" + tid + "'";
    dbclient.query(sql, function (err, results) {
        if (err || results.length <= 0)
            callback(false);
        else {
            console.log(results.length);
            for (var i = 0 ; i < results.length ; i++) {
                User_UidSearch(results[i].employee, function (user) {
                    console.log(i);
                    // results[i]['email'] = user.email;
                });
            }
            
            callback(results);
        }
    });
}

在这个例子中，results.length 为 2，但是 console.log(i) 输出的都是 2，而不是预期的 0 和 1。这是因为 JavaScript 中的闭包和作用域链导致的。

原因分析

在 JavaScript 中，每次循环迭代都会创建一个新的闭包环境，但所有闭包共享同一个 i 变量。当循环结束时，i 的值已经变成了 results.length（即 2），因此所有回调函数都访问到的是这个最终的 i 值。

解决方案

要解决这个问题，可以使用立即执行函数表达式（IIFE）来创建一个新的作用域，从而捕获每次循环中的 i 值。

Task_TidSearchExecution = function(tid, callback) {
    var sql = "SELECT * FROM execution WHERE task = '" + tid + "'";
    dbclient.query(sql, function (err, results) {
        if (err || results.length <= 0)
            callback(false);
        else {
            console.log(results.length);
            for (var i = 0 ; i < results.length ; i++) {
                (function(index) { // 使用 IIFE 创建一个新的作用域
                    User_UidSearch(results[index].employee, function (user) {
                        console.log(index);
                        // results[index]['email'] = user.email;
                    });
                })(i); // 将当前的 i 传递给 IIFE
            }
            
            callback(results);
        }
    });
}

通过这种方式，每次循环迭代都会创建一个新的 index 变量，这个变量会在回调函数中被正确地捕获，从而避免了作用域问题。

总结

在 Node.js 中处理异步操作时，理解变量的作用域和闭包是非常重要的。使用 IIFE 或其他技术可以帮助我们更好地管理这些复杂性，确保我们的回调函数能够正确地访问所需的变量。


经典问题。

for (var i = 0 ; i < results.length ; i++) {
  (function (j) {
    User_UidSearch(results[j].employee, function (user) {
        console.log(j);
        // results[j]['email'] = user.email;
    });
  })(i);
};

主要思路就是，把 i 变成局部变量，而不是引用，就好了。

经过前辈们指教，解决的上述的问题，以及衍生出来的问题。

程式码如下，不过觉得可读性有点低，想请教有没有更精炼的优化方式。

Task_TidSearchExecution = function(tid, callback) {
	var sql = "SELECT * FROM execution WHERE task = '" + tid + "'";
	dbclient.query(sql, function (err, results) {
		if (err || results.length <= 0)
			callback(false);
		else {
			AddEmployeesInfo(function (results) {
				console.log(results);
				callback(results);
			});

		function AddEmployeesInfo(callback) {
			for (var i = 0 ; i &lt; results.length ; i++) {
				(function (j) {
					User_UidSearch(results[j].employee, function (user) {
						results[j]['uid'] = user.uid;
						results[j]['email'] = user.email;
						if (j == results.length-1)
							callback(results);
					});
				})(i);
			}
		};
	}
});

}

看这术语用的你是台湾人吗？

在Node.js中，callback函数的scope问题常常导致变量作用域混乱。在这个例子中，由于循环中的异步操作，导致所有的回调函数在执行时，for循环已经结束，此时i的值已经是results.length。因此，所有的回调函数都会打印出相同的i值。

解决这个问题的一种方法是利用闭包来保存每次循环中的i值。可以将i作为参数传递给User_UidSearch函数，从而确保每个回调函数都能访问到正确的i值。

下面是修改后的代码示例：

function Task_TidSearchExecution(tid, callback) {
    var sql = "SELECT * FROM execution WHERE task = '" + tid + "'";
    dbclient.query(sql, function (err, results) {
        if (err || results.length <= 0)
            callback(false);
        else {
            console.log(results.length);
            for (var i = 0 ; i < results.length ; i++) {
                (function(index) { // 使用立即执行函数表达式(IIFE)创建一个闭包
                    User_UidSearch(results[index].employee, function (user) {
                        console.log(index);
                        // results[index]['email'] = user.email;
                    });
                })(i);
            }
            
            callback(results);
        }
    });
}

通过使用立即执行函数表达式（IIFE），我们为每次循环创建了一个新的作用域，并将当前的i值传递给index参数。这样，每个回调函数都能正确地获取并使用其对应的index值，而不会受到后续循环迭代的影响。

回到顶部