`審核人：白占寧` `被審核代碼負責人：廖爽` `代碼地址：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、圖片命名問題（已修改） 不要以`_`下劃線形式命名。