是标题党，这是一位leader在一次代码评审会对小组成员发出的“怒吼”，原因是在代码评审中发现很多地方还是采用ES5的写法，也不是说用ES5写法不行，会有BUG，只是造成代码量增多，可读性变差而已。
恰好，这位leader有代码洁癖，面对3~5年经验的成员，还写这种水平的代码，极为不满，不断对代码进行吐槽。不过对于他的吐槽，我感觉还是有很大收获的，故就把leader的吐槽记录下来，分享给大家，如果觉得有收获点个赞，有错误的或者更好的写法，非常欢迎在评论中留言。
ps：ES5之后的JS语法统称ES6！！！
01、关于取值的吐槽
取值在程序中非常常见，比如从对象obj中取值。
constobj={a:1,b:2,c:3,d:4,e:5,}吐槽：
consta=obj.a;constb=obj.b;constc=obj.c;constd=obj.d;conste=obj.e;或者
constf=obj.a+obj.d;constg=obj.c+obj.e;吐槽：“不会用ES6的解构赋值来取值吗？5行代码用1行代码搞定不香吗？直接用对象名加属性名去取值，要是对象名短还好，很长呢？搞得代码中到处都是这个对象名。”
改进：
const{a,b,c,d,e}=obj;constf=a+d;constg=c+e;反驳
不是不用ES6的解构赋值，而是服务端返回的数据对象中的属性名不是我想要的，这样取值，不是还得重新创建个遍历赋值。
吐槽
看来你对ES6的解构赋值掌握的还是不够彻底。如果想创建的变量名和对象的属性名不一致，可以这么写：
const{a:a1}=obj;console.log(a1);//1补充
ES6的解构赋值虽然好用。但是要注意解构的对象不能为undefined、null。否则会报错，故要给被解构的对象一个默认值。
const{a,b,c,d,e}=obj||{};02、关于合并数据的吐槽
比如合并两个数组，合并两个对象。
consta=[1,2,3];constb=[1,5,6];constc=a.concat(b);//[1,2,3,1,5,6]constobj1={a:1,}constobj2={b:1,}constobj=Object.assign({},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}03、关于拼接字符串的吐槽
constname='小明';constscore=59;letresult='';if(score>60){result=`${name}的考试成绩及格`;}else{result=`${name}的考试成绩不及格`;}吐槽
像你们这样用ES6字符串模板，还不如不用，你们根本不清楚在中可以做什么操作。在{}中可以放入任意的JavaScript表达式，可以进行运算，以及引用对象属性。
改进
constname='小明';constscore=59;constresult=`${name}${score>60?'的考试成绩及格':'的考试成绩不及格'}`;04、关于if中判断条件的吐槽
if(type==1||type==2||type==3||type==4||){//...}吐槽
ES6中数组实例方法includes会不会使用呢？
改进
constcondition=[1,2,3,4];if(condition.includes(type)){//...}05、关于列表搜索的吐槽
在项目中，一些没分页的列表的搜索功能由前端来实现，搜索一般分为精确搜索和模糊搜索。搜索也要叫过滤，一般用filter来实现。
consta=[1,2,3,4,5];constresult=a.filter(item=>{returnitem===3})吐槽
如果是精确搜索不会用ES6中的find吗？性能优化懂么，find方法中找到符合条件的项，就不会继续遍历数组。
改进
consta=[1,2,3,4,5];constresult=a.find(item=>{returnitem===3})06、关于扁平化数组的吐槽
一个部门JSON数据中，属性名是部门id，属性值是个部门成员id数组集合，现在要把有部门的成员id都提取到一个数组集合中。
constdeps={'采购部':[1,2,3],'人事部':[5,8,12],'行政部':[5,14,79],'运输部':[3,64,105],}letmember=[];for(letitemindeps){constvalue=deps[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],}letmember=Object.values(deps).flat(Infinity);其中使用Infinity作为flat的参数，使得无需知道被扁平化的数组的维度。
补充
flat方法不支持IE浏览器。
07、关于获取对象属性值的吐槽
constname=obj&&obj.name;吐槽
ES6中的可选链操作符会使用么？
改进
constname=obj?.name;08、关于添加对象属性的吐槽
当给对象添加属性时，如果属性名是动态变化的，该怎么处理。
letobj={};letindex=1;letkey=`topic${index}`;obj[key]='话题内容';吐槽
为何要额外创建一个变量。不知道ES6中的对象属性名是可以用表达式吗？
改进
letobj={};letindex=1;obj[`topic${index}`]='话题内容';09、关于输入框非空的判断
在处理输入框相关业务时，往往会判断输入框未输入值的场景。
if(value!==null&&value!==undefined&&value!==''){//...}吐槽
ES6中新出的空值合并运算符了解过吗，要写那么多条件吗？
if((value??'')!==''){//...}10、关于异步函数的吐槽
异步函数很常见，经常是用 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)})})}吐槽
如果这样调用异步函数，不怕形成地狱回调啊！
改进
constfn=async()=>{constres1=awaitfn1();constres2=awaitfn2();console.log(res1);//1console.log(res2);//2}补充
但是要做并发请求时，还是要用到Promise.all()。
constfn=()=>{Promise.all([fn1(),fn2()]).then(res=>{console.log(res);//[1,2]})}如果并发请求时，只要其中一个异步函数处理完成，就返回结果，要用到Promise.race()。
欢迎来对以上十点leader的吐槽进行反驳。