10个JavaScript常见BUG及修复方法

译者按: JavaScript语言设计太灵活,用起来不免要多加小心掉进坑里面。

为了保证可读性,本文采用意译而非直译。另外,本文版权归原作者所有,翻译仅用于学习。

如今网站几乎100%使用JavaScript。JavaScript看上去是一门十分简单的语言,然而事实并不如此。它有很多容易被弄错的细节,一不注意就导致BUG。

1. 错误的对this进行引用

在闭包或则回调中,this关键字的作用域很容易弄错。举个例子:

Game.prototype.restart = function () { this.clearLocalStorage(); this.timer = setTimeout(function() { this.clearBoard(); // 此处this指的是? }, 0);};

如果执行上面的代码,我们会看到报错:

Uncaught TypeError: undefined is not a function

出错的原因在于:当你调用setTimeout函数,你实际上调用的是window.setTimeout()。在setTimeout中传入的匿名函数是在window这个对象环境下,所以this是指向window,但是window并没有clearBoard方法。

如何解决呢?定义新的变量引用指向Game对象的this,然后就可以使用啦。

Game.prototype.restart = function () { this.clearLocalStorage(); var self = this; // 将this指向的对象绑定到self this.timer = setTimeout(function(){ self.clearBoard(); }, 0);};

或则使用bind()函数:

Game.prototype.restart = function () { this.clearLocalStorage(); this.timer = setTimeout(this.reset.bind(this), 0); // bind to 'this'};Game.prototype.reset = function(){ this.clearBoard(); // 此处this的引用正确};

2. 和块作用域(block scope)有关的BUG

在大多数程序语言中,每一个函数块都有一个独立的新的作用域,但是在JavaScript中并不是。例如:

for (var i = 0; i < 10; i++) { /* ... */}console.log(i); // 会输出什么呢?

通常在这种情况下,调用console.log()会输出undefined或则报错。不过呢,这里会输出10。在JavaScript中,即使for循环已经结束,变量i依然存在,并且记录最后的值。有些开发者会忘记这一点,然后导致许多bug。我们可以使用let而不是for来杜绝这一问题。

3. 内存泄漏

你需要监控内存使用量,因为泄露很难避免。内存泄露可能由于引用不存在的对象或则循环引用导致。

  • 如何避免:关注对象的可访问性(reachability)。
  • 可访问的对象:
    • 现有的call stack任何位置可以访问的对象
    • 全局对象

当一个对象可以通过引用访问到,那么会在内存中保存。浏览器的垃圾回收器仅仅会把那些不可访问的对象回收。

4. 混淆的相等判断

JavaScript自动将所有在布尔环境下的变量类型转换为布尔类型,但是可能导致bug。举例:

// 所有都是trueconsole.log(false == '0');console.log(null == undefined);console.log(" \t\r\n" == 0);console.log('' == 0);// 注意:下面两个也是if ({}) // …if ([]) // …

{}[]都是对象,他们都会被转换为true。为了防止bug出现,推荐使用===!==来做比较,因为不会隐式做类型转换。

5. 低效的DOM操作

在JavaScript中,你可以轻松操作DOM(添加、修改和删除),但是开发者往往很低效地去操作。这会导致bug出现,因为这些操作非常耗费计算资源。为了解决这个问题,推荐使用文档碎片(Document Fragment),如果你需要操作多个DOM元素。

广告: 你的线上代码真的没有BUG吗?欢迎免费使用Fundebug!我们可以帮助您第一时间发现BUG!

6. 在for循环中错误的定义函数

举例:

var elements = document.getElementsByTagName('input');var n = elements.length; // 假设我们有10个元素for (var i = 0; i < n; i++) { elements[i].onclick = function() { console.log("元素编号#" + i); };}

如果我们有10个元素,那么点击任何一个元素都会显示“元素编号#10”!因为在onclick被调用的时候,for循环已经结束,因此所有的i都是10。

解法:

var elements = document.getElementsByTagName('input');var n = elements.length; // 假设有10个元素var makeHandler = function(num) { // outer function return function() { // inner function console.log("元素编号##" + num); };};for (var i = 0; i < n; i++) { elements[i].onclick = makeHandler(i+1);}

makeHandler在for循环执行的时候立即被调用,获取到当前的值i+1,并且存储在变量num中。makeHandler返回一个函数使用num变量,该函数被绑定到元素的点击事件。

7. 通过原型错误地继承

开发者如果没能正确理解继承的原理,那么就可能写出有bug的代码:

BaseObject = function(name) { if(typeof name !== "undefined") { this.name = name; } else { this.name = 'default' }};var firstObj = new BaseObject();var secondObj = new BaseObject('unique');console.log(firstObj.name); // -> 输出'default'console.log(secondObj.name); // -> 输出'unique'

但是,如果我们做如下操作:

delete secondObj.name;

那么:

console.log(secondObj.name); // -> 输出'undefined'

而我们实际上想要的结果是打印默认的name。

BaseObject = function (name) { if(typeof name !== "undefined") { this.name = name; }};BaseObject.prototype.name = 'default';

每一个BaseObject都继承name属性,并且默认值为default。此时如果secondObjname属性被删除掉,通过原型链查找会返回正确的默认值。

var thirdObj = new BaseObject('unique');console.log(thirdObj.name); // -> 输出'unique'delete thirdObj.name;console.log(thirdObj.name); // -> 输出'default'

8. 实例方法中的无效引用

我们来实现一个简单的构造函数用来创建对象:

var MyObject = function() {}MyObject.prototype.whoAmI = function() { console.log(this === window ? "window" : "MyObj");};var obj = new MyObject();

为了使用方便,我们定义变量whoAmI来引用obj.whoAmI

var whoAmI = obj.whoAmI;

打印出来看看:

console.log(whoAmI);

控制台会输出:

function () { console.log(this === window ? "window" : "MyObj");}

现在我们来对比一下两者调用的区别:

obj.whoAmI(); // 输出"MyObj" (和期望一致)whoAmI(); // 输出"window" (竟然输出了window)

当我们把obj.whoAmI赋值给whoAmI的时候,这个新的变量whoAmI是定义在全局下,因此this指向全局的window,而不是MyObj。如果我们真的要获取对MyObj的函数的引用,需要在其作用域下。

var MyObject = function() {}MyObject.prototype.whoAmI = function() { console.log(this === window ? "window" : "MyObj");};var obj = new MyObject();obj.w = obj.whoAmI; // 任然在obj的作用域obj.whoAmI(); // 输出"MyObj"obj.w(); // 输出"MyObj"

9. setTimeout/setInterval函数第一个参数误用字符串

如果你将一个字符串作为setTimeout/setTimeInterval,它会被传给函数构造函数并构建一个新的函数。该操作流程很慢而且低效,并导致bug出现。

var hello = function(){ console.log("hello, fundebug !");}setTimeout("hello", 1000);

一个好的替代方法就是传入函数作为参数:

setInterval(logTime, 1000); // 将logTime函数传入setTimeout(function() { // 传入一个匿名函数 logMessage(msgValue); }, 1000);

10. 未能成功使用strict mode

使用strict model会增加很多限制条件来加强安全和防止某些错误的出现,如果不使用strict mode,你就相当于少了一个得力的助手帮你避免错误:

  • 更加容易debug
  • 避免不小心定义了不该定义的全局变量
  • 避免this隐式转换
  • 避免属性名字或则参数值的重复使用
  • eval()更加安全
  • 无效地使用delete会自动抛出错误
版权声明:
转载时请注明作者Fundebug以及本文地址:
https://blog.fundebug.com/2017/11/15/top_10_bugs_and_fixing_method/

您的用户遇到BUG了吗?

体验Demo 免费使用

.copyright * { box-sizing: border-box; }

本文参与腾讯云自媒体分享计划,欢迎正在阅读的你也加入,一起分享。

发表于

我来说两句

0 条评论
登录 后参与评论

相关文章

来自专栏高性能服务器开发

(四)sds字符串

今天分析的是Redis源码中的字符串操作类的代码实现。有了上几次的分析经验,渐渐觉得我得换一种分析的方法,如果每个API都进行代码分析,有些功能性的重复,导致分...

36210
来自专栏Ryan Miao

oracle表数据类型number对应java中BIgDecimal转int

oracle中id为number类型,在java获取id时用getBigDecimal 相匹配, 如果想转换成int,重写model中的getInt方法: 1 ...

2816
来自专栏zhisheng

#每日一题#3

如下代码,执行test()函数后,屏幕打印结果为() public class Test2 { public void add(Byte b) ...

36111
来自专栏web前端教室

重学javascript 红皮高程(4)

继续啊,顺着JS高程的目录往下走,今天是3.4.4 Boolean类型。 这个Boolean一般来说它只有二个值,true和false。但其实它还有第三种值, ...

2109
来自专栏函数式编程语言及工具

泛函编程(25)-泛函数据类型-Monad-Applicative

    上两期我们讨论了Monad。我们说Monad是个最有概括性(抽象性)的泛函数据类型,它可以覆盖绝大多数数据类型。任何数据类型只要能实现flatMap+u...

2429
来自专栏一枝花算不算浪漫

一道笔试题来理顺Java中的值传递和引用传递

37011
来自专栏前端新视界

TypeScript 之类型判断

在使用 Angular 做项目的时候,对 TypeScript 的类型判断不太熟练,为了方便查找,特意对 TypeScript 的类型判断做了简单梳理。

2320
来自专栏屈定‘s Blog

(转)Java--栈与队列

Java中栈与队列相比集合来说不是很常用的数据结构,因此经常被忽略.个人觉得还是有必要掌握下,以备不时之需. Java中实际上提供了java.util.Stac...

3213
来自专栏禅林阆苑

golang 利用指针导出变量 【原创】

golang 利用指针导出变量 Write By CS逍遥剑仙 我的主页: www.csxiaoyao.com GitHub: github....

46414
来自专栏Golang语言社区

Golang语言断言

golang中的所有程序都实现了interface{}的接口,这意味着,所有的类型如string,int,int64甚至是自定义的struct类型都就此拥有了i...

31011

扫码关注云+社区