为规范前端开发代码,提高代码质量,特制定此文档,其中声明,安全和分号这三节是必须执行的,组件类必须遵循注释规范。
- 变量声明必须加var关键字,严格控制作用域;
- 建议使用驼峰式命名变量和函数,如:functionNamesLikeThis, variableNamesLikeThis, ClassNamesLikeThis,namespaceNamesLikeThis;
- 私有成员变量和方法命名以下划线开头,如:var _this;
- 常量定义单词全部大写,以下划线连接,但不要用const关键字来声明,如:SOME_CONSTANTS;
- 函数参数大于3个时,应以对象形式作为参数集传递;
- 禁止在代码块中声明函数,错误的范例:if (true) {function foo() {}};
- 禁止用new来实例化基本类型,错误的范例:var x = new Boolean(false);
- 直接定义数组或对象,而不使用new关键字声明,错误的范例:var a = new Array();var o = new Object();
- 使用单引号来定义字符串;
- 文件名必须全部用小写,文件名分隔符用中划线连接,版本连接符用实心点,合并文件的文件名连接符用下划线,如:passport-core.min.js和reset-1.0_utils-1.0.css;
- 审查用户输入,如:从URL获取参数,使用跳转页面的referer,用于eval或DOM操作的用户数据;
- 禁止通过在iframe使用script进行跨域回调;
- 警惕jquery xss,禁止这样的写法:$(window.location.hash);
- 禁止引用站外资源;
函数表达式(有别于函数声明,如:function x(){})必须用分号结束,下面是错误的范例,第三行漏掉分号,后面的函数被当作参数传递给myMethod,导致解析错误:
var myMethod = function() {
return 42;
}
(function() {
// 一个匿名函数,在这里会被错误解析当作参数调用导致报错
})();
由于无法完美的控制都不出现异常,所以尽量在可能但不确定出现异常的地方(大量运算,AJAX请求,数组操作或DOM操作等)用try-catch(e)来抛出异常,这样有利于规模较大的项目中排查错误。使用自定义异常抛出错误信息,更有利于错误信息阅读,且更加通用,因为不同浏览器抛出的信息不一样,有时候可能很难定位到问题所在位置,特别是IE下可能遇到的如下报错信息:
Stack overflow at line: 0
或:
未知的运行时错误
为获取最大化的可移植性和兼容性,代码中应使用标准中支持的方式来书写代码,虽然浏览器支持某些语法但不在规范中,你无法保证你将来不会遇到兼容其他浏览器的情况,如果这个浏览器完全或部分特性只按规范实现。
//错误的范例
var char = 'hello world'[3];
var myForm = document.myForm;
//正确的写法
var char = 'hello world'.charAt(3)
var myForm = document.forms[0];或者var myForm = document.getElementsByTagName('form')[0];
- 类中的成员变量使用构造函数来初始化;
- 除非是必须移除类的成员,否则析构函数中对成员的销毁应通过将其设置为null,而不是用delete,因为重新赋值方式性能比用delete好;
- 避免通过prototype方式污染内置对象原型链;
-
不使用with关键字,容易造成作用域混乱;
-
this仅用于类成员函数或对象中;
-
通用全局函数,特别是通用组件代码应将业务逻辑放入闭包中,并通过“命名空间”将其引入;
-
若函数中使用到全局变量,则访问全局变量时应使用window来引入,如:
var somevar = 10; function getvar(){ var num = window.somevar + 1; //... return num; }
- 尽量使用优雅的模版写法,避免多行字符串用\加换行的方式或使用+运算连接字符串这种难维护的编码方式;
- 禁止使用IE条件编译@cc_on;
没有强制要求代码需格式化成什么样子,但尽量使代码具有较高的可读性。下面给出一些建议的代码风格:
// 花括号起始位置与语句开始在同一行
if (something) {
// ...
} else {
// ...
}
// 数组和对象书写
var arr = [1, 2, 3]; // [之后,]之前没有空格
var obj = {a: 1, b: 2, c: 3}; // {之后,}之前没有空格
// 多行数组和对象书写
var a = [
'huangrongrong',
'chenying',
'small grey grey',
'shenshunlin'
];
var pos = {
top: 10,
right: 20,
bottom: 15,
left: 12
};
// 将业务逻辑相关的代码写一起,不相关的以空行分隔
doSomethingTo(x);
doSomethingElseTo(x);
andThen(x);
nowDoSomethingWith(y);
andNowWith(z);
var x = a ? b : c; // 一行能写完
// 写不完可以换行加缩进
var y = a ?
longButSimpleOperandB : longButSimpleOperandC;
// 或者这样
var z = a ?
moreComplicatedB :
moreComplicatedC;
对于代码中需要进行==逻辑判断的变量,建议进行强制类型转换或使用===代替。
下列值在布尔表达式中结果为false:
- null
- undefined
- '' //空字符串
- 0 //数字
而下面的为true:
- '0' //字符串
- [] //空数组
- {} //空对象
还有一些难以区分的表达式,以下表达式结果全为true:
Boolean('0') == true
'0' != true
0 != null
0 == []
0 == false
Boolean(null) == false
null != true
null != false
Boolean(undefined) == false
undefined != true
undefined != false
Boolean([]) == true
[] != true
[] == false
Boolean({}) == true
{} != true
{} != false
文档注释遵循YUIDoc规范(http://yui.github.io/yuidoc/syntax/),所有的文件、类、方法和属性都应该用合适的标记和类型进行注释。
- YUIDoc要求文档中至少要有一个class和constructor标记来定义类和构造函数;
- YUIDoc中注释块必须以/**(至少两个星号)开头;
- 方法、参数和返回值等必须有注释说明;
- 单行注释使用//;
- 对于代码中特殊用途的变量、存在临界值、函数中使用的hack、使用了某种算法或思路等需要进行注释描述;
- 建议加入开发者个人信息及最后修改时间注释,虽然这些信息不会被YUIDoc解析显示;
下面给出代码注释实例作为参考,更多标记和类型定义请参考YUIDoc官方文档。
/**
这是一个UELib类
@class UELib
@constructor
@author hiwanz <princeb4d@gmail.com>
@modified someone
@date 2013-06-25
**/
function UELib = function(){};
/**
My method description. Like other pieces of your comment blocks,
this can span multiple lines.
@method css
@param prop {String} set/get a css property
@param [value] {String} A value you want to set to a css property
@example
ele.css('width', '100%');
@return {Boolean} Returns true on success
**/
UELib.prototype.css = function(prop,value){
};
待定
目前规范还无法完全自动化审查,团队根据实际情况,在不与规范冲突的前提下,使用JSHint进行审查,配置文件如下:
{
"jquery": true,//检查预定义的全局变量,防止出现$未定义,该项根据实际代码修改
"bitwise": false,//不检查位运算
"browser": true,//通过浏览器内置的全局变量检测
"devel":true,//允许对调试用的alert和console.log的调用
"camelcase": false,//不强制验证驼峰式命名
"curly": true,//强制使用花括号
"eqeqeq": false,//不强制使用===比较运算符
"es3":true,//兼容es3规范,针对旧版浏览器编写的代码
"esnext": false, //不使用最新的es6规范
"expr": true,//允许未赋值的函数名表达式,例如console && console.log(1)
"forin":false,//不强制过滤遍历对象继承的属性
"freeze":false,//不限制对内置对象的扩展
"immed": true,//禁止未用括号包含立即执行函数
"indent": false,//不强制缩进
"latedef": true,//禁止先调用后定义
"maxdepth":false,//不限制代码块嵌套层数
"maxparams":false,//不限制函数参数个数
"newcap": false,//不对首字母大写的函数强制使用new
"noarg": false,//不禁止对arguments.caller和arguments.callee的调用
"noempty":false,//不禁止空代码块
"nonew":false,//允许直接new实例化而不赋值给变量
"plusplus":false,//允许++和--运算符使用
"quotmark": "single",//字符串使用单引号
"scripturl": true,//允许javascript伪协议的url
"smarttabs": true,//允许混合tab和空格缩进
"strict": false,//不强制使用es5严格模式
"sub": true,//允许用[]形式访问对象属性
"undef": true,//禁止明确未定义的变量调用,如果你的变量(myvar)是在其他文件中定义的,可以使用/*global myvar */绕过检测
"unused": false,//允许定义没用的变量,在某些函数回调中,经常出现多个参数,但不一定会用
"multistr": false,//禁止多行字符串,改用加号连接
"globals": {
"jQuery": true
}
}