V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
pupboss
V2EX  ›  Node.js

遇到一个很诡异的 bug,关于 routes 的

  •  
  •   pupboss · 2016-03-14 10:47:26 +08:00 · 2803 次点击
    这是一个创建于 3166 天前的主题,其中的信息可能已经有所发展或是发生改变。

    用的 express

    项目结构是这么回事,其实就是个模拟登录:

    • agent
      • dom_agent.js

    传入 url ,账户,密码三个参数,来获取 html

    • parser
      • a.js
      • b.js

    传入 url ,账户,密码三个参数,转发给 dom_agent.js ,然后解析各个类型的 html ,并且按照定义好的格式返回字典( JSON )

    • routes
      • a.js
      • b.js

    var router = express.Router(); router.get 来定义路由,转发请求给 parser ,

    很诡异的问题就是,明明请求的 a 路由, parser a 打印出来的 url 也是正确的,最后返回的数据格式也是正确的,就是数据不对,数据是另外一个网址的

    这还不是最诡异的,最诡异的是,低频率测试的时候,都是没问题的(比如上线前的本地测试),一到生产环境,如果持续请求一个 url ,过不了几次,返回的数据就变成了其余 url 的,而且不固定,不固定啊不固定,不固定啊不固定😂

    数据格式对,就是内容完全混乱

    已经排查过的有:

    开始我以为是没有 return 掉,所以接着执行了,哪个快就返回哪个,所以

    • 每个 callback ,前面都加上 return

    但是发现没用,再一想,根本不是哪个快就返回哪个,因为数据格式是对的,请求的肯定是对应的路由,所以才能发给对应的 parser ,并没有往下执行

    然后我怀疑是模块的缓存,查查资料,说 Node.js 的模块缓存是用来提高加载速度,貌似没关系

    再有可能就是传入的参数,也就是那个 url 参数,被缓存了,但是貌似也不对,因为帐号密码没缓存(并没有发现返回其余用户的数据,只是返回自己其余 url 的数据)

    也可能跟代码风格有关,我现在写法是

    • 每个文件第一行都 'use strict';
    • 每个 callback 前都加上 return
    • 每个 res.xx 前都加上 return
    • 几乎所有情况都是用的单引号
    • 貌似是没了,有的话再补充
    第 1 条附言  ·  2016-03-14 14:35:01 +08:00
    我已经解决这个问题了,在 dom_agent.js 里面

    是全局变量的问题, target 变量不小心搞成全局的了,导致每个用户都会修改这个参数,所以也就导致了刚才的诡异
    pupboss
        1
    pupboss  
    OP
       2016-03-14 10:50:08 +08:00
    对了还要补充几个问题,每个 parser ,里面的方法名,是一样的,不知道这样会不会有影响
    klesh
        2
    klesh  
       2016-03-14 13:20:04 +08:00
    感觉是 dom_agent.js 的问题,这个是不是一个类?每次有 new 吗?
    AccIdent
        3
    AccIdent  
       2016-03-14 13:29:16 +08:00
    能把代码贴出来看下么?
    pupboss
        4
    pupboss  
    OP
       2016-03-14 14:34:30 +08:00
    @AccIdent 贴出来略多。。怕大家懒得看嘛,就自己总结了下问题,项目在 GitHub 上放着呢~

    @klesh 对!!我已经解决这个问题了,就是在 dom_agent.js 里面

    是全局变量的问题, target 变量不小心搞成全局的了,导致每个用户都会修改这个参数,所以也就导致了刚才的诡异
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   5399 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 30ms · UTC 07:26 · PVG 15:26 · LAX 23:26 · JFK 02:26
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.