`审核人：张红桥` `被审核代码负责人：赵家红` `代码地址：http://192.168.1.170/hyfe/demo-project/tree/master/demo-zhaojiahong` #### 1、js变量名称不规范（已修改） 文件名：`scripts/fileInto.js` ```javascript var car_number = require('car_number') require('car_numberData') ``` 解决方案: 变量命名统一使用驼峰命名法，如:carNumber #### 2、JS语句尾部分号（已修改） 解决方案: JS文件中建议省略所有分号。 #### 3、JS中统一使用单引号（已修改） 解决方案: JS文件中建议将所有双引号改单引号。 #### 4、mock地址不规范（已修改） 文件名： `mockData/*.js` （road_rate.js除外） 错误code: `util.urlReg('.car_brands')` ``` define(function(require){ require('mock') var util = require('util') Mock.mock(util.urlReg('.car_brands'),{ 'code':'1', 'msg':'success', 'result':{ 'car_brand|6':[{ 'name|+1':['奔驰','宝马','大众','比亚迪','奥迪','其他'], 'value|+1':[35,67,78,56,68,75] }] } }); }); } ``` 解决方案: url地址不要出现 `.` ``` util.urlReg('car_brands') ``` #### 5、头部注释不规范，有多余的注释符（已修改） 文件名： `mockData/infraction.js 、city_traffic.js` ``` // /** // * @Author: zhaojiahong // * @DateTime: 2017-11-3 16:14:29 // * @Description: 驾驶人交通违规 // * @Last Modified By: zhaojiahong // * @Last Modified Time: // */ ``` 解决方案：删除前面的// ``` /** * @Author: zhaojiahong * @DateTime: 2017-11-3 16:14:29 * @Description: 驾驶人交通违规 * @Last Modified By: zhaojiahong * @Last Modified Time: */ ``` #### 6、js中出现无用的console.log()（已修改） 文件名： `scripts/index.js` ``` console.log('dom', $('body')) ``` 解决方案：删除无用的console.log() #### 7.代码结构混乱（已修改） 文件名： `charts/city_traffic.js` ``` var yMark= [-40,-30,-20,-10,0,10, 20, 30];//y轴坐标刻度 var gridLine= [-40,-30,-20,-10,10, 20, 30];//网格线 ``` 解决方案：同级代码应对齐 ``` var yMark= [-40,-30,-20,-10,0,10, 20, 30];//y轴坐标刻度 var gridLine= [-40,-30,-20,-10,10, 20, 30];//网格线 ``` #### 8.变量命名示不语义化（已修改） 文件名： `charts/*.js` ``` var Num = function(data){ ... } ``` 解决方案：建议变量命名时尽量与该方法的功能相符合 #### 9.方法注释不规范（已修改） 文件名： `charts/*.js` ``` function addTc(id, yi) { .... } ``` 解决方案：按注释的规范进行注释(参数要有具体说明) ``` /** * 创建背景填充 * @param {[type]} id [description] * @param {[type]} yi [description] */ function addTc(id, yi) { .... } ``` #### 10、对象里面创建了多个函数（修改中） 文件名：`charts/*.js` 该文件夹下面的文件都有些问题 ](images/a.png) 这样创建方法会影响图表渲染性能，每次在调用的时候都要重新创建该方法 解决方案：建议将这些方法定义成对象里面的属性，如: ``` // var draw = function(data) { addJb: function(opt) { // code ... }, addTc: function(addTc) { // code ... }, addTc: function(nums, rectPadding) { // code ... } } ``` #### 11、头部注释错误，与本人不对应（已修改） 文件名： `charts/infraction.js` ``` /** * @Author: asuangs * @DateTime: 2017-10-24 14:21:40 * @Description: 车辆管理（驾驶人交通违规数） * @Last Modified By: asuangs * @Last Modified Time: 2017-10-24 14:21:40 */ ``` 解决方案：Author者应该与开发者本人对应 #### 12、头部注释与功能不对应，以下三个文件头部注释相同（已修改） 文件名： `charts/road_rate.js、car_brands.js、city_traffic.js` ``` /** * @Author: zhaojiahong * @DateTime: 2017-10-26 14:14:29 * @Description: 高速路口流量 * @Last Modified By: zhaojiahong * @Last Modified Time: */ ``` 解决方案：将头部注释与该文件的功能对应 #### 13、饼图功能未实现完，缺少value值的显示（已修改）  解决方案：增加value值的显示 #### 14、图例与折线图颜色不匹配（已修改）  解决方案：图例颜色与拆线图颜色要对应