`审核人：张红桥` `被审核代码负责人：廖爽` `代码地址：http://192.168.1.170/hyfe/demo-project/tree/master/demo-liaoshuang-es6` #### 1.页面不能正常展示，未做缩放，出现滚动条（这个/） 文件名：`car.html` 、 解决方案: 在loader/base.config.js文件中开启缩放，并设置高宽 ``` zoom: true, // 固定的页面宽(与body的width相同) pageWidth: 3456, // 固定的页面高(与body的height相同) pageHeight: 1944 ``` 在css添加样式 ``` html { height: 100%; } ``` #### 2.所有图表未做数据更新处理（修改中） 文件名：`charts/*/*.js` 解决方案: 写一个定时器模拟数据更新，使用update,enter,exit对图表做数据更新处理 #### 3.汽车品牌分布图表未展示值（修改中） 文件名：`charts/brandDis/brandDis.js` 解决方案: 组件中添加value #### 4.图表组件头部注释与图表功能不对应（已修改） 文件名：`charts/brandDis/brandDis.js` ``` /** * @Author: liaos * @DateTime: 2017-11-06 14:50:26 * @Description: 车辆页面-左上角数字渲染 * @Last Modified By: liaos * @Last Modified Time: 2017-11-06 14:50:26 */ ``` 解决方案: 注释与功能对应，该组件是汽车品牌分布 #### 5、ES6语法可优化（已修改） 文件名：`charts/brandDis/brandDis.js` 、 `charts/*/*.js` charts下面的好多文件都可优化 ```javascript this.pie = d3.layout.pie() .value((d) => { return d.value }) ``` 解决方案: 返回的语句只有一句省略{ } ```javascript this.pie = d3.layout.pie() .value((d) => d.value) ``` #### 6、变量命名可优化（已修改） 文件名：`script/render.js` ``` constructor() { this.brand = new Brand('.brand-dis') this.Rule = new Rule('.rule-num') this.Traf = new Traf('.traf-num') this.Road = new Road('.road-flow') } ``` 解决方案: this.Rule、this.Traf、this.Road建议改成小写，类名首字母大写 #### 7、linearGradient建议写filter.hbs模板添加（修改中） 文件名：`charts/util/util.js` 解决方案: 建议把gradient、filter等元素写到filter.hbs模板，通过模板添加。减少js的dom操作