奇怪的 for 循环问题

[InFaNg]

代码如下，每次运行后items中只有i=100时的情况

var express = require('express');
var cheerio = require('cheerio');
var superagent = require('superagent');
var items = [];

var app = express();
for(var i=1;i<100;i++) {
  app.get('/', function (req, res, next) {
    superagent.get('http://xxx/' + i + '.html')
      .end(function (err, sres) {
        if (err) {
          return next(err);
        }
        var $ = cheerio.load(sres.text);
        items.push($('.d1').attr('href'));
        res.send(items);
      });
  });
}

app.listen(3000, function () {
  console.log('app is listening at port 3000');
});

[bdbai]

这是闭包教程的经典例子😂
另外你这个逻辑似乎有点问题，回去重新写一下。

[lijsh]

这个和闭包无关，你这里相当于创建了 100 个 app.get('/', fn)的路由处理， url 都一样的，估计是最后一个才起效。

这里都不用 express ，你直接用 superagent 不就行了。

[wodesuck]

没点进来就知道是这样😂
因为你里面的 function 实际是在 for 循环结束之后执行的，那时候 i 已经改变了
for (var i ...) {
function(i) {
...
}(i);
}
试一下这样，把 i 作为参数传进去，可以避免受外部的 for 影响

[bdbai]

给你一个改过的例子，用了一些略带花哨的 ES6 特性。要抓的东西你自己改一下吧。

https://gist.github.com/bdbai/3c4d1507c47874efe002683465a7bc25

[zhqy]

@lijsh 你这第一句话全说错了。

[Arrowing]

看了标题就想到了结果，果然是这样 😂
还是闭包的问题啊。

[phxsuns]

是你的逻辑有点怪。
创建了 100 个路由处理，还是同一个路由。
那肯定只有一个结果了。

[zhouyg]

猜猜，果然就是闭包和异步引用的问题。

[isbase]

用 ES6 的 let

var express = require('express');
var cheerio = require('cheerio');
var superagent = require('superagent');
var items = [];

var app = express();
for(var i=1;i<100;i++) {
let j = i;
app.get('/', function (req, res, next) {
superagent.get('http://xxx/' + j + '.html')
.end(function (err, sres) {
if (err) {
return next(err);
}
var $ = cheerio.load(sres.text);
items.push($('.d1').attr('href'));
res.send(items);
});
});
}

app.listen(3000, function () {
console.log('app is listening at port 3000');
});

[Biwood]

这种基础的 JS 面试常见知识点在 V2 论坛都快成月经贴了

[xcodebuild]

https://codefalling.com/2016/07/23/var-let-in-javascript/

用 let

[InFaNg]

@phxsuns for 写错位置了。。。。

[m31271n]

看见标题，想到结果系列。

涉及到的知识点：
* var 的特性
* 块级作用域
* 作用域链