你会用ES6，那倒是用啊！！！

            
不是标题党，这是一位leader在一次代码评审会对小组成员发出的“怒吼”，原因是在代码评审中发现很多地方还是采用ES5的写法，也不是说用ES5写法不行，会有BUG，只是造成代码量增多，可读性变差而已。恰好，这位leader有代码洁癖，面对3~5年经验的成员，还写这种水平的代码，极为不满，不断对代码进行吐槽。不过对于他的吐槽，我感觉还是有很大收获的，故就把leader的吐槽记录下来，分享给掘友们，觉得有收获点个赞，有错误的或者更好的写法，非常欢迎在评论中留言。ps：ES5之后的JS语法统称ES6！！！一、关于取值的吐槽取值在程序中非常常见，比如从对象obj中取值。

1. const obj = {
   
2.     a:1,
   4 x2 Q, z+ n0 y3 f
3.     b:2,
   
4.     c:3,
     f5 N- P0 Q2 @* u/ y1 m- \! b* q
5.     d:4,
   
6.     e:5,
   
7. }

复制代码

+ @; u$ v* U* b吐槽：
& |* i$ z; g9 s! f

1. const a = obj.a;
   % g3 n* P* y  N. c/ y
2. const b = obj.b;
   
3. const c = obj.c;
   
4. const d = obj.d;
   
5. const e = obj.e;

复制代码

5 ?- g5 Q8 c0 N
或者

1. const f = obj.a + obj.d;
   
2. const g = obj.c + obj.e;

复制代码

吐槽：“不会用ES6的解构赋值来取值吗？5行代码用1行代码搞定不香吗？直接用对象名加属性名去取值，要是对象名短还好，很长呢？搞得代码中到处都是这个对象名.
5 |& ?6 F' D5 n  V. M; I”改进：

1. const {a,b,c,d,e} = obj;
   
2. const f = a + d;
   1 u' z- J/ f4 ?+ W# o2 f; h: L' @
3. const g = c + e;

复制代码

; O6 c8 v, {! c1 @9 V% W: t反驳不是不用ES6的解构赋值，而是服务端返回的数据对象中的属性名不是我想要的，这样取值，不是还得重新创建个遍历赋值。吐槽看来你对ES6的解构赋值掌握的还是不够彻底。如果想创建的变量名和对象的属性名不一致，可以这么写：

1. const {a:a1} = obj;
   
2. console.log(a1);// 1补充ES6的解构赋值虽然好用。但是要注意解构的对象不能为undefined、null。否则会报错，故要给被解构的对象一个默认值。
   
3. const {a,b,c,d,e} = obj || {};

复制代码

" b& d+ r# |% Y+ K+ r二、关于合并数据的吐槽比如合并两个数组，合并两个对象。const a = [1,2,3];
8 z$ |8 j- Z7 Y9 @8 N6 g, U

1. const b = [1,5,6];
   
2. const c = a.concat(b);//[1,2,3,1,5,6]
   
3. const obj1 = {
   6 N2 T7 Y1 r7 T# K3 D
4.   a:1,
   8 ]/ s2 s& @/ d
5. }
   $ ~& |$ @; |& k& I( W
6. const obj2 = {
   
7.   b:1,
   
8. }
   
9. const obj = Object.assign({}, obj1, obj2);//{a:1,b:1}吐槽ES6的扩展运算符是不是忘记了，还有数组的合并不考虑去重吗？改进const a = [1,2,3];
   
10. const b = [1,5,6];
    " Q. Y8 A1 O' V& A# J& v
11. const c = [...new Set([...a,...b])];//[1,2,3,5,6]
    % p* {0 S: y0 ~1 I
12. const obj1 = {
    
13.   a:1,
    
14. }
    
15. const obj2 = {
    . S) S7 M4 v0 M, E: K9 j
16.   b:1,
    7 g9 s' v% l- \9 X& K& `
17. }
    ' n! g1 L/ X' T* h' w  A0 D  {
18. const obj = {...obj1,...obj2};//{a:1,b:1}

复制代码

* c1 A: b) N  Q+ f; q; Z三、关于拼接字符串的吐槽
0 k9 M5 A! k* t  r3 T1 A

