InFaNg
V2EX  ›  Node.js

奇怪的 for 循环问题

  •  
  •   InFaNg · Aug 16, 2016 · 2965 views
    This topic created in 3571 days ago, the information mentioned may be changed or developed.

    代码如下,每次运行后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');
    });
    
    Supplement 1  ·  Aug 16, 2016
    原来 var 是函数级的, let 是块级的,之前一直没注意

    还有一不小心把 express 写在 for 循环里面了😂
    bdbai
        1
    bdbai  
       Aug 16, 2016 via Android   ❤️ 1
    这是闭包教程的经典例子😂
    另外你这个逻辑似乎有点问题,回去重新写一下。
    lijsh
        2
    lijsh  
       Aug 16, 2016   ❤️ 1
    这个和闭包无关,你这里相当于创建了 100 个 app.get('/', fn)的路由处理, url 都一样的,估计是最后一个才起效。

    这里都不用 express ,你直接用 superagent 不就行了。
    wodesuck
        3
    wodesuck  
       Aug 16, 2016 via Android   ❤️ 1
    没点进来就知道是这样😂
    因为你里面的 function 实际是在 for 循环结束之后执行的,那时候 i 已经改变了
    for (var i ...) {
    function(i) {
    ...
    }(i);
    }
    试一下这样,把 i 作为参数传进去,可以避免受外部的 for 影响
    bdbai
        4
    bdbai  
       Aug 16, 2016   ❤️ 1
    给你一个改过的例子,用了一些略带花哨的 ES6 特性。要抓的东西你自己改一下吧。

    https://gist.github.com/bdbai/3c4d1507c47874efe002683465a7bc25
    zhqy
        5
    zhqy  
       Aug 16, 2016
    @lijsh 你这第一句话全说错了。
    Arrowing
        6
    Arrowing  
       Aug 16, 2016
    看了标题就想到了结果,果然是这样 😂
    还是闭包的问题啊。
    phxsuns
        7
    phxsuns  
       Aug 16, 2016
    是你的逻辑有点怪。
    创建了 100 个路由处理,还是同一个路由。
    那肯定只有一个结果了。
    zhouyg
        8
    zhouyg  
       Aug 16, 2016
    猜猜,果然就是闭包和异步引用的问题。
    isbase
        9
    isbase  
    PRO
       Aug 16, 2016   ❤️ 1
    用 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
        10
    Biwood  
       Aug 16, 2016
    这种基础的 JS 面试常见知识点在 V2 论坛都快成月经贴了
    xcodebuild
        11
    xcodebuild  
       Aug 16, 2016 via Android
    InFaNg
        12
    InFaNg  
    OP
       Aug 16, 2016 via Android
    @phxsuns for 写错位置了。。。。
    m31271n
        13
    m31271n  
       Aug 17, 2016
    看见标题,想到结果系列。

    涉及到的知识点:
    * var 的特性
    * 块级作用域
    * 作用域链
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Solana   ·   952 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 73ms · UTC 21:12 · PVG 05:12 · LAX 14:12 · JFK 17:12
    ♥ Do have faith in what you're doing.