可测试的javascript代码
首先,作为一个正在Javascript中挣扎的人来说,写这些东西着实很紧张,毕竟,个人资质平平,不知道这些东西对大家有没有帮助,但从我个人角度来看,觉得非常重要,对于规范化自己的js代码,增强它的持久性和可维护性,实在是非常有帮助。
为什么提到‘圈复杂度’这个概念,是因为在我们编写js代码的过程中,随着功能的扩展和增加,代码复杂度也在上涨,为了减少复杂性,我们必须确保我们不会使用任何过于复杂或容易出错的构造,保证代码的合理性和可读性,你不能保证几个月之后你是否还会记得你当时写这段代码的意图,所以,重要性不言而喻,而常用的如 'Lintjs' 等工具就可以帮我们很好的规范化我们的代码,避免代码出现难以维护的场景,而圈复杂度这个概念被定义为 “表示代码中独立现行路径的数量”,什么意思,就是说这个复杂度的值代表着 测试当前代码,所需要编写的单元测试的最小数量。
例如:
function sum(a,b){
if(typedef(a) !== typedef(b){
throw new Error("Cann't sum different types!");
} else{
return a + b;
}
}
该方法的圈复杂度是2,就意味着,要测试每个分之,并获得百分之百的代码覆盖率,要编写两个单元测试,虽然上述代码中部分写法不是必须的,但是规范化,注意细节,保持好会让代码在后续迭代维护中避免很多问题。
一般来说,任何方法的圈复杂度都不应该大于10,这个值不是臆想出来的,而是规律总结出的一个合理数值,一旦当值超过25,就意味着这段代码非常复杂,换句话说,你一旦修改,无论修改前有无BUG,修改产生新的Bug几乎是不可避免的。
那么,又是什么引起的呢?一般来说,圈复杂度通常是由很多if/then/else 语句造成的,最简单的重构修复,是将方法分解成更小的方法。
例如:
function doSomething(a){
if(a==='x'){
doX();
}else if(a==='y'){
doY();
}else{
doZ();
}
}
重构之后
function doSomething(a){
var lookup={x:doX:y:doY},def=doZ;
lookup[a] ? lookup[a]() : def();
}
重构之后,并没有减少必须的单元测试量,而是分解成更小的量,这样化难为简,不用再为一个单独的函数编写大量的测试代码,函数代码更具有可维护性。
当然,我们不希望在代码提交后才发现此类问题,所以,我们可以在本地先检测js代码是否符合基本的规范和要求。
在此推荐一个计算圈复杂度的工具----- jscheckstyle,它可以计算每个函数和方法的圈复杂度,也包括其中的匿名函数。
sudo npm install jscheckstyle
![Paste_Image.png](http://upload-images.jianshu.io/upload_images/2088873-be5d9d49b052b4c3.png?imageMogr2/auto-orient/strip%7CimageView2/2/w/1240)
简单的导入一个文件测试样例:
![Paste_Image.png](http://upload-images.jianshu.io/upload_images/2088873-84d287fb692d5542.png?imageMogr2/auto-orient/strip%7CimageView2/2/w/1240)
其中,返回的值分别是 代码所在的 绝对行数(因为期间注释,空行都会计算在内)、函数的参数数量、以及 圈复杂度。这样一个大致的预览会让我们对代码的性能有一个基本的认识,更好的能够保证代码的可维护性,当然,这里所说的只是一个宏观的认识,涉及的东西也只是冰山一角。
如果你也正面临着同样的问题,推荐阅读:
* 《编写可维护的javascript》
* 《编写可测试的javascript》
肯定能让你获益良多,共勉。