10个JavaScript常见BUG及修复方法

发表于:2017-12-07 09:48

字体: | 上一篇 | 下一篇 | 我要投稿

 作者:佚名    来源:SegmentFault

  如今网站几乎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元素。
  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会自动抛出错误

上文内容不用于商业目的,如涉及知识产权问题,请权利人联系博为峰小编(021-64471599-8017),我们将立即处理。
《2023软件测试行业现状调查报告》独家发布~

关注51Testing

联系我们

快捷面板 站点地图 联系我们 广告服务 关于我们 站长统计 发展历程

法律顾问:上海兰迪律师事务所 项棋律师
版权所有 上海博为峰软件技术股份有限公司 Copyright©51testing.com 2003-2024
投诉及意见反馈:webmaster@51testing.com; 业务联系:service@51testing.com 021-64471599-8017

沪ICP备05003035号

沪公网安备 31010102002173号