`審核人：白占寧` `被審核代碼負責人：覃英琪` `代碼地址：http://192.168.1.170/hyfe/demo-project/tree/master/demo-qinyingqi` #### 1、文件命名不規范（已改） `scripts/test.js` 解決方案：`test.js`文件重命名，命名需要和具體業務相關聯。 #### 2、Mock數據引入問題（已改） `mockData/` 業務代碼里面，不需要`require`多個mock文件。 解決方案：在`mockData/`下新建一個`index.js`文件，該文件中`require`其他所有的mock文件。 #### 3、缺失必要的注釋（已改） 完善JS文件中的注釋，不僅包括各個方法及文件需要注釋，關鍵的代碼處也需要明確的注釋。 #### 4、代碼格式混亂（已改）  解決方案：嚴格遵守《JavaScript編碼規范》中的要求，該縮進的地方就縮進，注意代碼的可讀性。 #### 5、無用的`else`分支（已改） ``` .attr('fill', function(d, i) { if(i == 3) { return 'none' }else{ return '#0c1746' } }) ``` 解決方案：類似這種`if`分支中已經return了，則不再需要`else`分支，直接return即可，如下： ``` .attr('fill', function(d, i) { if(i == 3) { return 'none' } return '#0c1746' }) ``` #### 6、圖表組件封裝不合理（已改） `charts`目錄下所有圖表組件封裝不合理，參考開發框架中的示例代碼。 #### 7、CSS中class命名不規范（已改） `total_car_frame`。 解決方案：參考《CSS編碼規范》，修改為`total-car-frame`。 #### 8、CSS書寫不規范（已改） ``` .total_car_quantity{ font-family: digifacewide; font-size:35px; color:#fff; float:right; margin-right:5px; line-height:45px; display:inline-block; background:url(../images/bac_total_car_numberbac.png); width:31px; height:45px; vertical-align:middle; text-align: center; } ``` class命名、圖片命名及屬性順序都不正確。 解決方案：參考《CSS編碼規范》。 #### 9、JS中變量命名不規范（已改） ``` var svg_width = 530 var svg_height = 355 ``` 解決方案：參考《JavaScript編碼規范》。 #### 10、空`if`語句（已改） ``` if(a[i].length == 7) { //調試 }else{ a[i].unshift('') if(a[i].length == 7) { // 調試 }else{ a[i].unshift('') } } ``` 禁止出現空的if語句。 解決方案：參考《JavaScript編碼規范》。