摘要：在中傳入的匿名函數是在這個對象環境下，所以是指向，但是并沒有方法。自從年雙十一正式上線，累計處理了億錯誤事件，得到了金山軟件等眾多知名用戶的認可。

譯者按： JavaScript語言設計太靈活，用起來不免要多加小心掉進坑里面。

原文: Top 10 bugs and their bug fixing

譯者: Fundebug

為了保證可讀性，本文采用意譯而非直譯。另外，本文版權歸原作者所有，翻譯僅用于學習。

如今網站幾乎100%使用JavaScript。JavaScript看上去是一門十分簡單的語言，然而事實并不如此。它有很多容易被弄錯的細節，一不注意就導致BUG。

在閉包或則回調中，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的引用正確
};

在大多數程序語言中，每一個函數塊都有一個獨立的新的作用域，但是在JavaScript中并不是。例如：

for (var i = 0; i < 10; i++) {
  /* ... */
}
console.log(i);  // 會輸出什么呢？

通常在這種情況下，調用console.log()會輸出undefined或則報錯。不過呢，這里會輸出10。在JavaScript中，即使for循環已經結束，變量i依然存在，并且記錄最后的值。有些開發者會忘記這一點，然后導致許多bug。我們可以使用let而不是var來杜絕這一問題。

你需要監控內存使用量，因為泄露很難避免。內存泄露可能由于引用不存在的對象或則循環引用導致。

如何避免：關注對象的可訪問性(reachability)。

可訪問的對象：

現有的call stack任何位置可以訪問的對象

全局對象

當一個對象可以通過引用訪問到，那么會在內存中保存。瀏覽器的垃圾回收器僅僅會把那些不可訪問的對象回收。

JavaScript自動將所有在布爾環境下的變量類型轉換為布爾類型，但是可能導致bug。舉例：

// 所有都是true
console.log(false == "0");
console.log(null == undefined);
console.log(" 	
" == 0);
console.log("" == 0);

// 注意：下面兩個也是
if ({}) // …
if ([]) // …

{}和[]都是對象，他們都會被轉換為true。為了防止bug出現，推薦使用===和!==來做比較，因為不會隱式做類型轉換。

在JavaScript中，你可以輕松操作DOM(添加、修改和刪除)，但是開發者往往很低效地去操作。這會導致bug出現，因為這些操作非常耗費計算資源。為了解決這個問題，推薦使用文檔碎片(Document Fragment)，如果你需要操作多個DOM元素。

廣告： 你的線上代碼真的沒有BUG嗎？歡迎免費使用Fundebug!我們可以幫助您第一時間發現BUG!

舉例：

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變量，該函數被綁定到元素的點擊事件。

開發者如果沒能正確理解繼承的原理，那么就可能寫出有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"

我們來實現一個簡單的構造函數用來創建對象：

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"

如果你將一個字符串作為setTimeout/setTimeInterval，它會被傳給函數構造函數并構建一個新的函數。該操作流程很慢而且低效，并導致bug出現。

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

一個好的替代方法就是傳入函數作為參數：

setInterval(logTime, 1000);   // 將logTime函數傳入

setTimeout(function() {       // 傳入一個匿名函數
    logMessage(msgValue);     
  }, 1000);

使用strict model會增加很多限制條件來加強安全和防止某些錯誤的出現，如果不使用strict mode，你就相當于少了一個得力的助手幫你避免錯誤：

更加容易debug

避免不小心定義了不該定義的全局變量

避免this隱式轉換

避免屬性名字或則參數值的重復使用

eval()更加安全

無效地使用delete會自動拋出錯誤

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/