博客
关于我
强烈建议你试试无所不能的chatGPT,快点击我
如何快速定位不小心暴露到全局的变量
阅读量:6444 次
发布时间:2019-06-23

本文共 1625 字,大约阅读时间需要 5 分钟。

今天在查看页面控制台的时候,无意中看到了一个暴露到全局的变量 i,全局变量是不会被压缩工具压缩成简写的字母,这个被频繁使用的变量名暴露到全局也是个相当大的隐患,可能一个不小心就覆盖了第二次暴露到全局的同名变量。

刚开始我就怀疑是自己出现了这样愚蠢的错误:

function A() {    // 在一个函数中多次用到了 for 循环,为了节省变量,都是用了变量 i    for(var i = 0; ...) {        //...    }    for(i = 0; ...) {        //...    }    for(i = 0; ...) {        //...    }}

结果在某次拆分函数的时候,忘记定义:

function A(){    for(var i = 0; ...) {        //...    }    for(i = 0; ...) {        //...    }}function B(){    for(i = 0; ...) {        //...    }}

这个时候,变量 i 在 B 函数执行的时候就暴露到了全局。抱着这样的怀疑,我搜索了 50 多个模块的代码,一无所获...此时,我依然十分怀疑是自己的程序哪里疏忽了,全局搜索 i =i++,五分钟过去了,未果...

找到这个变量

如果这个变量名叫做 fuckIE,分分钟全局搜索就出来了,类似这种简短的常用的变量,着实让人头疼了好一会儿。后来想到了这个方案:

Object.defineProperty(window, "i", {    get : function(){ return window.i; },    set : function(newValue){ debugger;window.i = newValue; },    enumerable : true,    configurable : true});

在全局定义变量 i 的时刻,打一个断点,然后 F10 往前走一步,果然,在控制台右侧的 Call Stack 中找到了端倪!

这个变量是从第三方组件中(offline组件,使用相当频繁的一个组件)暴露出来的,估计出错的方式同我上面的描述差不多,拆分函数的时候忘记重新定义变量 i。

当然还有更快的方式:

window.__defineSetter__('i', function(){ debugger })

不挖坑才是最好的解决方案

1. 使用 use strict;

在严格模式下,这种问题暴露无遗,每个函数内都加上 use strict;,虽然在语言上有所限制,但是低级错误一定不会出现,因为严格模式会给你报错!

2. 使用 jslint/jshint 等 js 分析工具

这些东西除了配置上较为繁琐,用起来还是很顺手的,做过配置的错误都会直接在 IDE 上标红显示出来,很容易发现问题,但是不建议一个项目中途使用,因为代码习惯的问题,很多地方被 js 分析工具作为错误抛出来,改动量是相当大的。

我有次也犯了个比较隐晦的错误:

$(window).on('click', function(evt){    var target = event.target.nodeName.toLowerCase();    if(target !== 'ul'){        //...    }});

在 IE 和 Chrome 下,代码跑得好好的,但是到了测试较少的 FF 下,问题出来了,event is not defined.,IE 和 Chrome 是支持 window.event 抓取当前事件对象的,而 FF 不支持,所以每次点击页面上都会报错。。。

诸如此类的问题,在我们的平时编码之中不胜枚举,所以有一个编码规范作为强约束是十分有必要的!

 

转载地址:http://vupwo.baihongyu.com/

你可能感兴趣的文章
20180416
查看>>
判断一个数是否为素数
查看>>
15年用canvas画的
查看>>
HTML5新特性
查看>>
log4j.appender.AFile.File日志的相对路径
查看>>
乒乓球比赛模拟分析
查看>>
多元线性回归的预测
查看>>
Django——Model
查看>>
servlet体系和生命周期
查看>>
C#前台线程和后台线程区别
查看>>
服务器开发工具
查看>>
个人项目1修改版——自动生成四则表达式
查看>>
更改mysql数据库所在目录
查看>>
在 IIS6 中使用 GZIP
查看>>
Powershell DSC 5.0 - 资源的使用
查看>>
力求企业的内部公平性
查看>>
MySQL5.6优化了Order by排序limit
查看>>
Gartner公司警告,由于企业削减投入,大数据泡沫或破裂
查看>>
TCP/IP协议栈中,为什么选择IP层负责分片?
查看>>
不治已病治未病
查看>>