`审查人：白占宁` `被审核代码负责人：欧阳德才&谢洋` `代码地址：http://10.0.91.5/opt/svn/DataCommandScheduling/project/bjDataVisualization` #### 1、组件结构及目录结构重构（已修改） - 区分清楚container和component的区别； - 部分组件划分不合理。 #### 2、文件注释不统一（已修改） ``` /** * 总览页 核查录入 mock数据 * @author: xieyang@hiynn.com * @datetime: 2018-03-23 14:11:20 */ ``` 统一文件注释，参考以下注释： ``` /* * @Author: xxx * @Email: xxx@hiynn.com * @Description: 模板Store主文件，创建store文件 * @Date: 2018-02-08 14:03:30 * @Last Modified by: xxx * @Last Modified time: 2018-02-11 15:37:33 */ ``` #### 3、`sagas`文件夹下的子文件夹处理方式参考贵州项目重构（已修改） 子模块的mock和业务逻辑统一提供一个`index`文件，外面文件只访问该文件。 #### 4、不符合基本的编码规范（已修改） ``` // {} 两边留空格 const {marginLeft} = this.props // 参数id没有使用 const apifetchCaseToday = (id) => fetch('fetchCaseToday') ``` #### 5、代码严重缺少注释（已修改） 代表文件：`/components/importantDomainPerson/index.js` 其他文件也存在类似问题，如sagas文件夹中业务逻辑文件。 #### 6、删除无用的文件或文件夹（已修改） `chart / components / container`等文件夹下没有使用到的文件或文件夹全部删除掉。