漫潮者,私有云,NAS,新媒体电商.软件开发,商乾

标题: 你会用ES6，那倒是用啊！！！ [打印本页]

---

作者: 商乾美工    时间: 2022-4-29 17:32
标题: 你会用ES6，那倒是用啊！！！
(, 下载次数: 13)

上传

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

1. const obj = {
   % Z% s! Z9 `' @/ N7 Y8 b
2.     a:1,
   
3.     b:2,
   , K1 A) g) Y9 Z) n
4.     c:3,
   ) Z# V- P: t" E$ ~- F3 n0 {
5.     d:4,
   
6.     e:5,
     J& \9 p. b5 T0 p6 X4 ^) R
7. }

复制代码

( z' ?  z' P8 k+ i$ ^  g吐槽：

1. const a = obj.a;
   / H3 U6 _$ E. Q( d* M' ]1 @% E
2. const b = obj.b;
   
3. const c = obj.c;
   6 \. e5 F# c  c% x7 ^1 H
4. const d = obj.d;
   
5. const e = obj.e;

复制代码

或者
+ _% y: s% W6 [$ h7 S1 G

1. const f = obj.a + obj.d;
   2 e7 q8 u: |3 S4 C9 R
2. const g = obj.c + obj.e;

复制代码

吐槽：“不会用ES6的解构赋值来取值吗？5行代码用1行代码搞定不香吗？直接用对象名加属性名去取值，要是对象名短还好，很长呢？搞得代码中到处都是这个对象名.
* w9 w& f; c6 K' Y”改进：
; c2 _3 k) d3 @% f

1. const {a,b,c,d,e} = obj;
   & z- T* V6 i+ K, H1 @
2. const f = a + d;
   
3. const g = c + e;

复制代码

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

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

复制代码

二、关于合并数据的吐槽比如合并两个数组，合并两个对象。const a = [1,2,3];
1 D. j  h5 x$ S5 Z

1. const b = [1,5,6];
   7 Q' T/ _% ^9 K: ~) F
2. const c = a.concat(b);//[1,2,3,1,5,6]
   
3. const obj1 = {
   ( n( U, a4 N+ C# ]3 H; h0 x4 d
4.   a:1,
   
5. }
     c0 v1 ~' O2 K, u/ p# T; t' X/ F  F. c' I$ t
6. const obj2 = {
   1 }2 {! k7 j1 Z% s2 k3 U5 E6 W
7.   b:1,
   . F. g8 k1 E" j, P! }( ?2 Z
8. }
   " g8 V# w7 d( S
9. const obj = Object.assign({}, obj1, obj2);//{a:1,b:1}吐槽ES6的扩展运算符是不是忘记了，还有数组的合并不考虑去重吗？改进const a = [1,2,3];
   : g) J' a: m  i8 c2 n; m1 j
10. const b = [1,5,6];
    6 D( A" r- `1 L2 \
11. const c = [...new Set([...a,...b])];//[1,2,3,5,6]
    
12. const obj1 = {
    
13.   a:1,
    
14. }
    
15. const obj2 = {
    
16.   b:1,
    
17. }
    
18. const obj = {...obj1,...obj2};//{a:1,b:1}

复制代码

三、关于拼接字符串的吐槽
/ g" F0 {. Y& ~, ~5 ]5 Q9 d5 |! _

1. const name = '小明';
   
2. const score = 59;
   4 ]; W) U0 ?) [' Z5 ?6 }; w
3. let result = '';
   , C8 \. {) ^9 c# ]2 p" Q- E; T
4. if(score > 60){
   - i4 x0 B9 Z( \1 v) N2 ^
5.   result = `${name}的考试成绩及格`;
   
6. }else{
   
7.   result = `${name}的考试成绩不及格`;
   
8. }

复制代码

0 i+ i$ s* u4 U7 f( T吐槽像你们这样用ES6字符串模板，还不如不用，你们根本不清楚在${}中可以做什么操作。在${}中可以放入任意的JavaScript表达式，可以进行运算，以及引用对象属性。改进const name = '小明';

1. const score = 59;
   ! A! D- D1 W) [) A3 Z
2. const result = `${name}${score > 60?'的考试成绩及格':'的考试成绩不及格'}`;

复制代码

; K9 i3 B& j7 n6 m% T5 I5 f四、关于if中判断条件的吐槽if(

1.     type == 1 ||
   
2.     type == 2 ||
   
3.     type == 3 ||
   
4.     type == 4 ||
   
5. ){
   
6.    //...
   
7. }

复制代码

, \( i/ f/ {1 v吐槽ES6中数组实例方法includes会不会使用呢？
: }* C+ D: ?7 J9 \0 Q改进const condition = [1,2,3,4];

1. if( condition.includes(type) ){
   
2.    //...
   
3. }

复制代码

. E5 d# G# k8 s  T0 A五、关于列表搜索的吐槽在项目中，一些没分页的列表的搜索功能由前端来实现，搜索一般分为精确搜索和模糊搜索。搜索也要叫过滤，一般用filter来实现。const a = [1,2,3,4,5];

1. const result = a.filter(
   
2.   item =>{
   
3.     return item === 3
   ) v  ?7 \' e# a1 V: G0 T
4.   }
   
5. )

复制代码

  N2 E2 w' j  x* D1 I1 L# D2 w9 Y$ p7 Z吐槽如果是精确搜索不会用ES6中的find吗？性能优化懂么，find方法中找到符合条件的项，就不会继续遍历数组。改进const a = [1,2,3,4,5];

1. const result = a.find(
   
2.   item =>{
   7 d( t, ]! P' E9 z
3.     return item === 3
   
4.   }
   ) [( x! s4 n2 E/ d$ {
5. )

复制代码

六、关于扁平化数组的吐槽一个部门JSON数据中，属性名是部门id，属性值是个部门成员id数组集合，现在要把有部门的成员id都提取到一个数组集合中。
4 [% _' D! k! e  `( T5 t3 x) \9 |

1. const deps = {
   
2. '采购部':[1,2,3],
   * H4 w" l! O' Q+ s7 l% r
3. '人事部':[5,8,12],
   1 Z% ]* n- a; A1 r, N( j4 u$ c
4. '行政部':[5,14,79],
   
5. '运输部':[3,64,105],
   ! _, S# u: e" o  L/ U* }
6. }
   / X- Z# o" n' z7 h
7. let member = [];
   
8. for (let item in deps){
   7 r8 G" y( w' r3 Y0 f) W- s' _
9.     const value = deps[item];
   3 H& _1 h0 _0 c  i
10.     if(Array.isArray(value)){
    & k) m7 f, s' ?+ w
11.         member = [...member,...value]
    ' u7 r% H6 @, V* A+ U6 r
12.     }
    ) u# L, I' f* s1 B* s: |
13. }
    1 {/ G- j! v' n2 h' m. g8 w
14. member = [...new Set(member)]

复制代码

. p( ~! {+ Z( ~, Z3 E9 j吐槽获取对象的全部属性值还要遍历吗？Object.values忘记了吗？还有涉及到数组的扁平化处理，为啥不用ES6提供的flat方法呢，还好这次的数组的深度最多只到2维，还要是遇到4维、5维深度的数组，是不是得循环嵌套循环来扁平化？
+ W  R8 [4 B" z" y6 }0 k$ o改进

1. const deps = {
   
2.     '采购部':[1,2,3],
   / b9 {! r' j- T0 t' M
3.     '人事部':[5,8,12],
   1 F2 ~9 |- I- q" G
4.     '行政部':[5,14,79],
   ( ^5 m; S/ v, e$ b' R
5.     '运输部':[3,64,105],
   
6. }
   ) n" M% C) T( [/ a
7. let member = Object.values(deps).flat(Infinity);

复制代码

其中使用Infinity作为flat的参数，使得无需知道被扁平化的数组的维度。补充flat方法不支持IE浏览器。七、关于获取对象属性值的吐槽const name = obj && obj.name;吐槽ES6中的可选链操作符会使用么？改进const name = obj?.name;八、关于添加对象属性的吐槽当给对象添加属性时，如果属性名是动态变化的，该怎么处理。let obj = {};
# D, b4 p9 u) o4 n

1. let index = 1;
   
2. let key = `topic${index}`;
   % Q5 B+ G% ~% h% U6 F2 N4 d5 e8 W
3. obj[key] = '话题内容';吐槽为何要额外创建一个变量。不知道ES6中的对象属性名是可以用表达式吗？改进let obj = {};
   & `$ T+ ?% v9 Z9 t3 \9 b* t" K6 j# _
4. let index = 1;
   
5. obj[`topic${index}`] = '话题内容';

复制代码

( f, w+ M; M5 n  n" ]+ r# F
1 r9 A. |' x! o! ^! L0 w# w九、关于输入框非空的判断在处理输入框相关业务时，往往会判断输入框未输入值的场景。if(value !== null && value !== undefined && value !== ''){
    //...
}
2 ?  M( ~8 q5 V7 m- E吐槽ES6中新出的空值合并运算符了解过吗，要写那么多条件吗？
) E4 w: u8 q2 L# uif((value??'') !== ''){
  //...
}
. N' ^0 _/ O% s8 d8 X; o- f* y9 J% n十、关于异步函数的吐槽异步函数很常见，经常是用 Promise 来实现。
. D. ~& @+ h  t7 ?  f8 ?$ e

1. const fn1 = () =>{
   
2.   return new Promise((resolve, reject) => {
   8 T2 `0 Q7 K! Y  R+ V: v) @+ M! Y
3.     setTimeout(() => {
   ; O8 L4 h) {* ]8 D8 G/ y
4.       resolve(1);
   9 y) [8 V7 w& R2 ~3 e3 R
5.     }, 300);
   
6.   });
   ) h2 j8 u2 T1 f  d
7. }
   ; m0 x. Z9 f) d' D
