
代码如下,每次运行后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'); }); 1 bdbai 2016 年 8 月 16 日 via Android 这是闭包教程的经典例子 另外你这个逻辑似乎有点问题,回去重新写一下。 |
2 lijsh 2016 年 8 月 16 日 这个和闭包无关,你这里相当于创建了 100 个 app.get('/', fn)的路由处理, url 都一样的,估计是最后一个才起效。 这里都不用 express ,你直接用 superagent 不就行了。 |
3 wodesuck 2016 年 8 月 16 日 via Android 没点进来就知道是这样 因为你里面的 function 实际是在 for 循环结束之后执行的,那时候 i 已经改变了 for (var i ...) { function(i) { ... }(i); } 试一下这样,把 i 作为参数传进去,可以避免受外部的 for 影响 |
4 bdbai 2016 年 8 月 16 日 给你一个改过的例子,用了一些略带花哨的 ES6 特性。要抓的东西你自己改一下吧。 https://gist.github.com/bdbai/3c4d1507c47874efe002683465a7bc25 |
6 Arrowing 2016 年 8 月 16 日 看了标题就想到了结果,果然是这样 还是闭包的问题啊。 |
7 phxsuns 2016 年 8 月 16 日 是你的逻辑有点怪。 创建了 100 个路由处理,还是同一个路由。 那肯定只有一个结果了。 |
8 zhouyg 2016 年 8 月 16 日 猜猜,果然就是闭包和异步引用的问题。 |
9 isbase PRO 用 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'); }); |
10 Biwood 2016 年 8 月 16 日 这种基础的 JS 面试常见知识点在 V2 论坛都快成月经贴了 |
11 xcodebuild 2016 年 8 月 16 日 via Android |
13 m31271n 2016 年 8 月 17 日 看见标题,想到结果系列。 涉及到的知识点: * var 的特性 * 块级作用域 * 作用域链 |