聊聊jQuery的反模式

摘要：如果我们认为模式代表一个最佳的实践，那么反模式将代表我们已经学到一个教训。受启发于的设计模式，在年的月的报告大会上首次提出反模式。参考链接反模式学用设计模式极客学院

</>复制代码 
如果我们认为模式代表一个最佳的实践，那么反模式将代表我们已经学到一个教训。受启发于Gof的《设计模式》，Andrew Koeing在1995年的11月的C++报告大会上首次提出反模式。在Koeing的报告中，反模式有着两种观念:
描述对于一个特殊的问题，提出了一个糟糕的解决方案，最终导致一个坏结果发生
描述如何摆脱上述解决方案并能提出一个好的解决方案

在如今这个前端发展如火如荼的时代，谈及jq总是显得非常的low，但实际上，在学校，在很多前端新人以及所谓“页面仔 || 切图工”之类的同行之间，jq的活力还是远超各种框架的时候，之所以想写这样一篇文章，一是因为见到了身边的jq烂代码，二是因为我在百度jQuery反模式的时候居然什么有价值的相关结果都没有，所以觉得还是有必要聊聊的。

</>复制代码 
// 反模式
$.each(reallyLongArray, function(count, item) {
    var newLi = "
" + item + "
";
    $("#ballers").append(newLi)
})
// 更好的实践
var frag = document.createDocumentFragment()
$.each(reallyLongArray, function(count, item) {
    var newLi = "
" + item + "
";
    frag.appendChild(newLi[0])
})
$("#ballers")[0].appendChild(frag)
// 你也可以用字符串
var myHTML = ""
$.each(reallyLongArray, function(count, item) {
    myHTML += "
" + item + "
";
})
$("#ballers").html(myHTML)

DocumentFragment是浏览器为了减少DOM操作中的更新所使用的API，详情请查阅MDN相关文档。

</>复制代码 
if ($a.data("current") != "showing") {
    $a.stop()
}
if ($b.data("current") != "showing") {
    $b.stop()
}
if ($c.data("current") != "showing") {
    $c.stop()
}
// 用数组来保存不同的主体
var elems = [$a, $b, $c]
$.each(elems, function(k, v) {
    if (v.data("current") != "showing") {
        v.stop()
    }
})

用数组或对象来保存重复片段的差异参数是一种很常见的方法。更多内容可以参考常见的JavaScript设计模式中的“九、策略模式”

</>复制代码 
$(document).ready(function() {
  $("#button").click(function() {
    $.get("http://api.github.com/repos/facebook/react/forks", function(data) {
      alert(data[0].owner.login)
    })
  })
})
// 以前有这么一种优化的方法，使用对象字面量保存回调使其扁平化
var cbContainer = {
  initApp: function() {
    $(document).ready(cbContainer.readCb)
  },
  readyCb: function() {
    $("#button").click(cbContainer.clickCb)
  },
  clickCb: function() {
    $.get("http://api.github.com/repos/facebook/react/forks", function(data) {
      cbContainer.getCb(data)
    })
  },
  getCb: function(data) {
    alert(data[0].owner.login)
  }
}
cbContainer.initApp()
// 不过现在流行Promise
var initApp = function() {
  return new Promise(function(resolve, reject) {
      $(document).ready(resolve)
  })
}
var readyCb = function() {
    return new Promise(function(resolve, reject) {
        $("#button").click(resolve)
    })
}
var clickCb = function() {
    return new Promise(function(resolve, reject) {
        $.get("http://api.github.com/repos/facebook/react/forks", function(data) {
      resolve(data)
    })
    })
}
var getCb = function(data) {
    alert(data[0].owner.login)
}
initApp()
  .then(readyCb)
  .then(clickCb)
  .then(getCb)

用对象将回调扁平还好，Promise是什么鬼。不是比回调还恶心，好吧，示例确实是这样。其实之所以用Promise除了将回调转成链式调用以外，主要还是为了用它的reject函数获取回调中的错误。像示例这种一路resolve的，没必要这么用。这里只是提一句。

如果希望了解更多关于回调相关的知识,可以看看Promise, generator, async與ES6这篇文章。

