快速概览

当我们说函数是“一等公民”的时候，我们实际上说的是它们和其他对象都一样…所以就是普通公民（坐经济舱的人？）。函数真没什么特殊的，你可以像对待任何其他数据类型一样对待它们——把它们存在数组里，当作参数传递，赋值给变量…等等。

这是 JavaScript 语言的基础概念，不过还是值得提一提的，因为在 Github 上随便一搜就能看到对这个概念的集体无视，或者也可能是无知。我们来看一个杜撰的例子：

var hi = function(name){
  return "Hi " + name;
};
var greeting = function(name) {
  return hi(name);
};

这里 greeting 指向的那个把 hi 包了一层的包裹函数完全是多余的。为什么？因为 JavaScript 的函数是可调用的，当 hi 后面紧跟 () 的时候就会运行并返回一个值；如果没有 ()，hi 就简单地返回存到这个变量里的函数。我们来确认一下：

hi;
// function(name){
//  return "Hi " + name
// }
hi("jonas");
// "Hi jonas"

greeting 只不过是转了个身然后以相同的参数调用了 hi 函数而已，因此我们可以这么写：

var greeting = hi;
greeting("times");
// "Hi times"

换句话说，hi 已经是个接受一个参数的函数了，为何要再定义一个额外的包裹函数，而它仅仅是用这个相同的参数调用 hi？完全没有道理。这就像在大夏天里穿上你最厚的大衣，只是为了跟热空气过不去，然后吃上个冰棍。真是脱裤子放屁多此一举。

用一个函数把另一个函数包起来，目的仅仅是延迟执行，真的是非常糟糕的编程习惯。（稍后我将告诉你原因，跟可维护性密切相关。）

充分理解这个问题对读懂本书后面的内容至关重要，所以我们再来看几个例子。以下代码都来自 npm 上的模块包：

// 太傻了
var getServerStuff = function(callback){
  return ajaxCall(function(json){
    return callback(json);
  });
};
// 这才像样
var getServerStuff = ajaxCall;

世界上到处都充斥着这样的垃圾 ajax 代码。以下是上述两种写法等价的原因：

// 这行
return ajaxCall(function(json){
  return callback(json);
});
// 等价于这行
return ajaxCall(callback);
// 那么，重构下 getServerStuff
var getServerStuff = function(callback){
  return ajaxCall(callback);
};
// ...就等于
var getServerStuff = ajaxCall; // <-- 看，没有括号哦

各位，以上才是写函数的正确方式。一会儿再告诉你为何我对此如此执着。

var BlogController = (function() {
  var index = function(posts) {
    return Views.index(posts);
  };
  var show = function(post) {
    return Views.show(post);
  };
  var create = function(attrs) {
    return Db.create(attrs);
  };
  var update = function(post, attrs) {
    return Db.update(post, attrs);
  };
  var destroy = function(post) {
    return Db.destroy(post);
  };
  return {index: index, show: show, create: create, update: update, destroy: destroy};
})();

这个可笑的控制器（controller）99% 的代码都是垃圾。我们可以把它重写成这样：

var BlogController = {index: Views.index, show: Views.show, create: Db.create, update: Db.update, destroy: Db.destroy};

…或者直接全部删掉，因为它的作用仅仅就是把视图（Views）和数据库（Db）打包在一起而已。