A,不易懂的JavaScript代码
1.判断
// 数据类型判断 if(Object.prototype.toString.call(str) === “[object String]”){ // doSomething(); }; // 文件类型判断 if(/.*\.css(?=\?|$)/.test(“/path/to/main.css”)){ // doSomething(); }
2.清空一个队列
var Queue = ["test1","test2","test3"]; //常见方式 Queue.length =0; Queue =[];
3.注册一个变量
// 注册 var repos = {}; repos[“a”] = { name: “a”, content: {} }; repos[“b”] = { name: “b”, content: {} };
上面几个例子倒不至于看不懂,程序都特别简单,第一个例子中,我们通过 Object 原型链上的 toString 方法来判断一个变量是否为 string 类型,以及使用正则来判断一个文件是不是 css 文件。代码写起来比较轻松,倘若我们同时需要判断多个对象是否为多个类型中的一种呢?再比如我们需要在一串代码中提取 require 依赖关系呢,是否应该思考下如何组织自己的代码?
在第二个例子中,将数组的长度设置为 0,或者使用空数组来重置这个变量,都是十分常见的方式,但目前的场景是清空一个队列,我们是否可以使用更加语义化的形式来呈现?比如我们只需要清空该队列的前五个和后三个 item 呢?
第三个例子中,变量的注册,把一堆注册放在一起,上面的形式确实也是一目了然,如果 a b c d 等都是分隔穿插在几百行代码之间呢?突然来个 repos["x"] 这样是否显得有些不太直观。
为了说明本文所倡导的思想,上面的几个解释都有些含糊和牵强,请往下看。
B,他高代码语义性
针对上述三个案例,用更加语义化的方式来呈现代码:
1.语义化变量
// 类型判断 function isType(type){ return function(o){ return Object.prototype.toString.call(o) === ‘[object ‘ + type + ‘]‘; } } var isString = isType(“String”); var isObject = isType("Object"); var isArray = isType("Array"); isString("I‘m Barret Lee."); isArray([1,2,3]); isObject({});
我觉得不需要太多的解释,对比
if(Object.prototype.toString.call(str) === “[object String]”){ // code here... }
显得清新多了吧。
// 提取常量 var isCss = /.*\.css(?=\?|$)/; isCss.test(“/path/to/main.css”);
不管 isCss 这个正则代码有多长,当我们看到 isCss 这个单词就可以顾名思义。很多人写正则都不会将正则单独提取出来使用某个有意义的变量去存储,加注释还好,要是不加注释,后续开发者就只能硬着头皮看懂正则去理解代码的含义。
这样的处理,实际上是增加了代码量,不过从工程角度去审视,有助于提高开发效率以及代码的组织性。
2.语义化行为
var Queue = ["test1", "test2", "test3"]; Queue.splice(0, Queue.length);
上面代码具有很强的语义性,从索引为 0 的地方开始,直到队列最后,删除 Queue 中所有的 item。这种写法的扩展性也更好:
Queue.splice(2,4);//删除从索引为2,往后的4个元素
这只是个小例子,有些行为是需要很多代码组合处理的,如果没有很好的组合同一行为的代码,整个结构就显得十分涣散,不利于阅读。
//注册 var repos=[]; function register(){ repos[o.name] = o; } register({ name:"a", content;{} });
对比我们之前
repos["a"] ={ name:"a", content;{} };
语义化程度是不是有所提高
C,小结
代码的优化,需要考虑的维度很多。但是代码的优化并不是减少代码量,有的时候我们需要增加代码来提高代码的可阅读性。
- 正确标记变量
- 封装某个动作
- 注意函数的写法
- 不容易理解的东西,加注释
本文为抛砖引玉,希望可以触发你对代码优化的敏感度的思考,写出一手别人竖拇指的代码~