10个JavaScript常见BUG及修复方法

忘是亡心i 2021-09-20 09:18 284阅读 0赞

如今网站几乎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。举例：

// 所有都是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。

解法：

`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]: https://link.zhihu.com/?target=https%3A//fundebug.com/