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

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

---

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

上传

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

1. const obj = {
   9 z  u  s0 r" {; |( N, a( i
2.     a:1,
   ) O- G" V% g9 ?: A: A
3.     b:2,
   8 R6 k1 X" N4 B3 \4 f7 f/ b
4.     c:3,
   
5.     d:4,
   
6.     e:5,
   
7. }

复制代码

$ `- y3 a) l# ~1 ]! y4 i  r吐槽：
, G2 K' Z) B$ j3 s7 e6 {) r

1. const a = obj.a;
   - u4 M1 U0 @8 ~& J  N
2. const b = obj.b;
   
3. const c = obj.c;
   7 M  M) _" u9 C% ^4 x; P, i" Q
4. const d = obj.d;
   
5. const e = obj.e;

复制代码

! ^6 j* n* E. q. T- i
或者
7 P2 B) u' c3 h6 ?

1. const f = obj.a + obj.d;
   . a1 y0 @4 `0 {8 l7 |; ]- [2 j
2. const g = obj.c + obj.e;

复制代码

吐槽：“不会用ES6的解构赋值来取值吗？5行代码用1行代码搞定不香吗？直接用对象名加属性名去取值，要是对象名短还好，很长呢？搞得代码中到处都是这个对象名.
- W8 l8 V) w9 i”改进：
& ?. G8 M) e+ K; A5 H2 {: p

1. const {a,b,c,d,e} = obj;
   
2. const f = a + d;
   
3. const g = c + e;

复制代码

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

1. const {a:a1} = obj;
   & |5 x: g) N# \2 c' l
2. console.log(a1);// 1补充ES6的解构赋值虽然好用。但是要注意解构的对象不能为undefined、null。否则会报错，故要给被解构的对象一个默认值。
   , A% @4 I7 R5 ?7 h4 N
3. const {a,b,c,d,e} = obj || {};

复制代码

二、关于合并数据的吐槽比如合并两个数组，合并两个对象。const a = [1,2,3];
9 X  `/ G) |# M( n! K3 ^

1. const b = [1,5,6];
   
2. const c = a.concat(b);//[1,2,3,1,5,6]
   
3. const obj1 = {
   
4.   a:1,
   
5. }
   # S) w  j) ]- _% K" ?
6. const obj2 = {
   
7.   b:1,
   3 E" Q9 B8 r. f7 o1 o
8. }
   7 K2 J# @+ c3 e$ {( t0 {
9. const obj = Object.assign({}, obj1, obj2);//{a:1,b:1}吐槽ES6的扩展运算符是不是忘记了，还有数组的合并不考虑去重吗？改进const a = [1,2,3];
   
10. const b = [1,5,6];
    * y7 c) i- R+ R1 m7 M  E! J  E
11. const c = [...new Set([...a,...b])];//[1,2,3,5,6]
    $ z$ F+ ^, d" R
12. const obj1 = {
    . M, K) m2 `- y' @
13.   a:1,
    5 @1 m% u7 ?! W( Z! |) Y
14. }
    
15. const obj2 = {
    . ~  \! E0 m; i. k& o$ x
16.   b:1,
      }7 }2 P6 c2 o2 Y& b
17. }
    
18. const obj = {...obj1,...obj2};//{a:1,b:1}

复制代码

三、关于拼接字符串的吐槽
* M0 p/ r) \, ], H1 @% ~, n

