被 lodash 的一个方法给坑了

前言

大家好,我是林三心,用最通俗易懂的话讲最难的知识点是我的座右铭,基础是进阶的前提是我的初心~

背景

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

接着需要判断这个对象是否为空,为空的话就去执行另一个很重要的逻辑,这里判断对象为空,我们选择了lodashisEmpty方法

出问题了

然后就出事了,上线后发现有 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

结语 & 加学习群 & 摸鱼群

我是林三心

  • 一个待过小型toG型外包公司、大型外包公司、小公司、潜力型创业公司、大公司的作死型前端选手;
  • 一个偏前端的全干工程师;
  • 一个不正经的掘金作者;
  • 一个逗比的B站up主;
  • 一个不帅的小红书博主;
  • 一个喜欢打铁的篮球菜鸟;
  • 一个喜欢历史的乏味少年;
  • 一个喜欢rap的五音不全弱鸡

如果你想一起学习前端,一起摸鱼,一起研究简历优化,一起研究面试进步,一起交流历史音乐篮球rap,可以来俺的摸鱼学习群哈哈,点这个,有7000多名前端小伙伴在等着一起学习哦 --> 摸鱼沸点

相关推荐
anOnion5 小时前
构建无障碍组件之Carousel Pattern
前端·html·交互设计
ssshooter5 小时前
Tauri 2 iOS 开发避坑指南:文件保存、Dialog 和 Documents 目录的那些坑
前端·后端·ios
Можно6 小时前
深入理解 ES6 Proxy:与 Object.defineProperty 的全面对比
前端·javascript·vue.js
Birdy_x6 小时前
接口自动化项目实战(1):requests请求封装
开发语言·前端·python
天天向上10248 小时前
vue el-table实现拖拽排序
前端·javascript·vue.js
柳杉9 小时前
Three.js × Blender:从建模到 Web 3D 的完整工作流深度解析
前端·javascript·数据可视化
reembarkation10 小时前
vue3中使用howler播放音频列表
前端·vue.js·音视频
手握风云-10 小时前
基于 Java 的网页聊天室(三)
服务器·前端·数据库
weixin1997010801610 小时前
《识货商品详情页前端性能优化实战》
前端·性能优化
Forever7_10 小时前
重磅!Vue3 手势工具正式发布!免费使用!
前端·前端框架·前端工程化