不是标题党,这是一位leader在一次代码评审会对小组成员发出的怒吼,原因是在代码评审中发现很多地方还是采用ES5的写法,也不是说用ES5写法不行,会有BUG,只是造成代码量增多,可读性变差而已。 文章转载:乐字节 恰好,这位leader有代码洁癖,面对35年经验的成员,还写这种水平的代码,极为不满,不断对代码进行吐槽。不过对于他的吐槽,我感觉还是有很大收获的,故就把leader的吐槽记录下来,分享给掘友们,觉得有收获点个赞,有错误的或者更好的写法,非常欢迎在评论中留言。 ps:ES5之后的JS语法统称ES6!!!一、关于取值的吐槽 取值在程序中非常常见,比如从对象obj中取值。constobj{a:1,b:2,c:3,d:4,e:5,} 吐槽:constaobj。a;constbobj。b;constcobj。c;constdobj。d;consteobj。e; 或者constfobj。aobj。d;constgobj。cobj。e; 吐槽:不会用ES6的解构赋值来取值吗?5行代码用1行代码搞定不香吗?直接用对象名加属性名去取值,要是对象名短还好,很长呢?搞得代码中到处都是这个对象名。 改进:const{a,b,c,d,e}obj;constfad;constgce; 反驳 不是不用ES6的解构赋值,而是服务端返回的数据对象中的属性名不是我想要的,这样取值,不是还得重新创建个遍历赋值。 吐槽 看来你对ES6的解构赋值掌握的还是不够彻底。如果想创建的变量名和对象的属性名不一致,可以这么写:const{a:a1}obj;console。log(a1);1 补充 ES6的解构赋值虽然好用。但是要注意解构的对象不能为undefined、null。否则会报错,故要给被解构的对象一个默认值。const{a,b,c,d,e}obj{};二、关于合并数据的吐槽 比如合并两个数组,合并两个对象。consta〔1,2,3〕;constb〔1,5,6〕;constca。concat(b);〔1,2,3,1,5,6〕constobj1{a:1,}constobj1{b:1,}constobjObject。assgin({},obj1,obj2);{a:1,b:1} 吐槽 ES6的扩展运算符是不是忘记了,还有数组的合并不考虑去重吗? 改进consta〔1,2,3〕;constb〔1,5,6〕;constc〔。。。newSet(〔。。。a,。。。b〕)〕;〔1,2,3,5,6〕constobj1{a:1,}constobj2{b:1,}constobj{。。。obj1,。。。obj2};{a:1,b:1}三、关于拼接字符串的吐槽constname小明;constscore59;constresult;if(score60){result{name}的考试成绩及格;}else{result{name}的考试成绩不及格;} 吐槽 像你们这样用ES6字符串模板,还不如不用,你们根本不清楚在{}中可以做什么操作。在{}中可以放入任意的JavaScript表达式,可以进行运算,以及引用对象属性。 改进constname小明;constscore59;constresult{name}{score60?的考试成绩及格:的考试成绩不及格};四、关于if中判断条件的吐槽if(type1type2type3type4){。。。} 吐槽 ES6中数组实例方法includes会不会使用呢? 改进constcondition〔1,2,3,4〕;if(condition。includes(type)){。。。}五、关于列表搜索的吐槽 在项目中,一些没分页的列表的搜索功能由前端来实现,搜索一般分为精确搜索和模糊搜索。搜索也要叫过滤,一般用filter来实现。consta〔1,2,3,4,5〕;constresulta。filter(item{returnitem3}) 吐槽 如果是精确搜索不会用ES6中的find吗?性能优化懂么,find方法中找到符合条件的项,就不会继续遍历数组。 改进consta〔1,2,3,4,5〕;constresulta。find(item{returnitem3})六、关于扁平化数组的吐槽 一个部门JSON数据中,属性名是部门id,属性值是个部门成员id数组集合,现在要把有部门的成员id都提取到一个数组集合中。constdeps{采购部:〔1,2,3〕,人事部:〔5,8,12〕,行政部:〔5,14,79〕,运输部:〔3,64,105〕,}letmember〔〕;for(letitemindeps){constvaluedeps〔item〕;if(Array。isArray(value)){member〔。。。member,。。。value〕}}member〔。。。newSet(member)〕 吐槽 获取对象的全部属性值还要遍历吗?Object。values忘记了吗?还有涉及到数组的扁平化处理,为啥不用ES6提供的flat方法呢,还好这次的数组的深度最多只到2维,还要是遇到4维、5维深度的数组,是不是得循环嵌套循环来扁平化? 改进constdeps{采购部:〔1,2,3〕,人事部:〔5,8,12〕,行政部:〔5,14,79〕,运输部:〔3,64,105〕,}letmemberObject。values(deps)。flat(Infinity); 其中使用Infinity作为flat的参数,使得无需知道被扁平化的数组的维度。 补充 flat方法不支持IE浏览器。七、关于获取对象属性值的吐槽constnameobjobj。name; 吐槽 ES6中的可选链操作符会使用么? 改进constnameobj?。name;八、关于添加对象属性的吐槽 当给对象添加属性时,如果属性名是动态变化的,该怎么处理。letobj{};letindex1;letkeytopic{index};obj〔key〕话题内容; 吐槽 为何要额外创建一个变量。不知道ES6中的对象属性名是可以用表达式吗? 改进letobj{};letindex1;obj〔topic{index}〕话题内容;九、关于输入框非空的判断 在处理输入框相关业务时,往往会判断输入框未输入值的场景。if(value!nullvalue!undefinedvalue!){。。。} 吐槽 ES6中新出的空值合并运算符了解过吗,要写那么多条件吗?if(value??!){。。。}十、关于异步函数的吐槽 异步函数很常见,经常是用Promise来实现。constfn1(){returnnewPromise((resolve,reject){setTimeout((){resolve(1);},300);});}constfn2(){returnnewPromise((resolve,reject){setTimeout((){resolve(2);},600);});}constfn(){fn1()。then(res1{console。log(res1);1fn2()。then(res2{console。log(res2)})})} 吐槽 如果这样调用异步函数,不怕形成地狱回调啊! 改进constfnasync(){constres1awaitfn1();constres2awaitfn2();console。log(res1);1console。log(res2);2} 补充 但是要做并发请求时,还是要用到Promise。all()。constfn(){Promise。all(〔fn1(),fn2()〕)。then(res{console。log(res);〔1,2〕})} 如果并发请求时,只要其中一个异步函数处理完成,就返回结果,要用到Promise。race()。十一、后续 欢迎来对以上十点leader的吐槽进行反驳,你的反驳如果有道理的,下次代码评审会上,我替你反驳。 此外以上的整理内容有误的地方,欢迎在评论中指正,万分感谢。 如果你还有其它想吐槽的,也非常欢迎在评论中留下你的吐槽。 读完有帮助,不妨关注一下,点个赞支持一下。 文章转载:乐字节 Java编程交流裙:785794074暗号:75