1. const name = '小明';
   2 L) Z( V( w% h7 c! V/ G  N
2. const score = 59;
   
3. let result = '';
   , z' ^; `- j% C/ L  T" g# Q
4. if(score > 60){
   ; E- j; H0 o$ f* O5 I
5.   result = `${name}的考试成绩及格`;
   0 n$ @+ \; F; r1 I( F  V7 _4 G6 r
6. }else{
   ' {' m2 Y" h+ G# H
7.   result = `${name}的考试成绩不及格`;
   
8. }

复制代码

吐槽像你们这样用ES6字符串模板，还不如不用，你们根本不清楚在${}中可以做什么操作。在${}中可以放入任意的JavaScript表达式，可以进行运算，以及引用对象属性。改进const name = '小明';
% z9 R1 s# x1 G8 s7 J8 G3 b) G1 \

1. const score = 59;
   + t  Z" c: h3 S+ P. h& \$ F" X$ B
2. const result = `${name}${score > 60?'的考试成绩及格':'的考试成绩不及格'}`;

复制代码

四、关于if中判断条件的吐槽if(
6 i: O" i6 @7 i( x( a

1.     type == 1 ||
   
2.     type == 2 ||
   
3.     type == 3 ||
   % O9 R) P" D4 T) {/ F
4.     type == 4 ||
   
5. ){
   
6.    //...
   
7. }

复制代码

$ O/ d) k1 Q$ J吐槽ES6中数组实例方法includes会不会使用呢？
改进const condition = [1,2,3,4];
9 _7 E+ G5 r( B: [, a5 E

1. if( condition.includes(type) ){
   - V* t" R: N+ h/ Q! l  {
2.    //...
   
3. }

复制代码

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

1. const result = a.filter(
   
2.   item =>{
   
3.     return item === 3
   8 e! f4 b" R6 z  e: y9 x
4.   }
   , U$ S0 t$ w. F! H/ M! _
5. )

复制代码

1 F  y) y- W( T吐槽如果是精确搜索不会用ES6中的find吗？性能优化懂么，find方法中找到符合条件的项，就不会继续遍历数组。改进const a = [1,2,3,4,5];

1. const result = a.find(
   
2.   item =>{
   
3.     return item === 3
   
4.   }
   + n& I" a, b+ g1 ~# k1 [  f- A
5. )

复制代码

7 z# g  p3 ^4 Q) L5 o1 E# M
六、关于扁平化数组的吐槽一个部门JSON数据中，属性名是部门id，属性值是个部门成员id数组集合，现在要把有部门的成员id都提取到一个数组集合中。

1. const deps = {
   4 z. u( v0 x. b. v
2. '采购部':[1,2,3],
   
3. '人事部':[5,8,12],
   
4. '行政部':[5,14,79],
   
5. '运输部':[3,64,105],
   2 d/ t* `* q( S
6. }
     o5 S! f0 |6 P
7. let member = [];
   ' R7 C/ v# w% `& @. ^
8. for (let item in deps){
   ) Z  o" @- f$ ]' @) n
9.     const value = deps[item];
   ; @$ t9 p( j8 }+ z3 i( R- G" v: g
10.     if(Array.isArray(value)){
    . S# a. p/ I  u& H, ]
11.         member = [...member,...value]
    7 l0 ^4 S) z5 d: w
12.     }
    / p9 v( O; j* D' [( {! Q
13. }
    
14. member = [...new Set(member)]

复制代码

) [0 \+ z8 R: p1 o9 P9 x
, [: A$ H, X7 T/ Y吐槽获取对象的全部属性值还要遍历吗？Object.values忘记了吗？还有涉及到数组的扁平化处理，为啥不用ES6提供的flat方法呢，还好这次的数组的深度最多只到2维，还要是遇到4维、5维深度的数组，是不是得循环嵌套循环来扁平化？
改进
- ~% t$ |- W) \" _( Z' I1 v

1. const deps = {
   
2.     '采购部':[1,2,3],
   
3.     '人事部':[5,8,12],
   . I2 ^+ |! n5 y" ]
4.     '行政部':[5,14,79],
   ) t0 }8 H; E. a, i/ `
5.     '运输部':[3,64,105],
   , r! b4 n$ y9 K. ?; g. _
6. }
   ) l, ?7 @+ x' ^
7. let member = Object.values(deps).flat(Infinity);

复制代码

  P( v/ _0 y& K( K其中使用Infinity作为flat的参数，使得无需知道被扁平化的数组的维度。补充flat方法不支持IE浏览器。七、关于获取对象属性值的吐槽const name = obj && obj.name;吐槽ES6中的可选链操作符会使用么？改进const name = obj?.name;八、关于添加对象属性的吐槽当给对象添加属性时，如果属性名是动态变化的，该怎么处理。let obj = {};
, }$ _; m% N8 X$ z. I

1. let index = 1;
   ' F4 x2 V! L  J  T0 a8 l- P6 r% v
2. let key = `topic${index}`;
   
3. obj[key] = '话题内容';吐槽为何要额外创建一个变量。不知道ES6中的对象属性名是可以用表达式吗？改进let obj = {};
   ! h/ I1 f. }, l
4. let index = 1;
   
5. obj[`topic${index}`] = '话题内容';

复制代码

6 s1 ^7 X5 A  b& r$ m九、关于输入框非空的判断在处理输入框相关业务时，往往会判断输入框未输入值的场景。if(value !== null && value !== undefined && value !== ''){
    //...
}
  |- b7 ], t; z1 H! ^吐槽ES6中新出的空值合并运算符了解过吗，要写那么多条件吗？
if((value??'') !== ''){
  //...
}
' H. }; ~5 O# r+ o十、关于异步函数的吐槽异步函数很常见，经常是用 Promise 来实现。

1. const fn1 = () =>{
   0 E5 g5 `2 N* `  p" D/ p
2.   return new Promise((resolve, reject) => {
   
3.     setTimeout(() => {
   
4.       resolve(1);
   
5.     }, 300);
   ' k  \! c$ u5 P' p" k; V
6.   });
   & {" h1 Y. T, n  L/ z
7. }
   
8. const fn2 = () =>{
   0 J& G' r8 I  ?3 G5 ~
9.   return new Promise((resolve, reject) => {
   ! h9 s) k4 H/ y; ?* h
10.     setTimeout(() => {
    % M0 J$ C% K' D* Q) T5 C0 M
11.       resolve(2);
    
12.     }, 600);
    
13.   });
    
14. }
    
15. const fn = () =>{
    . k  c4 ^2 J( Y" d+ c7 W4 m8 L) q
16.    fn1().then(res1 =>{
    4 k  @3 a9 t  B& K/ O+ Z
17.       console.log(res1);// 1
    
18.       fn2().then(res2 =>{
    / k! I7 H. A- ~% N, ?6 ]
19.         console.log(res2)
    
20.       })
    
21.    })
    - w- b2 W) o1 n9 A* L
22. }

复制代码

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

1. const fn = async () =>{
   
2.   const res1 = await fn1();
   8 M$ t- N! G" L( Q
3.   const res2 = await fn2();
   
4.   console.log(res1);// 1
   & K) t" {0 Z: t1 k( w
5.   console.log(res2);// 2
   8 k6 o+ z, @2 V
6. }

复制代码

# U( ~0 ~+ @5 D9 V9 C补充但是要做并发请求时，还是要用到
& }3 S9 ~! o- l( M( X

1. Promise.all()
   & ?" `9 S1 @. W; |# Y/ J
2. const fn = () =>{
   
3.    Promise.all([fn1(),fn2()]).then(res =>{
   0 e# H' }7 U" \9 n4 }) Q: I
4.        console.log(res);// [1,2]
   
5.    })
   0 k1 i- [' L1 E  d7 F
6. }

复制代码

如果并发请求时，只要其中一个异步函数处理完成，就返回结果，要用到Promise.race()。
  v( @- U  ^7 g4 k十一、后续欢迎来对以上十点leader的吐槽进行反驳，你的反驳如果有道理的，下次代码评审会上，我替你反驳。此外以上的整理内容有误的地方，欢迎在原文评论中指正，万分感谢。
- X* @% ~9 g' m
. M7 s) B& ~; F: }6 o

---

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

Powered by Discuz! X3.5