代码规范需要防微杜渐code review6个小错误纠正

 更新时间:2022年06月14日 10:12:16   作者:Sunshine_Lin  
这篇文章主要为大家介绍了代码规范需要防微杜渐code review中的6个小错误纠正,有需要的朋友可以借鉴参考下,希望能够有所帮助,祝大家多多进步,早日升职加薪

code review

所谓code review,意思很明确,就是代码回顾,这个环节能帮你发现一些你代码中的不好的习惯,或者一些错误的行为。这个工作一般是团队的老大来做的,但是

我们的团队人均大佬

所以我们都是一起code review的,人多力量大,参加的人越多,越能让你发现自己的错误,从而能及时改正,收益颇丰!!!

注:下方代码都是虚构,主要为了讲解代码错误习惯,请勿较劲~

分析一下我的错误代码行为

1、写没必要的函数

场景:我需要给某个html根标签设置一个属性,而这个行为是需要在项目初始化的时候去做的,我当时的做法是:

// utils
export const setXXXyyy = () => {
  const tag = document.tag;
  const xxx = tag.getAttribute('xxx');
  if (xxx || xxx === null) {
    tag.setAttribute('xxx', 'yyy');
  }
};
// app.tsx
import { setXXXyyy } from 'utils'
// 初始化时
setXXXyyy()

缺陷:可能我有考虑到以后会在此函数里去做更多事,但是现阶段这么写确实有点多余

改正:直接在模板html里把此属性加到标签上即可

// public/index.html
<tag xxx="yyy"></tag>

2、Promise传递不明值

场景:我需要请求一个接口,请求返回一组数据,我需要把这组数据中的某个参数通过Promise带出去

// 请求方法
const request = () => {
  return new Promise(async (resolve) => {
    const res = await axios(...)
    resolve(res)
  })
}
// 使用
const res = await request()
console.log(res.data.answer)

缺陷:直接把res传递出去了,如果没有ts的限制,那么下一个使用request的开发人员根本不知道这个res里有什么,也自然就不知道有answer这个目标数据了

改正:直接把目标数据answer传递出去

const request = () => {
  return new Promise(async (resolve) => {
    const res = await axios(...)
    // 改正
    resolve(res.data.answer)
  })
}
const answer = await request()
console.log(answer)

3、使用没必要try catch

场景:试异步操作1,如果失败的话试异步操作2,如果失败进行操作3

action1().catch(() => {
  try {
    await action2()
  }  catch(e) {
    throw e
  }
}).catch(() => {
  action3()
})

错误:没必要使用try catch,await失败之后,会自动返回一个失败的Promise,通过链式调用,会执行下一个catch

改正:去除try catch

action1().catch(() => {
  await action2()
}).catch(() => {
  action3()
})

4、Promise.all并发限制

场景:我需要用Promise.all去进行并行执行一些异步操作,这个操作是在服务端的。。

// promises可能会有几十个,几百个,上千个
await Promise.all(promises)

缺陷:众所周知,服务端有时候是很脆弱的,可能你几十个并发就会把服务端给折腾的不要不要的了,所以控制并发是很重要的

改正:既然承受不住,那就控制并发呗,网上很多控制并发的方案。这里我就不说哪个方案比较好了。。自己实现也行,用库也行,看你们团队需要哪个吧。。

// 控制并发
await promiseAllLimit(promises)

5、Nodejs中使用过多sync函数

场景:在后端那边进行文件操作,由于比较喜欢用同步方法,所以用了

const readData = fs.readFileSync(filepath);
fs.writeFileSync(targetPath, readData);
const workbook = xlsx.readFile(targetPath);
fs.unlinkSync(targetPath);

缺陷:Nodejs引以为傲的就是他的大部分方法都支持异步,所以它才能在高并发的场景中那么牛,所以尽量少用它的同步方法,性能会好一些

