JavaScript 编程风格

所谓"编程风格"（programming style），指的是编写代码的样式规则。不同的程序员，往往有不同的编程风格。

有人说，编译器的规范叫做"语法规则"（grammar），这是程序员必须遵守的；而编译器忽略的部分，就叫"编程风格"（programming style），这是程序员可以自由选择的。这种说法不完全正确，程序员固然可以自由选择编程风格，但是好的编程风格有助于写出质量更高、错误更少、更易于维护的程序。

所以，"编程风格"的选择不应该基于个人爱好、熟悉程度、打字量等因素，而要考虑如何尽量使代码清晰易读、减少出错。你选择的，不是你喜欢的风格，而是一种能够清晰表达你的意图的风格。这一点，对于JavaScript这种语法自由度很高的语言尤其重要。

必须牢记的一点是，如果你选定了一种“编程风格”，就应该坚持遵守，切忌多种风格混用。如果你加入他人的项目，就应该遵守现有的风格。

语法标记的风格

大括号的位置

绝大多数的编程语言，都用大括号（{}）表示区块（block）。起首的大括号的位置，有许多不同的写法。

最流行的有两种。一种是起首的大括号另起一行：


block
{
　　...
}

另一种是起首的大括号跟在关键字的后面：


block {
　　...
}

一般来说，这两种写法都可以接受。但是，JavaScript要使用后一种，因为JavaScript会自动添加句末的分号，导致一些难以察觉的错误。


return
{
　　key:value;
};

上面的代码的原意，是要返回一个对象，但实际上返回的是undefined，因为JavaScript自动在return语句后面添加了分号。为了避免这一类错误，需要写成下面这样：


return {
　　key : value;
};

因此，表示区块起首的大括号，不要另起一行。

圆括号

圆括号（parentheses）在JavaScript中有两种作用，一种表示函数的调用，另一种表示表达式的组合（grouping）。


console.log("abc") // 圆括号表示函数的调用
(1+2) * 3 // 圆括号表示表达式的组合

我们可以用空格，区分这两种不同的括号。

  1. 表示函数调用时，函数名与左括号之间没有空格。

  2. 表示函数定义时，函数名与左括号之间没有空格。

  3. 其他情况时，前面位置的语法元素与左括号之间，都有一个空格。

按照上面的规则，下面的写法都是不规范的：


foo (bar)
return(a+b);
if(a === 0) {...}
function foo (b) {...}
function(x) {...}

上面代码的最后一行是一个匿名函数，function是语法关键字，不是函数名，所以与左括号之间应该要有一个空格。

缩进

空格和Tab键，都可以产生缩进效果（intent）。Tab键可以节省击键次数，但不同的文本编辑器对Tab的显示不尽相同，有的显示四个空格，有的显示两个空格，所以有人觉得，空格键可以使得显示效果更统一。

无论你选择哪一种方法，都是可以接受的，要做的就是始终坚持这一种选择。不要一会使用tab键，一会使用空格键。

行尾的分号

分号表示语句的结束。大多数情况下，如果你省略了句尾的分号，JavaScript会自动添加。


var a = 1

等同于


var a = 1;

因此，有人提倡省略句尾的分号。但麻烦的是，如果句尾的最后一个字符与下一行的第一个字符，可以连在一起解释，JavaScript就不会自动添加分号，这会产生一些难以察觉的错误。


x = y
(function (){
　　...
})();

上面的代码等同于


x = y(function (){...})();

因此，不要省略句末的分号。

区块

如果循环和判断的代码体只有一行，JavaScript允许该区块（block）省略大括号。

下面的代码


if (a) b(); c();

原意可能是


if (a) { b(); c();}

但是，实际效果是


if (a) { b();} c();

因此，总是使用大括号表示区块。

语法命令的风格

全局变量

JavaScript最大的语法缺点，可能就是全局变量对于任何一个代码块，都是可读可写。这对代码的模块化和重复使用，非常不利。

因此，避免使用全局变量；如果不得不使用，用大写字母表示变量名，比如UPPER_CASE。

new命令

JavaScript使用new命令，从构造函数生成一个新对象。


var o = new myObject();

这种做法的问题是，一旦你忘了加上new，myObject()内部的this关键字就会指向全局对象，导致所有绑定在this上面的变量，都变成全局变量。

因此，尽量使用Object.create()命令，替代new命令。如果不得不使用new，为了防止出错，最好在视觉上把构造函数与其他函数区分开来。比如，构造函数的函数名，采用首字母大写（InitialCap）；其他函数名，一律首字母小写。

变量声明

JavaScript会自动将变量声明"提升"（hoist）到代码块（block）的头部。


if (!o) {
　　var o = {};
}

等同于


var o;
if (!o) {
　　o = {};
}

为了避免可能出现的问题，不如把变量声明都放在代码块的头部。


for (var i ...) {...}

最好写成：


var i;

for (i ...) {...}

因此，所有变量声明都放在函数的头部，所有函数都在使用之前定义。

with语句

with可以减少代码的书写，但是会造成混淆。


with (o) {
　　foo = bar;
}

上面的代码，可以有四种运行结果：


o.foo = bar;
o.foo = o.bar;
foo = bar;
foo = o.bar;

这四种结果都可能发生，取决于不同的变量是否有定义。因此，不要使用with语句。

相等和严格相等

JavaScript有两个表示"相等"的运算符："相等"（==）和"严格相等"（===）。

因为"相等"运算符会自动转换变量类型，造成很多意想不到的情况：


0 == ''// true
1 == true // true
2 == true // false
0 == '0' // true
false == 'false' // false
false == '0' // true
" \t\r\n " == 0 // true

因此，不要使用"相等"（==）运算符，只使用"严格相等"（===）运算符。

语句的合并

有些程序员追求简洁，喜欢合并不同目的的语句。比如，原来的语句是


a = b;
if (a) {...}

他喜欢写成下面这样:


if (a = b) {...}

虽然语句少了一行，但是可读性大打折扣，而且会造成误读，让别人误以为这行代码的意思是：


if （a === b）{...}

另外一种情况是，有些程序员喜欢在同一行中赋值多个变量：


var a = b = 0;

他以为，这行代码等同于


var a = 0, b = 0;

实际上不是，它的真正效果是下面这样：


b = 0;

var a = b;

因此，不要将不同目的的语句，合并成一行。

自增和自减运算符

自增（++）和自减（--）运算符，放在变量的前面或后面，返回的值不一样，很容易发生错误。

事实上，所有的++运算符都可以用"+= 1"代替。


++x

等同于


x += 1;

代码变得更清晰了。有一个很可笑的例子，某个JavaScript函数库的源代码中出现了下面的片段：


++x;
++x;

这个程序员忘了，还有更简单、更合理的写法：


x += 2;

因此，不要使用自增（++）和自减（--）运算符，用+=和-=代替。

switch...case结构

switch...case结构要求，在每一个case的最后一行必须是break语句，否则会接着运行下一个case。这样不仅容易忘记，还会造成代码的冗长。

而且，switch...case不使用大括号，不利于代码形式的统一。此外，这种结构类似于goto语句，容易造成程序流程的混乱，使得代码结构混乱不堪，不符合面向对象编程的原则。


function doAction(action) {
  switch (action) {
    case 'hack':
      return 'hack';
    break;

    case 'slash':
      return 'slash';
    break;

    case 'run':
      return 'run';
    break;

    default:
      throw new Error('Invalid action.');
    break;
  }
}

上面的代码建议改写成对象结构。


function doAction(action) {
  var actions = {
    'hack': function () {
      return 'hack';
    },

    'slash': function () {
      return 'slash';
    },

    'run': function () {
      return 'run';
    }
  };

  if (typeof actions[action] !== 'function') {
    throw new Error('Invalid action.');
  }

  return actions[action]();
}

因此，避免使用switch...case结构，用对象结构代替。

eval函数

eval函数的作用是将一段字符串当作语句执行。问题是eval不提供单独的作用域，而是直接在当前作用域运行。这会造成在不知不觉中，eval中的语句在当前作用域创建新变量、修改已有的变量，使得恶意代码有机可乘。下面就是一个例子。


eval('var x = 10');

因此，避免使用eval函数。

参考链接