`审核人：白占宁` `被审核代码负责人：廖爽` `代码地址：http://192.168.1.170/hyfe/demo-project/tree/master/demo-liaoshuang-es6` #### 1、mock中模拟数据（仅建议） ``` { 'name|+1': ['1-10次', '11-20次', '21-30次', '31-40次', '41-50次', '51-60次'], 'value|500-5000': 1 } ``` `name`字段可通过mock生成随机数据，不用手动输入，可以减少工作量。 #### 2、mock index.js中import时候的命名（已修改） ``` import brandDis from './brandDis' import carNum from './carNum' import roadFlow from './roadFlow' import ruleNum from './ruleNum' import trafNum from './trafNum' export default { brandDis, carNum, roadFlow, ruleNum, trafNum } ``` `import`时候的变量名建议添加`Fetch`后缀，如`carNumFetch`，因为发送请求时以该名称作为URL的，添加后缀，进一步减少出现命名冲突的可能性。 #### 3、CSS命名需要具有语义化（这个这个） ``` <li class="car-li-c flex-a"> ``` 解决方案：变量命名要有一定的语义化，让别人看到就能大概意会到该变量或class的作用。 #### 4、使用ES6封装组件的方式需要优化（已修改） 参考`charts`下，其他组件的封装方式进行修改。 #### 5、文件位置不对（已修改） `scripts`目录下放置业务代码，所有与图表相关的都放到`charts`目录下面。 #### 6、避免注释过多的代码（已修改） 不需要的代码就删除掉，不要大篇幅的注释掉。 #### 7、选择元素或绑定事件统一使用jQuery（已修改） 路径：`apps/keyMan/scripts/keyMan.js` ``` document.querySelector('#newadd').addEventListener('click', function(){ let lays = document.querySelector('#layers') ``` 解决方案：使用jQuery选择元素或绑定事件。 #### 8、组件划分结构不清晰（已修改） 路径：`apps/keyMan/scripts/keyMan.js` 重点人员页面应该划分为列表、弹出框和高级搜索三个组件，相应的文件也应包括三个，在外部的`index.js`中，实例化三个组件，调用渲染方法即可，其他图表组件也类似。 减少HTML中的代码，尽量使用模板实现，更一步细化组件。 #### 9、代码组织结构（已修改） 路径：`apps/keyMan/scripts/keyMan.js` 使用`class`来组织代码，尽量包括代码结构清晰，提升代码可读性。 #### 10、图片命名问题（已修改） 不要以`_`下划线形式命名。