被Lodash的方法骗了好几年,今天终于踩了大坑了!

2024年 1月 24日 56.4k 0

背景

我们项目中有一处业务代码,需要根据不同的条件,对某个对象进行属性的删除,大概代码如下:

接着需要判断这个对象是否为空,为空的话就去执行另一个很重要的逻辑,这里判断对象为空,我们选择了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:

相关文章

JavaScript2024新功能:Object.groupBy、正则表达式v标志
PHP trim 函数对多字节字符的使用和限制
新函数 json_validate() 、randomizer 类扩展…20 个PHP 8.3 新特性全面解析
使用HTMX为WordPress增效:如何在不使用复杂框架的情况下增强平台功能
为React 19做准备:WordPress 6.6用户指南
如何删除WordPress中的所有评论

发布评论