`審核人：張紅橋` `被審核代碼負責人：趙家紅` `代碼地址：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、圖例與折線圖顏色不匹配（已修改）  解決方案：圖例顏色與拆線圖顏色要對應