改正:使用它对应的异步方法吧,,不过要嵌套了。。很烦啊

6、判空要放前面

场景:有两个参数,我要对他们进行一系列判断:

  • 1、判断数据表里有没有这两个参数
  • 2、判断这两个参数是否重叠
  • 3、判断这两个参数是否为空
if (usename) {
  ...
}
if (password) {
  ...
}
if (!username || !password) {
  ...
}

缺陷:这个顺序是不对的,如果两个参数为空,则没必要进行另外两步

改正:改变顺序:

  • 1、这两个参数是否为空
  • 2、判断数据表里有没有这两个参数
  • 3、判断这两个参数是否重叠
if (!username || !password) {
  ...
}
if (usename) {
  ...
}
if (password) {
  ...
}

以上就是代码规范需要防微杜渐code review6个小错误纠正的详细内容,更多关于代码规范code review错误纠正的资料请关注脚本之家其它相关文章!

相关文章

  • CocosCreator实现技能冷却效果

    CocosCreator实现技能冷却效果

    这篇文章主要介绍了CocosCreator实现技能冷却效果,同学们可以跟着教程,亲手试一下,代码都是可以复用的
    2021-04-04
  • javascript获取系统当前时间的方法

    javascript获取系统当前时间的方法

    这篇文章主要介绍了javascript获取系统当前时间的方法,格式为:2015年11月19日 星期四09:21:05,需要的朋友可以参考下
    2015-11-11
  • Javascript调试之console对象——你不知道的一些小技巧

    Javascript调试之console对象——你不知道的一些小技巧

    这篇文章主要总结了console对象的一些有用的方法,非常不错,具有参考借鉴价值,需要的朋友参考下吧
    2017-07-07
  • TypeScript 类class与修饰符的详细使用教程

    TypeScript 类class与修饰符的详细使用教程

    通过 class 关键字定义一个类,然后通过 new 关键字可以方便的生产一个类的实例对象,这个生产对象的过程叫 实例化,类的成员就是类中所有的属性和方法,这篇文章主要介绍了TypeScript 类class与修饰符的详细使用,需要的朋友可以参考下
    2022-06-06
  • uniapp实现可滑动选项卡

    uniapp实现可滑动选项卡

    这篇文章主要为大家详细介绍了uniapp实现可滑动选项卡,文中示例代码介绍的非常详细,具有一定的参考价值,感兴趣的小伙伴们可以参考一下
    2020-10-10
  • 一个极为简单的requirejs实现方法

    一个极为简单的requirejs实现方法

    下面小编就为大家带来一篇一个极为简单的requirejs实现方法。小编觉得挺不错的,现在就分享给大家,也给大家做个参考。一起跟随小编过来看看吧
    2016-10-10
  • JS实现登录页密码的显示和隐藏功能

    JS实现登录页密码的显示和隐藏功能

    在登录页经常会用到通过点击文本框的类似小眼睛图片来实现隐藏显示密码的功能。下面给大家介绍基于JS实现登录页密码的显示和隐藏功能,需要的朋友参考下吧
    2017-12-12
  • Whatever:hover 无需javascript让IE支持丰富伪类

    Whatever:hover 无需javascript让IE支持丰富伪类

    绝大部分现代浏览器支持 css 中的 :hover 伪类选择器,可以用于所有 html 元素。
    2010-06-06
  • javascript简单比较日期大小的方法

    javascript简单比较日期大小的方法

    这篇文章主要介绍了javascript简单比较日期大小的方法,涉及JavaScript针对日期的转换与判定操作技巧,需要的朋友可以参考下
    2016-01-01
  • JavaScript实现table切换的插件封装

    JavaScript实现table切换的插件封装

    这篇文章主要为大家详细介绍了JavaScript实现table切换的插件封装,文中示例代码介绍的非常详细,具有一定的参考价值,感兴趣的小伙伴们可以参考一下
    2021-10-10

最新评论