8. const fn2 = () =>{
   
9.   return new Promise((resolve, reject) => {
   
10.     setTimeout(() => {
    % k3 J3 l6 H# K: Z* o6 [
11.       resolve(2);
    
12.     }, 600);
    
13.   });
    
14. }
    
15. const fn = () =>{
    ( }" k) n& C) i
16.    fn1().then(res1 =>{
    3 Q0 G& x* W: k% b' N
17.       console.log(res1);// 1
    $ |* {: L+ F. \. h0 u4 H
18.       fn2().then(res2 =>{
    2 o9 f. y) J: _& l7 w' b
19.         console.log(res2)
    
20.       })
    ) P% L/ q4 x2 h) C2 f% P* M
21.    })
    . h' q- z) n2 C
22. }

复制代码

吐槽如果这样调用异步函数，不怕形成地狱回调啊！改进
; Y$ X2 O( o8 Y5 Q. j! ^

1. const fn = async () =>{
   
2.   const res1 = await fn1();
   + A  X3 z7 Y. W3 O. X
3.   const res2 = await fn2();
   
4.   console.log(res1);// 1
   
5.   console.log(res2);// 2
   
6. }

复制代码

$ R( R# V9 P/ k补充但是要做并发请求时，还是要用到

1. Promise.all()
   
2. const fn = () =>{
   
3.    Promise.all([fn1(),fn2()]).then(res =>{
   3 U3 c  L+ m8 `# j5 w' A+ g; I
4.        console.log(res);// [1,2]
   - Q  A- s# r2 o0 u2 O
5.    })
   
6. }

复制代码

4 _% j! w# a5 `* y  B( F! Q" N. ]如果并发请求时，只要其中一个异步函数处理完成，就返回结果，要用到Promise.race()。
十一、后续欢迎来对以上十点leader的吐槽进行反驳，你的反驳如果有道理的，下次代码评审会上，我替你反驳。此外以上的整理内容有误的地方，欢迎在原文评论中指正，万分感谢。
7 Q! T2 J# [) w, A5 _6 Z

---

欢迎光临 漫潮者,私有云,NAS,新媒体电商.软件开发,商乾 (https://www.aurrel.com/)

Powered by Discuz! X3.5