</>复制代码 
$(document.body).append("
")
$(".baaron").click(function() {})
// 更好的方式
$("
")
    .appendTo(document.body)
    .click(function() {})

对于jq的选择器还有许多要注意的问题，因为jq的选择器是从右向左查询，所以请记住一个“左轻右重”的原则：

</>复制代码 
// 请看下面两个选择器
$("div.foo .bar")
$(".foo span.bar")        //右边更明确一点，会好不少
// 当左边确实要比右边明确的时候这么干
$("#foo .bar")
$("#foo").find(".bar")
// 尤其避免使用通配符
$("#foo > *")
$("#foo").children()
// 有些通配符是隐式的
$(".foo :radio")
$(".foo *:radio")        //和上边一样的
$(".foo input:radio")    //改成这样

接下来，让我们从优化一段jq代码开始，聊聊js中的依赖

</>复制代码 
$("#button").click(function() {
    $.get("http://xxxx", function(data) {
        $("#page").html(data.abc)
    })
})

这段代码有以下问题：

click事件绑定的匿名函数难以重复利用，也很难测试

click回调的匿名函数中的$是全局变量，ajax请求回调的匿名函数中的$("#page")也是用到了$这一全局变量，全局变量应该是要避免的

回调的问题前面也说过了，这里的回调还很清楚不至于说到地狱的程度

现在我们把代码这样改写：

</>复制代码 
var downJSON = function() {
    $.get("http://xxxx", function(data) {
        $("#page").html(data.abc)
    })
}
$("#button").click(downJSON)

现在匿名函数被我们拿出来了，可以重用了，但还是难以测试，且涵盖全局变量。

继续：

</>复制代码 
var downJSON = function($, $el) {
    $.get("http://xxxx", function(data) {
        $el.html(data.abc)
    })
}
$("#button").click(function() {
    downJSON($, $("#page"))
})

这样改写以后，没有了全局变量，函数已经独立出去。换一种说法就是，我们去除了函数中的隐式依赖（前面例子中的函数要运行需要全局变量$，但没有从函数声明中表现出来，我们称其为隐式依赖），现在，函数执行所需要的依赖被显示声明，使其具有更好的可控性。前端的依赖管理如今是一个很流行的话题，不过在这里就不废话了。

最后，对于几种比较常见的写法，我们也可以使用一些奇技淫巧，或能使代码更短，或能使代码更为易读：

</>复制代码 
// 常见的写法
if(!data) {
    data = {}
}
// 简化
data = data || {}

你可能觉得这不值一提，但可能有些时候你写着写着就忽视了。比如，js数组去重的4个方法中的第二个方法，就可以应用这个技巧：

</>复制代码 
// 原来的代码
Array.prototype.unique2 = function()
{
    var hashTable = {},res=[];                //n为hash表，r为临时数组
    for(var i = 0; i < this.length; i++) {    //遍历当前数组
        if (!hashTable[this[i]]) {            //如果hash表中没有当前项
            res.push(this[i]);                //把当前数组的当前项push到临时数组里面
            hashTable[this[i]] = true;        //存入hash表
        }
    }
    return res;
}
// 应用此技巧
Array.prototype.unique2 = function()
{
    var hashTable = {}, res = []
    for(var i = 0; i < this.length; i++) {
        !hashTable[this[i]] ? res.push(this[i]) : null
        hashTable[this[i]] = hashTable[this[i]] || true
    }
    return res
}

写成这样也未必说是优化，目测判断逻辑还多了一个哈哈，但是嵌套少了一层，怎么说呢，自行决定吧。

下面展示的一个技巧和上面这个也差不多：

</>复制代码 
// 正常写法
if(type === "foo" || type === "bar") {}
// 用对象有三种方法
// 方法1
if(({foo: 1, bar: 1})[type]) {}                    // type === "toString"可以绕过验证
// 方法2
if(type in ({foo: 1, bar: 1})) {}                // 和上一种等价但是慢点
// 方法3
if(({foo: 1, bar: 1}).hasOwnProperty(type)) {}    // 最严密，也最慢
// 用正则
if(/^(foo|bar)$/.test(type)) {}

这种技巧的话，使用与否同样是自己决定。很多有意思的东西，都是自己想着玩的，有些情况，我们倒不如好好写成switch - case，都看的懂，也挺清晰。

总结两句，虽说jQuery库和新式的框架相比老了，但我觉得它在DOM操作上真的做到了一个极致。我相信很长一段时间，前端开发人员入门，还是要从它开始的。个人认为jq不适应时代的原因，是因为它本身也仅仅限于DOM操作，没有其他限制，以至于当应用复杂时，你完全控制不住你的页面。当你用上流行的框架，按照他们的Best Practice去组织代码，我想你刚刚开始的时候，一定会怀念jQuery这个溺爱你的老朋友的。

反模式 - 学用 JavaScript 设计模式 - 极客学院
jQuery Anti-Patterns for Performance & Compression
Patterns of Large-Scale JavaScript Applications