别了，JavaScript中的isXX系列

我们很容易被漂亮的代码吸引，也不知不觉的在自己的代码库中加入这些。却没有冷静的想过它们的优劣。这不，我就收集了一系列形如 “是否为……？” 的判断的boolean函数。

 

isNull: function(a){
	return a === null;
},
isUndefined: function(a){
	return a === undefined;
},
isNumber: function(a){
	return typeof a === 'number';
},
isString: function(a){
	return typeof a === 'string';
},
isBoolean: function(a){
	return typeof a === 'boolean';
},
isPrimitive: function(b){
	var a = typeof b;
	return !!(b === undefined || b === null || a == 'boolean' || a == 'number' || a == 'string');		
},
isArray: function(a){
	return proto_obj.toString.call(a) === '[object Array]';
},
isFunction: function(a){
	return proto_obj.toString.call(a) === '[object Function]';
},
isPlainObject: function(o){
	if (!o || o === win || o === doc || o === doc.body) {
		return false;
	}
	return 'isPrototypeOf' in o && proto_obj.toString.call(o) === '[object Object]';		
},
isWindow: function(o){
	return o && typeof o === 'object' && 'setInterval' in o;
},
isEmptyObject: function(o){
    for(var a in o) {
	return false;
    }
    return true;
}

 

以上isXX系列中，isUndefined在类库中用的最多。如判断是否传入了某个参数，判断对象是否拥有某个属性等等。但这个函数是不必存在，我已将其移除。理由如下

 

1，isUndefined 与 使用全等（===）或typeof 多了一层函数调用。很明显多一层函数调用比直接使用原生的运算符效率会低（虽然有些微不足道），但如果isUndefined调用次数很多如上万次还是很明显的。我曾经在邮箱框架中加入了该函数，调用次数有4000多次，从性能分析工具看占用了近1%的时间。仅仅一个判断占1%的调用时间还是很可怕的。当然，邮箱框架内的isUndefined处在多层闭包的顶层，访问其也会占用较多时间。如果这一条还不足以让你放弃isUndefined，请看下面几条。

 

2，函数从一定程度上是对一些代码的封装，抽象。是组织良好代码的方式之一，且有利于降低代码的复杂性。但isNull/isUndefined/isBoolean/isNumber/isString函数内仅有一句，抽象层次很低。因此完全不必封装而提取出一个函数。

 

3，isUndefined(a) 与 a === undefined相比并不会节省几个字节（呵，你可以命名的更短但损失了可读性）。

 

 

综上，我去掉了类库中对基本类型判断的isNull/isUndefined/isBoolean/isNumber/isString，需要用到这些判断的时候直接使用typeof运算符等。

 

你呢？

 

 

 

 

 

评论

20 楼 tunpishuang 2011-05-18  

请问Snandy兄，你iteye和cnblogs上面的博客 我到底应该订阅哪一个？或者说着两个博客的区别是？

19 楼 tom33 2011-04-19  

智者见智，仁者见仁。如果对性能要求极端可以去掉，但多数时候是无所谓的。

18 楼 runjia1987 2011-04-19  

参考jquery就OK了，封装常用的工具方法

17 楼 useryouyou 2011-04-19  

EldonReturn 写道

除了isIE, isFF之类可以保留，其它都基本可以省掉

支持！

16 楼 lxs647 2011-04-18  

保留意见吧、、、、

合适的场合用合适的判断语句、、、、

没有必要一竿子打死、、、

15 楼 satanultra 2011-04-18  

这个不会成为性能瓶颈的，觉得封装起来好看易读就用，否则就不用，性能不是问题。

14 楼 clue 2011-04-18  

现在大多数JS都属于UI库，最大的性能消耗不在JS运行速度，而是Dom操作及触发的重渲染（好像还分reflow与repaint？没深究过）
so... 我觉得库里面用这些判断函数造成的消耗无所谓来着，但有时判断不是一个比较就能得出的，就有必要用函数封装了。
如果是纯数据处理什么的，那倒是可以直接写判断以优化性能

13 楼 srdrm 2011-04-18  

应用层使用应该问题不大。

如果是 jQuery 这种框架和库里要判断，也许性能上就要考量了。

12 楼 lobbychmd 2011-04-18  

想起了  !!a

11 楼 davepkxxx 2011-04-18  

我经常把两行封装成一个方法
也会把一句抛异常的封装成一个方法

10 楼 muchao_119 2011-04-18  

MARK一下，留着用~~

9 楼 wushipan_simple 2011-04-18  

如果访问次数非常多的话就不能用这个了，会很卡，小型的网站，没多少人访问的可以为了简单这样做

8 楼 pfans 2011-04-18  

JE帐号 写道

这个可以保留吧...
isPrimitive: function(b){ 
    var a = typeof b; 
    return !!(b === undefined || b === null || a == 'boolean' || a == 'number' || a == 'string');        
},

这个可以保留的，判断是否是JS的基本类型。楼主的意思应该是isUndefined、isNull无需保留。

7 楼 EldonReturn 2011-04-18  

除了isIE, isFF之类可以保留，其它都基本可以省掉

6 楼 JE帐号 2011-04-18  

这个可以保留吧...
isPrimitive: function(b){ 
    var a = typeof b; 
    return !!(b === undefined || b === null || a == 'boolean' || a == 'number' || a == 'string');        
},

5 楼 tiger82 2011-04-17  

作为基础类库的话，最好还是去掉。

4 楼 qjtttt 2011-04-16  

不完全支持，如果在性能要求不是很高的地方我还是会用isXXX，代码的可阅读性提高了许多，因为js是解释型语言所以效率和编译语言比肯定会慢一些，如果在程序中很多地方要判断类似 if(null==str && str.trim().length==0)
我倒是觉得一句 StringUtils.isBlank(str) 看起来更舒服些

3 楼 __游乐场 2011-04-16  

不太支持.单单一层函数调用的开销并没有那么可怕. 如果你已经写了
isArray: function(a){ 
    return proto_obj.toString.call(a) === '[object Array]'; 
}

保持代码风格的统一也是蛮有必要的.

2 楼 rabbit2011 2011-04-16  

我也支持这个观点！

1 楼 wmj007 2011-04-16  

我也支持这种观点，这不光是在JS里面，在所有的程序编程里面，都应该考虑这个问题，不必要的封装只是让代码量增大，并没有达到程序优化的目的