初学者应该了解的编程陷阱：javascript篇

日期：2012-12-4  来源：GBin1.com

对于初学编程语言的朋友来说，经常会犯一些比较“经典”的错误。在这一系列文章中，我们将介绍如何让初学编程的朋友避免一些基本的错误，并且了解如何正确的编程和开发代码。

这一篇文章中，我们介绍5个javascript代码开发中需要注意的问题和技巧。

冗余的DOM操作

DOM操作众所周知是比较重量级的。有效的限制交互可以大大的帮助你提高你的代码的性能。看看下面这段代码：

// anti-pattern
for (var i = 0; i < 100; i++){
   var li = $("<li>").html("This is list item #" + (i+1));
   $("#someUL").append(li);
}

这段代码循环修改DOM100次，但是创建了100个jQuery对象。更好的方式是使用document片段，或者创建一个String来包含100个<li/>元素，然后再添加到DOM中去。这里你只需要执行一次DOM操作。

优化后的代码：

var liststring = "";
for (var i = 100; i > 0; i--){
   liststring += "<li>This is list item #" + (99- i);
}
document.getElementById("someUL").innerHTML(liststring);

以上代码只调用了一次DOM操作，但是大量使用了字符串的连接操作。除了使用字符串连接，我们可以使用数组来显示。

var liststring = "<li>"
var lis = [];
for (var i = 100; i > 0; i--){
   lis.push("This is list item #" + (99- i));
}
liststring += lis.join("</li><li>") + "</li>";
document.getElementById("someUL").innerHTML(liststring);

当创建大量的字符串时，保存每一个字符串到数组里，然后调用join()方法来连接。在javascript中，这是不使用模板类库和框架外的最高效的字符串连接操作方式。

在线调试

变量和方法名不一致

这可能不是一个性能问题，但是对于编程来说非常重要，特别是当你需要维护别人的代码。看看如下例子：

var foo = "bar";
var plant = "green";
var car = "red";

如果你再添加一个变量叫“something”可能就不是那么合适了，你应该保持命名的一致。这也就是为什么在一些编程语言中，我们使用大写变量名来代表常量。

对于方法来说，我们也需要保持一致，如下：

function subtractFive(number){
   return number - 5;
}

如果你有一个以上的减5方法，那么你如果定义加5方法的话，应该使用如下命名方式：

function addFive(number){
   return number + 5;
}

有时候如果你定义一个返回方法，那么一般使用getXXX()，如果你只是执行操作不返回，那么最好使用doXXX()方法名。

构造器方法最好使用类似其它语言的命名方式，首字母大写，如下：

function Gbin1(color){
   this.color = color;
}

不管怎么，你都应该尽量的让你的命名更加富有意义，能够给别人更多的信息。

在for..in循环中使用 hasOwnProperty 方法

在javascript中数组是不关联的。而对象来说是hashtable类型的。你可以使用for ... in循环来迭代对象属性。如下：

for (var prop in someObject) {
    alert(someObject[prop]); // alert's value of property
} 

但是问题在于，以上代码会循环proptype chain中的所有的属性，有时候这样会出错，你可能只想使用实际存在的属性。使用hasOwnProperty方法可以帮助你解决这个问题。

for (var prop in someObject) {
    if (someObject.hasOwnProperty(prop)) {
        alert(someObject[prop]); // alert's value of property
    }
}

这个方法可以帮助你得到实际存在的属性值。

比较boolean值

比较boolean值非常浪费计算时间。看看下面这个例子：

if (foo == true) {
    // do something for true
} else {
    // do something for false
}

注意上面的==true条件，这个非常没有必要因为foo本身就是boolean值。如果比较，应该使用如下代码：

if (foo) {
    // do something for true
} else {
    // do something for false
}

或者测试foo为false，如下：

if (!foo) {
    // do something for true
} else {
    // do something for false
}

事件绑定

事件在javascript中是非常的复杂的话题。以往我们使用行内oncick事件的时代已经过去了。。

我们应该使用事件bubbling或者delegation。

举个例子，如果我们需要让一组图片展示到lightbox window里。那么下面这段代码你绝对不应该使用。

这里我们使用jQuery作为例子。

HTML：

<div id="grid-container">
   <a href="someimage.jpg"><img src="someimage-thumb.jpg"></a>
   <a href="someimage.jpg"><img src="someimage-thumb.jpg"></a>
   <a href="someimage.jpg"><img src="someimage-thumb.jpg"></a>
   ...
</div>

Javascript（不推荐的写法）：

$('a').on('click', function() {
   callLightbox(this);
});

这样书写会导致绑定事件到每一个链接元素，最好绑定到指定的图片容器，如下：

$("#grid-container").on("click", "a", function(event) {
   callLightbox(event.target);
});

在线调试

避免冗余的比较

在javascript和PHP中：

// javascript
return foo.toString() !== "" ? true : false;
    
// php
return (something()) ? true : false;

但是条件比较永远返回true或者false，所以你没有必要清楚的添加返回值。以下代码即可：

// javascript
return foo.toString() !== "";

// php
return something();

希望大家能够理解并且了解如何避免代码书写中的问题，如果你有其它的例子，请与我们分享！

来源：初学者应该了解的编程陷阱：javascript篇