背景
我们项目中有一处业务代码,需要根据不同的条件,对某个对象进行属性的删除,大概代码如下:
接着需要判断这个对象是否为空,为空的话就去执行另一个很重要的逻辑,这里判断对象为空,我们选择了lodash的isEmpty方法:
出问题了
然后就出事了,上线后发现有 BUG ,定位问题之后,发现就是明明对象不为空,然后isEmpty还是把这个对象判为空了,然后去执行了接下来的逻辑,导致出现 BUG
然后我就去看了一下 isEmpty的源码,总算是发现问题了!
可以看到最后一个环节,isEmpty 在判断对象是否为空的时候,用了for in + hasOwnProperty去判断某一个属性是否存在对象中,只要有一个存在,那么这个对象就不为空!
诶!那好像也没啥问题啊?但是我突然想到,那这个for in能遍历出 Symbol 类型的属性吗?于是我试了一下,发现 for in 并不会遍历 Symbol属性。
这也严重了问题出在lodash 的 isEmpty上,并且根本原因是 for in 并不会遍历 Symbol属性。
解决问题
所以还是自己实现一个来解决这个问题吧!!!我们可以获取到对象的属性个数,判断属性个数是否为0,为0那就是为空。
那可以用Object.keys吗?他的作用就是可以把对象的属性放到一个数组中,我们试试之后,发现Object.keys并不会把Symbol属性算进去:
我们换一下Reflect.ownKeys试试,发现就完全可以了!!!!
所以最终自己实现了一个 isEmpty: