1st-code-review summary
每次做code review,先贤谆谆教诲便在耳畔响起:
“There are only two hard problems in Computer Science: cache invalidation and naming things.” —Phil Karlton
将发现的问题整理如下:
Hard Code
这个永远是容易发现,也是最容易出现的问题,主要集中在 :
- 配置项
this.investmentList = [{
id: "ALL",
name: "All Investments"
}, {
id: "FE",
name: "ETF"
}, {
id: "FV",
name: "Insurance Product Fund"
}, {
id: "FO",
name: "Open-End Fund"
}, {
id: "SA",
name: "Separate Account"
}];
- Labels
if (data.length == 0) {
//message hard code
this.showMsgBox("No valid manager data available for the strategy.", true);
this.hideBodyEl();
}
- 逻辑标识
if (dataType == 1) { // dataType == what
//compare the number type
}
单var模式与多var模式
单var模式
在函数的顶部使用唯一一个var语句是非常推荐的一种模式,它有如下一些好处:
- 可以在同一个位置找到函数所需的所有变量
- 避免在变量声明之前使用这个变量时产生的逻辑错误(参考下一小节“声明提前:分散的var带来的问题”)
- 提醒你不要忘记声明变量,顺便减少潜在的全局变量
- 代码量更少(输入代码更少且更易做代码优化)
单var模式看起来像这样:
function func() {
var a = 1,
b = 2,
sum = a + b,
myobject = {},
i,
j;
// 函数体…
}
对于单 var 模式和多 var 模式,建议不做强行约定,但同一个文件里,风格必须一致。
命名规范
- 引入外部资源
- 应对引入的资源分组并以空行隔开,如mercury资源与mr资源,chartComponent资源与Grid资源,类与object...
- 类名大驼峰,变量小驼峰,常量全大写
- 不用给一些常用工具类起不同于exports的别名,如系统中是这样配置jquery,underscorejs
'jquery': {
deps: [],
exports: '$',
init: function() {
return this.$.noConflict();
}
},
'underscore': {
deps: [],
exports: '_',
init: function () {
return this._.noConflict();
}
}
- 方法名
- 方法名拼写错误,如restFilterOption..
- 太多的方法名以draw,load开头,有些名字不是很顾名思义
方法参数不需要带下划线
这一点我还没查到具体文档,待讨论
renderUI: function(_el) { //_el => el
_el.append(HTML);
this.initParam();
this._loadRequestParam();
this.drawHeader();
this._tableTemplate = null;
this._tableData = null;
}
多用$el.empty().append(value),少用$el.html(value)
$().html(value) vs $().empty().append(value)比较
refs :
single-var-mode
html-vs-empty-append
requireJs-config
Airbnb JavaScript Style Guide