1. const name = '小明';
   ( m+ e# J! {5 |! r! I
2. const score = 59;
   
3. let result = '';
   9 Z4 Q7 g' L! }6 ~; C* l
4. if(score > 60){
   
5.   result = `${name}的考试成绩及格`;
   
6. }else{
   
7.   result = `${name}的考试成绩不及格`;
   
8. }

复制代码

, x/ i7 O+ f) h0 X
0 L# H' \: S! \$ e% @0 X吐槽像你们这样用ES6字符串模板，还不如不用，你们根本不清楚在${}中可以做什么操作。在${}中可以放入任意的JavaScript表达式，可以进行运算，以及引用对象属性。改进const name = '小明';
1 [1 r1 w3 p7 `5 z$ L

1. const score = 59;
   - y5 o+ x1 e8 Z- `( E8 @+ s
2. const result = `${name}${score > 60?'的考试成绩及格':'的考试成绩不及格'}`;

复制代码

四、关于if中判断条件的吐槽if(
& m" p; S- r5 f8 G4 {

1.     type == 1 ||
   # Z4 D6 s+ l# p: R# W- b" x) k  c' S( M
2.     type == 2 ||
   
3.     type == 3 ||
   
4.     type == 4 ||
   
5. ){
   ( x, [& N% m/ P7 Y6 }' d
6.    //...
   
7. }

复制代码

: v# e# Y% w/ h: T( ~, |吐槽ES6中数组实例方法includes会不会使用呢？
4 x5 a! J6 |' ^改进const condition = [1,2,3,4];

1. if( condition.includes(type) ){
   2 h- Y' X& K% G
2.    //...
   & Y4 h  r6 ?; r5 q. P
3. }

复制代码

9 p$ A3 h( }8 x4 _4 I7 M8 y五、关于列表搜索的吐槽在项目中，一些没分页的列表的搜索功能由前端来实现，搜索一般分为精确搜索和模糊搜索。搜索也要叫过滤，一般用filter来实现。const a = [1,2,3,4,5];

1. const result = a.filter(
   7 w- \8 X+ w8 C. K: u
2.   item =>{
   1 i4 ^$ ?7 ~! Y" L4 G1 N) _9 a1 _7 }
3.     return item === 3
   , g& W2 k- \- m' }* q4 ~% @
4.   }
   
5. )

复制代码

# x% b4 f1 T8 o; N吐槽如果是精确搜索不会用ES6中的find吗？性能优化懂么，find方法中找到符合条件的项，就不会继续遍历数组。改进const a = [1,2,3,4,5];
) p8 v3 b4 b+ D# o) g# a; q

1. const result = a.find(
   
2.   item =>{
     p& e+ W# U7 M% V8 J4 d
3.     return item === 3
   
4.   }
   ) Z- ~0 o; W3 [: V: M/ H
5. )

复制代码

六、关于扁平化数组的吐槽一个部门JSON数据中，属性名是部门id，属性值是个部门成员id数组集合，现在要把有部门的成员id都提取到一个数组集合中。

1. const deps = {
   
2. '采购部':[1,2,3],
   1 F( \1 S& Q3 u, l, ^7 K( t
3. '人事部':[5,8,12],
   
4. '行政部':[5,14,79],
   
5. '运输部':[3,64,105],
   
6. }
   * ~' l9 _% L& \1 w% R" t
7. let member = [];
   
8. for (let item in deps){
   ' {% x+ y3 _+ P. v
9.     const value = deps[item];
   
10.     if(Array.isArray(value)){
    , N) p* @) k5 U5 W9 h3 u' U- m% u
11.         member = [...member,...value]
    , i% U& q8 F0 u% l
12.     }
    6 X( `5 G0 L2 N
13. }
    
14. member = [...new Set(member)]

复制代码

吐槽获取对象的全部属性值还要遍历吗？Object.values忘记了吗？还有涉及到数组的扁平化处理，为啥不用ES6提供的flat方法呢，还好这次的数组的深度最多只到2维，还要是遇到4维、5维深度的数组，是不是得循环嵌套循环来扁平化？
改进

1. const deps = {
   
2.     '采购部':[1,2,3],
   ' ~; z; x3 c+ y5 l/ U- _
3.     '人事部':[5,8,12],
   
4.     '行政部':[5,14,79],
   * y1 h  `( Z2 I% d# C
5.     '运输部':[3,64,105],
   
6. }
   
7. let member = Object.values(deps).flat(Infinity);

复制代码

2 m4 T  d" _7 J4 J# o其中使用Infinity作为flat的参数，使得无需知道被扁平化的数组的维度。补充flat方法不支持IE浏览器。七、关于获取对象属性值的吐槽const name = obj && obj.name;吐槽ES6中的可选链操作符会使用么？改进const name = obj?.name;八、关于添加对象属性的吐槽当给对象添加属性时，如果属性名是动态变化的，该怎么处理。let obj = {};
6 z0 O2 D' k) s! G

1. let index = 1;
   ! H8 R2 Q* B1 q& w# ?8 x3 j4 _
2. let key = `topic${index}`;
   ! L+ e+ a4 I, F) W5 N! d' A
3. obj[key] = '话题内容';吐槽为何要额外创建一个变量。不知道ES6中的对象属性名是可以用表达式吗？改进let obj = {};
   % \6 A7 O) m% _; h1 O) ]2 s
