10个JavaScript常见BUG及修复方法

译者按： JavaScript语言设计太灵活，用起来难免要多加当心掉进坑里面。

• 原文: Top 10 bugs and their bug fixing
• 译者: Fundebug

为了保证可读性，本文采用意译而非直译。另外，本文版权归原做者全部，翻译仅用于学习。

现在网站几乎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而不是var来杜绝这一问题。

3. 内存泄漏

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

• 如何避免：关注对象的可访问性(reachability)。

• 可访问的对象：

  • 现有的call stack任何位置能够访问的对象
  • 全局对象

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

4. 混淆的相等判断

JavaScript自动将全部在布尔环境下的变量类型转换为布尔类型，可是可能致使bug。举例：

// 全部都是true
console.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。此时若是secondObj的name属性被删除掉，经过原型链查找会返回正确的默认值。

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

Fundebug专一于JavaScript、微信小程序、微信小游戏、支付宝小程序、React Native、Node.js和Java实时BUG监控。 自从2016年双十一正式上线，Fundebug累计处理了6亿+错误事件，获得了Google、360、金山软件等众多知名用户的承认。欢迎免费试用！

版权声明

转载时请注明做者Fundebug以及本文地址：
https://blog.fundebug.com/2017/11/15/top_10_bugs_and_fixing_method/