Code Review

1、熟练讲解对应业务需求；

2、代码的运行流程清晰合理；

1、目录、文件分配合理；

2、正确并规范使用组件及工具方法；

3、设计模式正确；

相同功能的类、函数、对象、组件等应尽量抽为共用

代码耦合度合理

三、命名

适用于所有项目名，文件名，变量名，函数名，class， ID

做有意义的区分

比如applicationName 非写个applicationNameString 难道name会是个浮点型不成

又如enable 非写个isEnable 难道enable会是个字符串不成

相同行为的方法命名规则需要固定

比如：同一个人写3个component组件，都有一个组件创建时，加载数据的函数。一个叫initData, 一个叫loadData, 一个叫getData 这就不合适

又如：一个组件使用了三个弹窗，三个弹窗的事件响应函数命名各不相同，有的叫handleClick, 有的叫onClick, 有的叫afterClicked 这就不合适

有语境支撑的变量，可不带语境

没有语境支撑的变量，要自带语境

比如有Application.js的一个类，里面是一个Application类，在此类里面需定义一个状态枚举供此文件自己使用。可以定义为StatusEnum。因为此文件和Application类 已经提供了语境

如果枚举值需要给外部使用，或定义在enum.js里面。应命名为ApplicationStatusEnum

看一眼觉得精简

空间复杂度合理

时间复杂度合理

如：

1、不要有不必要的临时变量；

eg: _this = this

2、能用局部变量的，就不要使用全局变量；

3、使用合适的API方法，比如正确区分并使用map, filter, forEach,find,some等方法。

4、正确定义let/const变量

细看也觉得很精简

1、在最后的else逻辑中，有return方法，可去掉else，直接return（sonar要求）

名字和内容一致，不要做多件事。

判断标准

1.看能否再拆出一个函数，该函数不仅仅是单纯的重新诠释其实现

2.看是不是做的同一抽象层上的事

最好没有参数，第二选择是一个参数，最多三个, 当有多个参数时，建议使用对象解构赋值。

function({id: ‘', date: ‘’, type: '’})

参数命名也要严格按照规范来。

如:

setName(value) {

this.name = value

this.shortName = value.substring(0,40)

}

shortName的值本不该在这个函数修改，会让人忽略其变化过程。

要么做什么事，要么回答什么事，二者不可兼得。

比如 setAndCheckExist() =>

dataExist()

setData()

文件内代码编排混乱

相关联的函数尽量放到一起。

比如：

1、在使用async await 之前，想清楚当前场景是否真的需要同步处理。

2、对于promise的一些错误处理，promise本身已经做了很好的正确与错误的区分，就不需要在外面通过try catch 去捕获了。

数据没有判空

没有结束流程，比如依然在loading

重复点击引发错误

未校验用户输入

WAVE / Bookmarklets 工具检测

A/AA 的问题要清零

1、对于组件样式库就可以完全解决的问题，就不要自定义样式了。

2、合理定义自定义样式。