4. let index = 1;
   
5. obj[`topic${index}`] = '话题内容';

复制代码

九、关于输入框非空的判断在处理输入框相关业务时，往往会判断输入框未输入值的场景。if(value !== null && value !== undefined && value !== ''){
% l7 J4 C+ p! P! u, Q    //...
}
( m; m& o( E- |/ P& B9 w吐槽ES6中新出的空值合并运算符了解过吗，要写那么多条件吗？
if((value??'') !== ''){
2 @# y' r& e( Z' S7 A2 m1 L  //...
}
十、关于异步函数的吐槽异步函数很常见，经常是用 Promise 来实现。
- J/ g0 V9 k* \# X; J

1. const fn1 = () =>{
   + o4 n( h' ~! e, h3 v9 R
2.   return new Promise((resolve, reject) => {
   , I2 u" |! K* p  t. Q: t& l) b7 u
3.     setTimeout(() => {
   % G( I$ {. v2 r* Z& ~) H
4.       resolve(1);
   
5.     }, 300);
   4 {+ ]7 T; |; l4 e9 C3 l8 o5 \
6.   });
   
7. }
   
8. const fn2 = () =>{
   
9.   return new Promise((resolve, reject) => {
   , ^( e, ]" L( O* U7 M
10.     setTimeout(() => {
    
11.       resolve(2);
    ; N3 x; _) ^7 x8 C
12.     }, 600);
    4 Q: q& k1 u# D+ I
13.   });
    " T, X$ d5 u) u
14. }
    . a% n6 E+ s9 s2 }
15. const fn = () =>{
    
16.    fn1().then(res1 =>{
    
17.       console.log(res1);// 1
    
18.       fn2().then(res2 =>{
    / }4 t$ x) g; Z" G9 G' }
19.         console.log(res2)
    
20.       })
    5 p7 s6 _  z7 e4 U( G0 g4 z
21.    })
    
22. }

复制代码

吐槽如果这样调用异步函数，不怕形成地狱回调啊！改进

1. const fn = async () =>{
   
2.   const res1 = await fn1();
   * _: w: D5 ]; n; ]% d8 n) v2 _: n# J  ?
3.   const res2 = await fn2();
   # r" ]9 m1 h6 X/ e5 z% l( g( v6 K
4.   console.log(res1);// 1
   8 C1 X+ L" m0 q1 s) }) F
5.   console.log(res2);// 2
   ! F4 C) O8 g, d% B( l& H
6. }

复制代码

* f- `; i/ t! ~6 I, w% p补充但是要做并发请求时，还是要用到

1. Promise.all()
   , b- x0 U9 H2 A5 p$ ]
2. const fn = () =>{
   
3.    Promise.all([fn1(),fn2()]).then(res =>{
   
4.        console.log(res);// [1,2]
   2 I! ^8 X* a4 E+ k3 S
5.    })
   
6. }

复制代码

# \" r0 q: M; o0 ?7 M如果并发请求时，只要其中一个异步函数处理完成，就返回结果，要用到Promise.race()。
十一、后续欢迎来对以上十点leader的吐槽进行反驳，你的反驳如果有道理的，下次代码评审会上，我替你反驳。此外以上的整理内容有误的地方，欢迎在原文评论中指正，万分感谢。